Remove references from mozilla.com in our mozmill tests

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

unspecified

Firefox Tracking Flags

(firefox16 fixed, firefox17 fixed, firefox18 fixed, firefox19 fixed, firefox-esr10 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
To lower the amount of hosts we connect to for our mozmill tests we should get rid of mozilla.com. Right now only the testStopReloadButtons.js test makes use of it. We can easily port this to mozilla.org until we have a working testcase which works locally.
(Assignee)

Comment 1

5 years ago
Created attachment 675832 [details] [diff] [review]
Patch v1
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Attachment #675832 - Flags: review?(dave.hunt)
Comment on attachment 675832 [details] [diff] [review]
Patch v1

Review of attachment 675832 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the whitespace nit addressed.

::: tests/functional/testToolbar/testStopReloadButtons.js
@@ +9,4 @@
>  
>  var setupModule = function(module) {
>    module.controller = mozmill.getBrowserController();
> +  locationBar =  new toolbars.locationBar(controller);

nit: Extra whitespace.
Attachment #675832 - Flags: review?(dave.hunt) → review-
(Assignee)

Comment 3

5 years ago
http://hg.mozilla.org/qa/mozmill-tests/rev/8ae83bc17b52 (default)
status-firefox-esr10: --- → affected
status-firefox16: --- → affected
status-firefox17: --- → affected
status-firefox18: --- → affected
status-firefox19: --- → fixed
(Assignee)

Comment 4

5 years ago
No test failures so lets get this patch backported:

http://hg.mozilla.org/qa/mozmill-tests/rev/e9bea4e2f8ee (aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/0a14d82c63d3 (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/16f4876f7d86 (release)
http://hg.mozilla.org/qa/mozmill-tests/rev/4e732c0cff4e (esr10)
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-firefox-esr10: affected → fixed
status-firefox16: affected → fixed
status-firefox17: affected → fixed
status-firefox18: affected → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.