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

fresh installation of RP on Firefox 4.0b7+ broken #142

Closed
jsamuel opened this issue Dec 22, 2011 · 3 comments
Closed

fresh installation of RP on Firefox 4.0b7+ broken #142

jsamuel opened this issue Dec 22, 2011 · 3 comments
Labels

Comments

@jsamuel
Copy link
Member

@jsamuel jsamuel commented Dec 22, 2011

imported trac ticket
created: 2010-11-05 11:45:15
reporter: justin

It appears that installing RP for the first time on Fx 4.0b7+ results in a non-working RP. There's an error about the use of __LOCATION__. Probably related, Fx now keeps the xpi as an archive on disk, not extracted. This may explain why an existing RP install continues working after upgrading Fx: the xpi is extracted and __LOCATION__ works.

The full error:

{{{
Error: LOCATION is not defined
Source File: jar:file:///path/to/.mozilla/firefox/asfdsafd.profilename/extensions/requestpolicy@requestpolicy.com.xpi!/components/requestpolicyService.js
Line: 432
}}}

@jsamuel

This comment has been minimized.

Copy link
Member Author

@jsamuel jsamuel commented Dec 22, 2011

imported trac comment
created: 2010-11-05 11:51:24
author: justin

Forgot to say: Thanks to Aleksej for the report.

@jsamuel

This comment has been minimized.

Copy link
Member Author

@jsamuel jsamuel commented Dec 22, 2011

imported trac comment
created: 2010-11-06 15:21:04
author: justin

Fixed in r361. I have a feeling the reason I didn't use this solution before was probably because I wasn't adding the correct resource line to chrome.manifest when I had tried. I probably was using a jar: reference even though the modules directory isn't in requestpolicy.jar (which only includes content/, locale/ and skin/).

@jsamuel

This comment has been minimized.

Copy link
Member Author

@jsamuel jsamuel commented Dec 22, 2011

imported trac comment
created: 2010-11-07 11:52:52
author: justin

Thanks also to [http://forums.mozillazine.org/viewtopic.php?p=10075793#p10075793 aabbqrstp] for the initial report.

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

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.