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

uBO broken on Firefox mozilla-central tip #2493

Closed
rvandermeulen opened this issue Mar 31, 2017 · 4 comments
Closed

uBO broken on Firefox mozilla-central tip #2493

rvandermeulen opened this issue Mar 31, 2017 · 4 comments

Comments

@rvandermeulen
Copy link

This hasn't shipped in a nightly build yet since it was merged to mozilla-central after the 2017-03-31 nightlies were created, but bug 1145470 has broken uBO (won't even load).

The error in the browser console is:
Security Error: Content at resource://gre-resources/hiddenWindow.html may not load or link to chrome://ublock0/content/background.html#1.11.4.

@gijsk
Copy link
Contributor

gijsk commented Mar 31, 2017

Disclaimers: I wrote the change to Firefox that broke this. I also am not a regular user of uBlock Origin or any other similar ad-blocking tools, so YMMV with 'obvious' breakage that I didn't spot or whatever. I also only tested on m-c tip.

I wrote a quick patch that seems to make uBlock Origin work again on current mozilla-central tip. As github helpfully referenced above, it's at gijsk@dc06d5f . I would have done a pull request, but your Contributing.md file explicitly says not to do that, so I haven't. I'd happy to click the trivial github buttons to make that happen, of course, though you're also welcome to take that patch in some other way (and/or with tweaks). I tried to follow code style as I saw it, but I might have slipped up in places - habits are hard to shake.

@gorhill
Copy link
Owner

gorhill commented Apr 1, 2017

I would have done a pull request, but your Contributing.md file explicitly says not to do that, so I haven't.

It's really to prevent superficial and undesirable pull requests, I do actually accept pull requests from real developers which solve real issues.

@gorhill gorhill closed this as completed in a440bde Apr 1, 2017
@gorhill gorhill reopened this Apr 1, 2017
@gorhill
Copy link
Owner

gorhill commented Apr 1, 2017

@gijsk Can you submit the pull request?

I tried to create the pull request myself from your repo, but somehow as a result I couldn't see your name as the author of the commit, so I reverted it.

@gorhill
Copy link
Owner

gorhill commented Apr 1, 2017

Never mind, found this help on how to do this. Thanks for the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants