Closed Bug 1337729 Opened 7 years ago Closed 7 years ago

Reenable browser_privatebrowsing_concurrent.js for e10s

Categories

(Testing :: General, defect)

Product:
Testing
Component:
General
Version:
Version 3
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox52 unaffected, firefox-esr52 unaffected, firefox53 fixed, firefox54 fixed, firefox55 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla55
Iteration:
55.1 - Mar 20
Tracking Flags:
Tracking Status
firefox52 --- unaffected
firefox-esr52 --- unaffected
firefox53 --- fixed
firefox54 --- fixed
firefox55 --- fixed

People

(Reporter: gkrizsanits, Assigned: gkrizsanits)

References

Details

(Whiteboard: [e10s-multi:?])

Attachments

(1 file)

quickfix
2.37 KB, patch
mrbkap
: review+
Details | Diff | Splinter Review 
      No description provided.
Blocks: 1315042
Attached patch quickfix — — Splinter Review 
This is just another early return from browser loaded: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d70a62e8e18a5561c4581352dc9f303684ca34a8

It's weird that only this call has this issue... I'm not sure it is worth to look into this more given that you're working on a quite similar issue for the middle click referrer tests.
Attachment #8841923 - Flags: review?(mrbkap)
Whiteboard: [e10s-multi:?]
Attachment #8841923 - Flags: review?(mrbkap) → review+
The problem here is that browserLoaded basically can't be combined easily with any other way of waiting for a load to finish. In particular, anything that waits for STATE_STOP will race messages between the parent and the child for the onStateChange and the actual load event. This approach of fixing the intermittent is probably the most efficient way to do it.
Pushed by gkrizsanits@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9fcb7e13923d
Reenable browser_privatebrowsing_concurrent.js. r=mrbkap
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/mozilla-inbound/rev/081a720da263
Reenable browser_privatebrowsing_concurrent.js: use doublequotes to make eslint happy. r=eslint-fix
https://hg.mozilla.org/mozilla-central/rev/9fcb7e13923d
https://hg.mozilla.org/mozilla-central/rev/081a720da263
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Iteration: --- → 55.1 - Mar 20
Assignee: nobody → gkrizsanits
status-firefox52: --- → unaffected
status-firefox53: --- → affected
status-firefox54: --- → affected
status-firefox-esr52: --- → unaffected
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: