Closed Bug 1135317 Opened 9 years ago Closed 9 years ago

browser_change_app_handler.js is going to permafail on Windows when Gecko 38 merges to Beta

Categories

(Firefox :: Settings UI, defect)

Product:
Firefox
Component:
Settings UI
Platform:
x86_64
Windows 8.1
Type:
defect
Priority:
Not set
Severity:
normal
Points:
1

Tracking

()

Status:
VERIFIED FIXED
Milestone:
Firefox 38
Iteration:
38.3 - 23 Feb
Tracking Flags:
Tracking Status
firefox38 --- verified

People

(Reporter: RyanVM, Assigned: Gijs)

References

Details

[Tracking Requested - why for this release]: Test permafail when Gecko 38 merges to Beta.

Because the fun never ends with this test. This only affects Windows.

https://treeherder.mozilla.org/logviewer.html#?job_id=5084886&repo=try

14:55:48 INFO - 1368 INFO TEST-START | browser/components/preferences/in-content/tests/browser_change_app_handler.js
14:56:15 INFO - 1424472975042 Browser.Experiments.Experiments TRACE Experiments #0::enabled=true, true
14:56:15 INFO - 1424472975043 Browser.Experiments.Experiments TRACE Experiments #0::Registering instance with Addon Manager.
14:56:15 INFO - 1424472975044 Browser.Experiments.Experiments TRACE Experiments #0::Registering previous experiment add-on provider.
14:56:15 INFO - 1424472975044 Browser.Experiments.Experiments TRACE PreviousExperimentProvider #0::startup()
14:56:15 INFO - 1424472975045 Browser.Experiments.Experiments TRACE Experiments #0::_loadFromCache
14:56:15 INFO - 1424472975049 Browser.Experiments.Experiments TRACE Experiments #0::_loadTask finished ok
14:56:15 INFO - 1424472975050 Browser.Experiments.Experiments TRACE Experiments #0::_run
14:56:15 INFO - 1424472975050 Browser.Experiments.Experiments TRACE Experiments #0::_main iteration
14:56:15 INFO - 1424472975050 Browser.Experiments.Experiments TRACE Experiments #0::_evaluateExperiments
14:56:15 INFO - 1424472975052 Browser.Experiments.Experiments TRACE Experiments #0::_main finished, scheduling next run
15:02:16 INFO - 1369 INFO checking window state
15:02:16 INFO - 1370 INFO Entering test
15:02:16 INFO - 1371 INFO TEST-PASS | browser/components/preferences/in-content/tests/browser_change_app_handler.js | Should have at least one known handler
15:02:16 INFO - 1372 INFO Console message: 1424472975042 Browser.Experiments.Experiments TRACE Experiments #0::enabled=true, true
15:02:16 INFO - 1373 INFO Console message: 1424472975043 Browser.Experiments.Experiments TRACE Experiments #0::Registering instance with Addon Manager.
15:02:16 INFO - 1374 INFO Console message: 1424472975044 Browser.Experiments.Experiments TRACE Experiments #0::Registering previous experiment add-on provider.
15:02:16 INFO - 1375 INFO Console message: 1424472975044 Browser.Experiments.Experiments TRACE PreviousExperimentProvider #0::startup()
15:02:16 INFO - 1376 INFO Console message: 1424472975045 Browser.Experiments.Experiments TRACE Experiments #0::_loadFromCache
15:02:16 INFO - 1377 INFO Console message: 1424472975049 Browser.Experiments.Experiments TRACE Experiments #0::_loadTask finished ok
15:02:16 INFO - 1378 INFO Console message: 1424472975050 Browser.Experiments.Experiments TRACE Experiments #0::_run
15:02:16 INFO - 1379 INFO Console message: 1424472975050 Browser.Experiments.Experiments TRACE Experiments #0::_main iteration
15:02:16 INFO - 1380 INFO Console message: 1424472975050 Browser.Experiments.Experiments TRACE Experiments #0::_evaluateExperiments
15:02:16 INFO - 1381 INFO Console message: 1424472975052 Browser.Experiments.Experiments TRACE Experiments #0::_main finished, scheduling next run
15:02:16 INFO - 1382 INFO Console message: 1424473006088 Services.HealthReport.HealthReporter WARN Saved state file does not exist.
15:02:16 INFO - 1383 ERROR TEST-UNEXPECTED-TIMEOUT | browser/components/preferences/in-content/tests/browser_change_app_handler.js | application timed out after 330 seconds with no output on browser/components/preferences/in-content/tests
Flags: needinfo?(gijskruitbosch+bugs)
Blocks: 1111540
No longer blocks: 1044600
This is just because I assumed that this would ship with 38 where we plan to ship incontent prefs. I should have made that assumption explicit in the test. I'll fix this next iteration.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Iteration: --- → 39.1 - 9 Mar
Points: --- → 1
Flags: needinfo?(gijskruitbosch+bugs)
Thanks for the quick fix!
Filed bug 1135375 in order to just toggle the default, so I guess we can close this when it merges (assuming it does in time for uplift).
Keywords: leave-open
Iteration: 39.1 - 9 Mar → 38.3 - 23 Feb
https://hg.mozilla.org/mozilla-central/rev/0fd6c4c24dc9
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox38: --- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 38
Status: RESOLVED → VERIFIED
status-firefox38: fixed → verified
You need to log in before you can comment on or make changes to this bug.