Closed Bug 921935 Opened 11 years ago Closed 10 years ago

Breakdown: [e10s] re-enable tests disabled due to focus related failures

Categories

(Core :: XUL, defect)

defect
Not set
normal
Points:
5

Tracking

()

RESOLVED FIXED
Iteration:
38.3 - 23 Feb
Tracking Status
e10s m6+ ---

People

(Reporter: markh, Assigned: enndeakin)

References

(Blocks 1 open bug)

Details

eg, nsIFocusManager.getFocusedElementForWindow() will throw NS_ERROR_ILLEGAL_VALUE when called from chrome and passed a content window.
I don't think this really affects dogfooding, so I'm removing the dependency.
No longer blocks: e10s-it2
No longer blocks: fxdesktopbacklog
Flags: firefox-backlog+
Summary: [e10s] focusManager doesn't work correctly with e10s → Breakdown: [e10s] focusManager doesn't work correctly with e10s
Whiteboard: p=0 [qa-]
Blocks: 1030318
Blocks: 1083269
Flags: qe-verify-
Whiteboard: p=0 [qa-]
Points: --- → 5
Blocks: 1093642
These DevTools bugs should probably block e10s from riding to Aurora.
Blocks: 1100653
Blocks: 1100662
Blocks: 1102331
Summary: Breakdown: [e10s] focusManager doesn't work correctly with e10s → Breakdown: [e10s] focusManager.getFocusedElementForWindow doesn't work correctly with e10s
Blocks: 1125897
It's very unclear what this bug is and what needs to be done here. Neil can you take this and either fix or at least make it clear exactly what FIXED means for this bug?
Flags: needinfo?(enndeakin)
nsIFocusManager.getFocusedElementForWindow() doesn't work when passed a content window, so any callers of this will fail. This method is used to get the focused child frame within a window. We can't fix existing callers though without making this a synchronous call and waiting. Or existing callers will need to call an asynchronous version that could be implemented. I'm not sure what the original reason for the filing this bug though; that is if there was a specific caller that was broken. Mark? I guess the first step is to enumerate all of the callers and identify what changes are needed. Perhaps this should be a breakdown type of bug.
Flags: needinfo?(enndeakin) → needinfo?(mhammond)
This bug was opened to track the disabling of tests with e10s. It looks like 8 remain disabled - https://dxr.mozilla.org/mozilla-central/search?q=921935&case=false. I haven't looked into these too deeply - I just annotated the manifest with this bug number when ever a test looked like some method with "focus" in the name caused the test failure.
Flags: needinfo?(mhammond)
OK, so some of these tests are just expecting waitForFocus to work. That was fixed by bug 1083269, so these tests can be re-enabled, however there see to be other issues that prevent the tests from working: Tests: browser/base/content/test/general/browser_bug749738.js - appears to just work as is browser/base/content/test/general/browser_tabfocus.js - this is the main test for focus navigation in tabs. It will need to be adapted. browser/base/content/test/general/browser_typeAheadFind.js browser/base/content/test/general/browser_visibleFindSelection.js - minor changes needed to fix focus issues but these might rely on some findbar initialization timing browser/devtools/markupview/test/head.js - appears to have been rewritten to not use waitForFocus. This could just be undone. browser/devtools/storage/test/head.js - appears to have been rewritten to not use waitForFocus. This could just be undone. toolkit/content/tests/browser_bug295977_autoscroll_overflow.js toolkit/content/tests/browser_keyevents_during_autoscrolling.js - child process crashes when running these tests
Summary: Breakdown: [e10s] focusManager.getFocusedElementForWindow doesn't work correctly with e10s → Breakdown: [e10s] re-enable tests disabled due to focus related failures
Blocks: 1131562
Blocks: 1102015
Blocks: 1131575
Blocks: 1131579
Blocks: 1131626
Depends on: 1094252
Blocks: 1131656
Blocks: 1094252
No longer depends on: 1094252
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Iteration: --- → 38.3 - 23 Feb
I added some bugs and commented on each indicating what needs to be changed in order to get the test to work. This list is the focus related bugs that are disabled. Bug 1080801 contains other disabled browser tests. Bug 1131656 in a non-test bug that will fix or help fix some of these tests.
As a breakdown bug, this should be complete.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.