Closed
Bug 1159497
Opened 9 years ago
Closed 9 years ago
Intermittent browser_bug427559.js | body is focused - Got button, expected body
Categories
(Firefox :: Tabbed Browser, defect)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
Firefox 40
Tracking | Status | |
---|---|---|
e10s | ? | --- |
firefox38 | --- | unaffected |
firefox38.0.5 | --- | unaffected |
firefox39 | --- | unaffected |
firefox40 | --- | fixed |
firefox-esr31 | --- | unaffected |
firefox-esr38 | --- | unaffected |
People
(Reporter: KWierso, Assigned: ttaubert)
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
3.92 KB,
patch
|
MattN
:
review+
|
Details | Diff | Splinter Review |
14:55:48 INFO - 98 INFO TEST-START | browser/base/content/test/general/browser_bug424101.js 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages http://example.org/manifest.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages https://example.com/manifest.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages http://test1.example.org/manifest.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages http://test1.example.org:8000/manifest.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages http://sub1.test1.example.org/manifest.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_priv.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_cert.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_cert.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_inst.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_priv.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages https://acertified.com/manifest.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages https://aprivileged.com/manifest.webapp 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages http://127.0.0.1:8888/sample.manifest 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:48 INFO - -*-*- Langpacks: getAdditionalLanguages http://127.0.0.1:8888/sample.manifest 14:55:48 INFO - -*-*- Langpacks: Languages found: ({langs:{}}) 14:55:49 INFO - MEMORY STAT vsize after test: 3827585024 14:55:49 INFO - MEMORY STAT residentFast after test: 600399872 14:55:50 INFO - MEMORY STAT heapAllocated after test: 144290928 14:55:50 INFO - 99 INFO TEST-OK | browser/base/content/test/general/browser_bug424101.js | took 1482ms 14:55:50 INFO - 100 INFO TEST-START | browser/base/content/test/general/browser_bug427559.js 14:55:50 INFO - 101 INFO checking window state 14:55:50 INFO - 102 INFO Console message: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must be declared in the document or in the transfer protocol." {file: "data:text/html,<body><button%20onblur="this.parentNode.removeChild(this);"><script>document.body.firstChild.focus();</script></body>" line: 0}] 14:55:50 INFO - 103 INFO TEST-PASS | browser/base/content/test/general/browser_bug427559.js | button is focused 14:55:50 INFO - 104 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug427559.js | body is focused - Got button, expected body 14:55:50 INFO - Stack trace: 14:55:50 INFO - chrome://mochikit/content/browser-test.js:test_is:925 14:55:50 INFO - chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug427559.js:focusedWindow:34 14:55:50 INFO - resource://gre/modules/RemoteAddonsParent.jsm:EventTargetParent.dispatch/<:569 14:55:50 INFO - resource://gre/modules/Prefetcher.jsm:Prefetcher.withPrefetching:458 14:55:50 INFO - resource://gre/modules/RemoteAddonsParent.jsm:EventTargetParent.dispatch:565 14:55:50 INFO - resource://gre/modules/RemoteAddonsParent.jsm:EventTargetParent.receiveMessage:513 14:55:50 INFO - null:null:0 14:55:50 INFO - MEMORY STAT vsize after test: 3826339840 14:55:50 INFO - MEMORY STAT residentFast after test: 599142400 14:55:50 INFO - MEMORY STAT heapAllocated after test: 145071432 14:55:50 INFO - 105 INFO TEST-OK | browser/base/content/test/general/browser_bug427559.js | took 242ms 14:55:50 INFO - 106 INFO TEST-START | browser/base/content/test/general/browser_bug431826.js
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 32•9 years ago
|
||
Extremely frequent e10s failure that spiked in the last couple days on OSX/Win. Any chance you can take a look, Tim? :)
tracking-e10s:
--- → ?
Flags: needinfo?(ttaubert)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 34•9 years ago
|
||
Rewrote the test and made it for e10s. Will push to try in a minute.
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
Flags: needinfo?(ttaubert)
Attachment #8599955 -
Flags: review?(jaws)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 36•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=07f3649ca4a5
Assignee | ||
Comment 37•9 years ago
|
||
Try is looking good!
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•9 years ago
|
Attachment #8599955 -
Flags: review?(jaws) → review?(MattN+bmo)
Comment 40•9 years ago
|
||
Comment on attachment 8599955 [details] [diff] [review] 0001-Bug-1159497-Fix-intermittent-browser_bug427559.js-fa.patch Review of attachment 8599955 [details] [diff] [review]: ----------------------------------------------------------------- LGTM. I see a lot of focus related code is skipped in e10s so I'm hoping this would still fail before with if the fix regresses. ::: browser/base/content/test/general/browser_bug427559.js @@ +47,3 @@ > > + // Make sure focus is given to the window because the element is now gone. > + is((yield getFocusedLocalName(browser)), "body", "body is focused"); I'm a bit confused by the comment saying "window" instead of "body" but that's leftover from before.
Attachment #8599955 -
Flags: review?(MattN+bmo) → review+
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 44•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/25259bc37ac5
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Updated•9 years ago
|
status-firefox38:
--- → unaffected
status-firefox38.0.5:
--- → unaffected
status-firefox39:
--- → unaffected
status-firefox-esr31:
--- → unaffected
status-firefox-esr38:
--- → unaffected
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•