Skip to content

Loading…

[Firefox on Android] blocking (off) when opening new tabs in Firefox from 3rd party apps #1017

Closed
Snow8 opened this Issue · 3 comments

3 participants

@Snow8

Hi,

first off all I really appreciate the hard work from the devs that worked on uBlock for Firefox on Android.

I have the following issue. Whenever I open links from 3rd party apps (in my case Press, Reddit, Twitter, and a few others) in Firefox, ublock doesnt block anything until I reload the page manually after the tab was opened. Before reloading and ublock working as intended, it says in the "menu tab" --> ublock(off).

I am running Firefox Stable on Android 5.0.1 on a Nexus 5. All other apps are up to date as well for what it is worth.

My ublock settings are:

"I am an advanced user" enabled

  • * 3p block (disabled on a per site setting when css is broken)
  • * 3p-frame block 3rd Party Filters are pretty much standard with the German List + Social blocking enabled.

If you need any further infos please let me know.

@AlexVallat
Collaborator

Confirmed. I'm investigating.

@AlexVallat AlexVallat added a commit to AlexVallat/uBlock that referenced this issue
@AlexVallat AlexVallat Fixing #1017, only apply switch to about:blank or about:newtab if not…
… loading
094e24d
@AlexVallat AlexVallat added a commit to AlexVallat/uBlock that referenced this issue
@AlexVallat AlexVallat Fixing #1017, only apply switch to about:blank or about:newtab if not…
… loading
50804eb
@AlexVallat
Collaborator

OK, some analysis here. Firstly, when Fennec is opening the new tab from the 3rd party intent you get the same effect as in #1001 where the root document is treated as being behind the scenes. I applied the same test fix for a better tab ID, but there was then a further issue that the onTabSelect handler was being called for the new tab after it had started to load, but before the .currentURI had been updated from about:blank. This resulted in the pageStore for that tab being rebound to about:blank. To resolve this, I added a check to see if the page is currently loading before calling onNavigation for about:blank (or about:newtab).

Changes in gorhill/uBlock#1021

@gorhill

Fixed with #1021.

@gorhill gorhill closed this
@dungsaga dungsaga pushed a commit that referenced this issue
@gorhill gorhill this fixes #1017 eaec2b9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.