Closed Bug 1289723 Opened 3 years ago Closed 3 years ago
[e10s-multi] We need a new test for the multiple content process cases to extend browser
_crashed Tabs .js
In Bug 1285941 the decision was to keep the old tests for the cases where all the tabs are from the same process and invent some new tests for the cases where the tabs are from different content processes.
Whiteboard: [e10s-multi:?] → [e10s-multi:M1]
Sort this bug as ss-reliability(Bug 1330633), however could be ss-perf(Bug 1330635) as well from e10s standpoint.
Depends on: ss-reliability
I think the tests Mike added for bg tabs should already cover this bug (browser_background_tab_crash.js). I've enabled multi for this test with some fixes, and I experienced an intermittent exception that can cause failures so I guarded those too.
Comment on attachment 8835396 [details] [diff] [review] fixing tabcrash test for multi The test changes look good to me. I'm not a peer in browser/, though, so I'm deferring that part of the review to Felipe.
Attachment #8835396 - Flags: review?(felipc) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/59d4978e8726 Fixing crashedTabs test for e10s-multi. r=mrbkap
backed out for eslint failure like https://treeherder.mozilla.org/logviewer.html#?job_id=78995572&repo=mozilla-inbound
Backout by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/8babccb56f91 Backed out changeset 59d4978e8726 for eslint failure
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/2df453569f57 Fixing crashedTabs test for e10s-multi. r=mrbkap r=felipc
You need to log in before you can comment on or make changes to this bug.