Closed Bug 1070835 Opened 10 years ago Closed 10 years ago

browser_toolbox_window_reload_target.js fails in e10s

Categories

(DevTools :: Framework, defect)

defect
Not set
normal
Points:
2

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 35
Iteration:
35.2

People

(Reporter: markh, Assigned: markh)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

browser_toolbox_window_reload_target.js attaches load event handlers directly to the selectedBrowser, so sees events coming from the CPOW.  For some reason, it sees spurious events, so the test fails with incorrect reload counts.

Converting this test to use a frameScript and count the load events via that allows the test to pass.
Attachment #8492879 - Flags: review?(past)
Comment on attachment 8492879 [details] [diff] [review]
0002-Bug-XXXXXXX-allow-browser_toolbox_window_reload_targ.patch

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

OK.
Attachment #8492879 - Flags: review?(past) → review+
https://hg.mozilla.org/integration/fx-team/rev/157ef8d592ca
Status: NEW → ASSIGNED
Points: --- → 2
Flags: qe-verify-
Flags: firefox-backlog+
Assignee: nobody → mhammond
Iteration: --- → 35.2
https://hg.mozilla.org/mozilla-central/rev/157ef8d592ca
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: