e10s -- Fix browser/base/content/test/general/ browser chrome tests relying on browser.sessionHistory

RESOLVED FIXED in Firefox 40

Status

()

RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: Gijs, Assigned: enndeakin)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 40
Points:
3
Dependency tree / graph
Bug Flags:
firefox-backlog +
in-testsuite +
qe-verify -

Firefox Tracking Flags

(e10s+, firefox40 fixed)

Details

Attachments

(1 attachment)

All of the following hang or break with something like:

629 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug816527.js | uncaught exception - TypeError: testTab.linkedBrowser.sessionHistory is null at chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug816527.js:63
Stack trace:
chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:simpletestOnerror:1346
null:null:0
630 INFO Console message: [JavaScript Error: "TypeError: testTab.linkedBrowser.sessionHistory is null" {file: "chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug816527.js" line: 63}]


[browser_bug555767.js]
[browser_bug647886.js]
[browser_bug816527.js]
Flags: qe-verify-
Flags: in-testsuite+
Flags: firefox-backlog+
Blocks: 984139
tracking-e10s: --- → +
(Assignee)

Comment 1

4 years ago
Created attachment 8597984 [details] [diff] [review]
Fix for browser_bug647886
(Assignee)

Comment 2

4 years ago
Comment on attachment 8597984 [details] [diff] [review]
Fix for browser_bug647886

Test run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=ad8793ebfea1
Attachment #8597984 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8597984 [details] [diff] [review]
Fix for browser_bug647886

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

::: browser/base/content/test/general/browser_bug647886.js
@@ +17,4 @@
>    EventUtils.synthesizeMouseAtCenter(backButton, {type: "mousedown"});
>    EventUtils.synthesizeMouse(backButton, rect.width / 2, rect.height, {type: "mouseup"});
> +  let event = yield popupShownPromise;
> +  

Nit: trailing ws
Attachment #8597984 - Flags: review?(gijskruitbosch+bugs) → review+

Updated

4 years ago
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Iteration: --- → 40.3 - 11 May

Updated

4 years ago
Points: --- → 3
https://hg.mozilla.org/mozilla-central/rev/ba0460bfab05
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
The fix that landed a few months ago only addressed one of the three test cases reported as affected. I'll file a separate bug for browser_bug555767.js, as I have been trying to fix it. It appears that browser_bug816527.js no longer exists.
You need to log in before you can comment on or make changes to this bug.