Closed Bug 621018 Opened 9 years ago Closed 9 years ago

Some browser chrome tests in browser/base/content try to load AMO content


(Firefox :: General, defect)

Not set



Firefox 4.0b9


(Reporter: mak, Assigned: philor)



(1 file, 1 obsolete file)

these tests should define extensions.webservice.discoverURL to something local if they really need to access about:addons

No network access please!
The toolkit one is fine, because of

The fun part is that whether or not other ones are fine depends on the order the tests run in, since that clears the other two prefs it sets, but not that one.
Attached patch Fix v.1 (obsolete) — Splinter Review
A little copy, a little paste...
Assignee: nobody → philringnalda
Attachment #499690 - Flags: review?(dtownsend)
Comment on attachment 499690 [details] [diff] [review]
Fix v.1

I like it and I'd like to say yes, but really if we're just setting the pref and wanting it to stay set for hte lifetime of the tests then we should just do so in I probably should have put it there in the first place.
Attachment #499690 - Flags: review?(dtownsend) → review-
Attached patch Fix v.2Splinter Review
Sigh. I thought I remembered seeing where we set some server prefs, but I didn't see it in, so I just applied the power of my clipboard instead of my mind.
Attachment #499690 - Attachment is obsolete: true
Attachment #499691 - Flags: review?(dtownsend)
Comment on attachment 499691 [details] [diff] [review]
Fix v.2

browser_discover.js depends on PREF_DISCOVERURL being defined so define it there but otherwise this looks good.
Attachment #499691 - Flags: review?(dtownsend) → review+
Closed: 9 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b9
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.