Closed
Bug 819964
Opened 12 years ago
Closed 12 years ago
Updatable plugin test hits the network
Categories
(Core Graveyard :: Plug-ins, defect)
Core Graveyard
Plug-ins
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla20
People
(Reporter: neil, Assigned: mcsmurf)
References
Details
Attachments
(1 file, 1 obsolete file)
1.47 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
test18a() tests that the updatable plugin should open up the plugin check page. However it uses the real plugin check page on mozilla.org when it should use a test page created for the purpose, or indeed just some random page that is served as part of the test harnses would do.
Running locally with wireshark, I see no evidence that this actually does hit the network (the test closes the tab as soon as it opens), but since this is dependent on other behavior that may change in the future, this is probably a good thing to fix. I think the easiest thing to do is what the safe browsing tests do. bsmedberg - asking for review as you're a build/ peer. Let me know if someone else would be better suited.
Attachment #690493 -
Flags: review?(benjamin)
Assignee | ||
Comment 2•12 years ago
|
||
Actually it probably only hits the network for the SeaMonkey version of the plugin test ;) there the test waits for the onshow() event before closing the tab (as the UI works a bit different, so we did it that way).
Comment 3•12 years ago
|
||
Can we easily change the URL used for plugin check in the test harness? Looks like it's just a matter of setting plugins.update.url in automation.py.in? That would be preferable to making changes to server-hosts, since that has the potential to affect a bunch of other tests.
Comment 4•12 years ago
|
||
Comment on attachment 690493 [details] [diff] [review] patch Yes, this seems like an odd choice if it only affects one test. Going to clear the review for now, please rerequest review if changing the pref is not an option for some reason.
Attachment #690493 -
Flags: review?(benjamin)
Assignee | ||
Comment 5•12 years ago
|
||
Looks like the correct fix to me, changing the pref does the right thing.
Attachment #690493 -
Attachment is obsolete: true
Attachment #693058 -
Flags: review?(benjamin)
Updated•12 years ago
|
Attachment #693058 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 6•12 years ago
|
||
Pushed: https://hg.mozilla.org/integration/mozilla-inbound/rev/f9ea385ca51f
Assignee: nobody → bugzilla
Target Milestone: --- → mozilla20
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f9ea385ca51f
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•