Closed Bug 1242943 Opened 4 years ago Closed 4 years ago
[e10s] Enable browser
_webconsole _clickable _urls .js for e10s
No description provided.
This test is on the e10s test list. This patch: - uses add_task - switches from loadTab to BrowserTestUtils.openNewForegroundTab TBH, I'm not sure whether this will fix the problem. This test passes in e10s mode on my local, so I'm assuming it's an intermittent. I'm not sure the best way to conclusively determine if it's fixed.
Assignee: nobody → lclark
Status: NEW → ASSIGNED
Attachment #8712142 - Flags: review?(bgrinstead)
Attachment #8712142 - Flags: review?(bgrinstead) → review+
(In reply to Lin Clark from comment #1) > TBH, I'm not sure whether this will fix the problem. This test passes in > e10s mode on my local, so I'm assuming it's an intermittent. I'm not sure > the best way to conclusively determine if it's fixed. A try push with a fair number of retriggers on all platforms on the dt chunk that this test is running in should be enough. We could also do requestLongerTimeout here if needed since the test logic is simple - it's just that there might be too many test cases for linux debug. Randomly choosing where to split up the test cases into a second test file seems worse than just requesting more time if we need it.
The comment next to the skip-if says that it's a linux debug timeout, so I don't think there's a good way to test that this change worked yet. I've blocked it on Bug 1242986.
Depends on: 1242986
I'm not sure there is a good way to test this, or whether there will be. I'm going to go ahead and check it in. If it doesn't reduce the intermittents, then we can tackle it again in another issue.
You need to log in before you can comment on or make changes to this bug.