-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Cosmetic filters not always applied on specific website http://honyaku.yahoo.co.jp/ #150
Comments
I've tried with |
@gorhill I can't see |
Ok here are the steps to reproduce.
The cosmetic filters don't apply after the restart. Not sure if this is what happened to @runrunrirun. Here is a video. http://uux3.altervista.org/uBlockOriginRestart.webm
|
I can't reproduce. That will be the first thing for me do, find a case where I can consistently reproduce. @runrunrirun When the problem occurs, can you tell me if you see the same quirk as @my-password-is-password sees in the dynamic filtering pane, i.e. see if the first party domain is reported as |
@gorhill I think so. When I click to noop This is what it looks like expanding the dynamic filtering pane: |
I want @runrunrirun to check if he get the same result as you. If the domain is reported as |
@my-password-is-password By the way I cannot reproduce what you do in your video. |
@WyohKnott These are injected manually, when clicking the cosmetic filtering switch in the popup. |
@gorhill I can reproduce everytime. Maybe one of my other extensions. I really don't want to have to disable 30 addons. Dammit. |
@my-password-is-password What's your list of add-ons? I can try them one by one on my side. |
@gorhill I restarted my computer and booted Win 7 and was able to reproduce with just uBlock installed. So maybe its a windows issue. |
For uBlock to see the domain of I can't think of any reason why that would happen. Any error in the browser console about this? |
@gorhill Just this in the console, |
Browser console is Ctrl-Shift-J. |
Ok.
There was one line with a little badge to the right with the number 4 in it.
|
Alright, I can't reproduce in Windows (VM) either. So at this point I need someone with dev-fu who can reproduce to step into the code to find out why |
Actually I've just got an idea of what might be happening... I will try to provide a dev version within one hour so this can be tested. |
This problem is relatively unrelated to OP's issues though? The cosmetic filters are not applied. If I reload the page, there is no |
@WyohKnott They do for me if I don't do the restart thing. In your request log you don't see any yellow rows if you fitler for 'dom'? |
It is related. The cosmetic filters involved are specific, i.e. thet apply strictly to |
@my-password-is-password I do not. @gorhill I'm digging into |
@WyohKnott and what is being returned? |
@gorhill I can reproduce in chrome too. Steps to reproduce.
|
@gorhill |
@my-password-is-password Yes, I can reproduce this. |
New dev build released. This fixes the |
Problem however is that the domain name at cosmetic-filters-dictionary-build-time might have been the wrong one. |
It doesn't for me:
is still not applied. |
It could be that the compiled version of custom filter list needs to be re-compiled, this time with the proper domain name. I should have forced a re-compilation of all filter lists. You can force re-compilation of custom filters by just changing something in it and click apply changes. Then afterward, see if the issue is back. |
@gorhill I did think of that. Erase all cosmetics, rebooted the extension, reapplied the filters. Still not working. |
@WyohKnott I don't know what to say then, as said, I can't repro, so I can't step in the code to find out what is wrong. For sure an issue was fixed here, as this fixed the case reported by @my-password-is-password , which case could lead to specific cosmetic filters being improperly stored. If I could reproduce with your case, I would look at what the compiled filters look like for the user filter list, i.e. I would unselect all filter lists except the user one, and put a breakpoint where the compiled filter list is loaded, and look at its content ( |
@gorhill I'll do a step-by-step of getCompiledFilterList when I can. When is it triggered? |
@WyohKnott Note that the fact that with OP forcing a reload of the filter list fixes the problem in his case suggest strongly that the issue was indeed a bad domain name being used at load time. So the fix here would fix his issue. In your case you say forcing a reload of the filter list does not work? So that would be something else. Just to be sure, cosmetic filtering is not disabled for the site, neither globally? |
@gorhill No it's enabled everywhere. |
When the filter lists are reloaded, when the content of user filters changes, or when the user click Apply changes in the 3rd-party filters pane. |
@my-password-is-password Since you were able to reproduce in both cases (Firefox/Chromium), can you tell if the latest release helps? |
Details content contains some weird stuff:
But nothing about yahoo.co.jp, and I don't have these sites in my cosmetic filters, nor I have visited them (i think). Edit: oh okay, it's assets/ublock/unbreak.txt, the only other thing I have activated. |
@WyohKnott These are uBlock's own filters. They are of no interest. As said, you need to unselect all filter lists except My filters, so this way you will know that the content of |
@gorhill I tested the latest uBlock Origin dev build on both FF and Chromium and its working good now. No more |
Okay it's official I'm dumb. /no problem here anymore. |
I won't ask, but I also made mistakes. |
So I am going to guess with good confidence this was the issue. This might explain the occasional failure reported by some people, there was definitely a sort of race condition at extension load time. |
Maybe OP had the same brain f*** as me, so we'll see when they come back. |
gorhill/uBlock#150 + other fix re. tab-binding
Wow, apologies for slowpoke, wasn't expecting such fast response. Just installed 0.9.6.0 and I believe I can confirm the issue seems to be mostly fixed. The only caveat is that the cosmetic filters are still not applied if the page is being restored at browser startup (i.e. saved tab being reloaded by Firefox). This behavior may be expected, since the filtered content was actually loaded and is in the cache and may be rendered before uBlock runs. If I reload the page, filters are now applied every time without editing my local list. Personally, I can live with this. |
Browser: Firefox 37.0.2
Other extensions include: NoScript, HttpsEverywhere
Having a problem pretty much since starting to use uBlock a couple of months ago with a specific site:
http://honyaku.yahoo.co.jp/
The problem sounds similar to one that was reported as addressed in 0.9.5.1 but is still happening for me with that release. What happens is that the cosmetic filters for this site I have defined simply are not applied after a restart of Firefox. Reloading the page also does not apply the filters. However, if I do something to my filter list to mark it as changed, Apply Changes, and then reload the page above, the filters are then applied and will continue to be applied until I quit the browser.
Here is my filter list. The element picker was used to create the cosmetic rules.
classic.wunderground.com###wu4TDU > a[href="//www.wunderground.com/weather-forecast/zmw:60193.1.99999"]
eow.alc.co.jp###AreaUpperLeft > .pr_text
honyaku.yahoo.co.jp###msthd
honyaku.yahoo.co.jp###sub-contents
honyaku.yahoo.co.jp###guide-bar
honyaku.yahoo.co.jp###g-navi > .cf
honyaku.yahoo.co.jp###judg
honyaku.yahoo.co.jp###translate-exp
honyaku.yahoo.co.jp###footer
honyaku.yahoo.co.jp###dic-sult > div > div > p
honyaku.yahoo.co.jp###dic-sult > div > p
||honyaku.yahoo.co.jp/darla/2-2-4_7/html/ext-render-secure.html
honyaku.yahoo.co.jp###judg > p
translate.google.com###gt-imld-promo > .gt-imld-promo-img
classic.wunderground.com###wu4TDU
translate.google.com###gt-community-promo
translate.google.com###gba
!jisho.org##body > .row:nth-of-type(1) > .columns.large-2 > .logo > a[href="/"]
The text was updated successfully, but these errors were encountered: