Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Frequent random extension crashes after updating to Chrome Version 72.0.3595.2 #294

Closed
8 tasks done
CollinChaffin opened this issue Nov 1, 2018 · 12 comments
Closed
8 tasks done
Labels
Chromium specific to Chromium/Chrome external issue involving an external factor invalid not a uBlock issue

Comments

@CollinChaffin
Copy link

Prerequisites

  • I verified that this is not a filter issue
  • This is not a support issue or a question
  • I performed a cursory search of the issue tracker to avoid opening a duplicate issue
    • Your issue may already be reported.
  • I tried to reproduce the issue when...
    • uBlock Origin is the only extension
    • uBlock Origin with default lists/settings
    • using a new, unmodified browser profile
  • I am running the latest version of uBlock Origin
  • I checked the documentation to understand that the issue I report is not a normal behavior

Description

I've been using ublock for many years and always run DEV ver of Chrome (I am a DEV myself) and since the update to Version 72.0.3595.2 (Official Build) dev (64-bit) a few days ago, ublock origin is the only extension to very frequently and apparently randomly (even sitting idle with Chrome in background) pop open this notification:

snag_11-1-2018_01-29-25

Which then requires clicking or manually reloading the extension. I will be happy to help debug but uncertain which/if any specific crash logs exist.

A specific URL where the issue occurs

Unknown even happens idle I believe it last happened with only a google search page open, but note this is happening suddenly on multiple machines (all only after they updated to this specific Chrome DEV version) that all obviously have different configs/profiles/etc. Only the Chrome version appears to be common.

Steps to Reproduce

  1. Open Chrome
  2. Wait a random amount of time until receiving the extension crash message

Expected behavior:

No crash.

Actual behavior:

snag_11-1-2018_01-29-25

Your environment

  • uBlock Origin version: 1.17.0
  • Browser Name and version: Chrome Version 72.0.3595.2 (Official Build) dev (64-bit)
  • Operating System and version: Windows 7 x64
@uBlock-user
Copy link
Contributor

uBlock-user commented Nov 1, 2018

Yes, dev and canary builds have regressions that are causing the crash, happens with uMatrix and other extensions too. Downgrade to Stable or Beta builds.

@uBlock-user uBlock-user added invalid not a uBlock issue Chromium specific to Chromium/Chrome labels Nov 1, 2018
@CollinChaffin
Copy link
Author

Unfortunately that is not an option for me and probably many others that require at least DEV builds. Any idea/word on the cause and do you know (I'm searching now again as well but did not see it) if there is a specific open Chromium issue on this I can join?

Thanks!

@uBlock-user
Copy link
Contributor

do you know (I'm searching now again as well but did not see it) if there is a specific open Chromium issue on this I can join?

Sorry, don't know, you can report the crash on Chromium tracker, so the devs can look at it, if downgrade is not an option then you will have to live it with it until it gets fixed.

Any idea/word on the cause

For me it began with 72.0.3594.0 and happened with 72.0.3592.0 too downgrading a bit back should work as a workaround for the time being.

Try - https://github.com/RobRich999/Chromium_Clang/releases/tag/v72.0.3587.0-r601406-win64

That build is no-sync, so you will need Google API key to make use of Google specific services.

@gorhill
Copy link
Member

gorhill commented Nov 1, 2018

If you start Chrome from the command line, does it leave information in the shell after uBO crashes?

@rprimus
Copy link

rprimus commented Nov 1, 2018

Thu Nov 1 13:39:54 GMT 2018

Environment:
OS: macOS 10.14.1 18B75
Chrome: Version 72.0.3595.2 (Official Build) dev (64-bit)
Chrome flags enabled:

  • Fast tab/window close
  • Strict site isolation

I've also experienced the random crashes when using the dev version (1.17.3.4).
I then started using the stable version (1.17.0) - which also randomly crashes.

Here's the output from the command line:

#
# Fatal error in , line 0
# Check failed: size <= kMaxRegularHeapObjectSize.
#
#
#
#FailureMessage Object: 0x7ffeee43cca00   Google Chrome Framework             0x00000001049d99bf ChromeMain + 41792447
1   Google Chrome Framework             0x00000001079c891b ChromeMain + 92054299
2   Google Chrome Framework             0x00000001079b8ef5 ChromeMain + 91990261
3   Google Chrome Framework             0x0000000103e04258 ChromeMain + 29383768
4   Google Chrome Framework             0x0000000104089eeb ChromeMain + 32028907
5   Google Chrome Framework             0x000000010405de7c ChromeMain + 31848572

@gorhill
Copy link
Member

gorhill commented Nov 1, 2018

Best is to report to Chromium devs, along with a crash report. uBO is HTML/CSS/JS (i.e. should not hard crash), and since it works fine on stable versions of Chromium, this suggest a regression in dev build of Chromium, which only Chromium devs can address.

@CollinChaffin
Copy link
Author

Yes but I reported it here knowing it most likely is Chromium related and hoping you either had a workaround or already were or might want to help get involved as your report/involvement as author of such a major extension.

Let's remember that if we all just ignore it, it's not like the DEV version now crashing is just a pipe-dream "maybe"......it's an already-implemented reality that somehow has already survived through CANARY and WILL become the stable very soon and if we all wait until then.....well then we're scrambling to get a Chromium DEV even assigned let alone tracing and implementing the fix and in my experience that itself may become the real pipe dream. And while all that is happening, the "appearance" right or wrong to most of the masses seeing that cute little "uBlock has crashed" will leave them not knowing any better than to wrongfully complain about uBlock or worse yet all come here to open issues on it. :)

And you are absolutely correct I have now begun to see similar crashing with a few other extensions. I am baffled how as bad as it is that this issue made it this far past CANARY and that there is not an active open issue on it....unless my search skills are just off today which is why I asked if you happened to have the issue # and hoping you had already complained. 😉

@uBlock-user
Copy link
Contributor

uBlock-user commented Nov 1, 2018

Just an FYI - Canary builds are not auto-tested by Google build bots, neither manually tested via simulations, so browser crashes and extensions breakage should be expected and Google does warn about this on their chromium page too. Dev builds on the other hand do get tested, but google warns to expect bugs. The reason I don't report these things because they always get patched in the next/upcoming versions of dev and canary.

So just beware, these kind of things will occur in the future builds too as these builds are only meant for testing and not meant to be used for everyday browsing. You could report it, but in the long game it's a pain in the ass and crashes occurr every few new builds in Canary and dev. Sometimes it takes time and goes on for few builds and sometimes it gets quickly patched in the next build.

@CollinChaffin
Copy link
Author

CollinChaffin commented Nov 1, 2018

Thanks for replying so quickly. FYI, I opened this issue and would urge anyone else experiencing this to comment and star the issue as well:

https://bugs.chromium.org/p/chromium/issues/detail?id=901010

EDIT: Sorry I should have qualified that I'm not necessarily doing all my browsing with DEV branch but I'm a DEV too so have the need and I've run this way for years and do remember the last bad build where Chrome wouldn't even launch etc. but like you said that only lasted a single build and was caught fast hopefully this will be too now!

@uBlock-user
Copy link
Contributor

uBlock-user commented Nov 2, 2018

Btw, I don't experience this issue on 32 bit build of 72.0.3594.0 so you can use 32 bit builds until this gets fixed.

@CollinChaffin
Copy link
Author

FYI, not sure if you looked but others already also chimed in to provide detail and looks like it just made it into a revert canary commit has to do with fastpath array they were able to see ublock crashing here is the revert commit:

https://chromium.googlesource.com/v8/v8/+/f88a1d87d681adfb80be22c2f2af54852df7f92c

@uBlock-user uBlock-user added the external issue involving an external factor label Nov 3, 2018
@gorhill
Copy link
Member

gorhill commented Nov 3, 2018

From their findings, most likely the crash was occurring when uBO was trying to save a selfie -- as this causes Array.from to be called with large Set (the more filter lists, the larger). A selfie is generated after a few minutes the filter lists have reloaded (due to update, changes, etc.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chromium specific to Chromium/Chrome external issue involving an external factor invalid not a uBlock issue
Projects
None yet
Development

No branches or pull requests

4 participants