1136910 - Intermittent browser_tab_dragdrop.js | uncaught exception - uncaught exception: undefined at :0 | Test timed out | Found a browser window

Reporter

Description

•

10 years ago

11:28:42 INFO - 791 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tab_dragdrop.js | uncaught exception - uncaught exception: undefined at :0 11:28:42 INFO - Stack trace: 11:28:42 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:simpletestOnerror:1476 11:28:42 INFO - null:null:0 11:28:42 INFO - JavaScript error: , line 0: uncaught exception: undefined 11:28:42 INFO - 792 INFO Console message: [JavaScript Warning: "unsafe CPOW usage" {file: "chrome://global/content/bindings/remote-browser.xml" line: 162}] 11:28:42 INFO - 793 INFO Console message: [JavaScript Error: "uncaught exception: undefined"] 11:28:42 INFO - 794 INFO Console message: [JavaScript Error: "Error: operation not possible on dead CPOW"] 11:29:25 INFO - libpng warning: zero length keyword 11:29:25 INFO - libpng warning: Empty language field in iTXt chunk 11:29:25 INFO - TEST-INFO | screencapture: exit 0 11:29:25 INFO - 795 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tab_dragdrop.js | Test timed out - expected PASS 11:29:25 INFO - 796 INFO MEMORY STAT vsize after test: 3790696448 11:29:25 INFO - 797 INFO MEMORY STAT residentFast after test: 671367168 11:29:25 INFO - 798 INFO MEMORY STAT heapAllocated after test: 144823592 11:29:25 INFO - 799 INFO TEST-OK | browser/base/content/test/general/browser_tab_dragdrop.js | took 45071ms 11:29:25 INFO - 800 INFO checking window state 11:29:25 INFO - 801 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tab_dragdrop.js | Found a browser window after previous test timed out - expected PASS

Comment hidden (Legacy TBPL/Treeherder Robot)

Dave Townsend [:mossop]

Assignee

Updated

•

10 years ago

Flags: needinfo?(dtownsend)

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=7228890 repository: mozilla-inbound start_time: 2015-03-04T18:12:59 who: philringnalda[at]gmail[dot]com machine: t-snow-r4-0084 buildname: Rev4 MacOSX Snow Leopard 10.6 mozilla-inbound opt test mochitest-e10s-browser-chrome-1 revision: dfaf613b7be5 790 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tab_dragdrop.js | uncaught exception - uncaught exception: undefined at :0 794 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tab_dragdrop.js | Test timed out - expected PASS 800 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_tab_dragdrop.js | Found a browser window after previous test timed out - expected PASS 1398 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_586068-select.js | Uncaught exception - [Exception... "[JavaScript Error: "child process crashed or timedout" {file: "resource:///modules/sessionstore/TabState.jsm" line: 96}]'[JavaScript Error: "child process crashed or timedout" {file: "resource:///modules/sessionstore/TabState.jsm" line: 96}]' when calling method: [nsISessionStore::setBrowserState]" nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIP 1454 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got 0, expected 1 1455 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got 0, expected 1 1456 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got , expected qux 1464 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got 0, expected 1 1465 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got 0, expected 1 1466 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got , expected qux 1473 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got 0, expected 1 1474 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got 0, expected 1 1475 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got , expected qux 1486 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got 0, expected 1 1487 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | undefined assertion name - Got 0, expected 1 1501 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | [test_setBrowserState] window3 busy event count correct - Got 0, expected 1 1502 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_615394-SSWindowState_events.js | [test_setBrowserState] window3 ready event count correct - Got 0, expected 1 1578 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_636279.js | Test timed out - expected PASS 1602 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_662812.js | uncaught exception - Error: Not a number at chrome://browser/content/tabview.js:7904 1604 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_662812.js | Test timed out - expected PASS 1609 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_662812.js | Found a tab after previous test timed out: about:mozilla - expected PASS 1717 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_cleaner.js | uncaught exception - Error: Not a number at chrome://browser/content/tabview.js:7904 1738 INFO TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_cleaner.js | uncaught exception - Error: Not a number at chrome://browser/content/tabview.js:7904 PROCESS-CRASH | Main app process exited normally | application crashed [None]

Comment hidden (Legacy TBPL/Treeherder Robot)

Jim Mathies [:jimm]

Comment 15

•

10 years ago

I just hit this on try. Above the test failure there are a ton of PBackgroundTest related ipc failures, I wonder if that has something to do with this. https://treeherder.mozilla.org/#/jobs?repo=try&revision=1e6c41b44a9d

Comment hidden (Legacy TBPL/Treeherder Robot)

Dave Townsend [:mossop]

Assignee

Comment 28

•

10 years ago

Looks like I might have mucked up fixing this test in bug 1087966 so let's correct that and remove all the CPOW traffic and see where that gets us.

Assignee: nobody → dtownsend

Iteration: --- → 39.1 - 9 Mar

Points: --- → 2

Flags: needinfo?(dtownsend) → firefox-backlog+

Marco Mucci [:MarcoM]

Updated

•

10 years ago

Status: NEW → ASSIGNED

Flags: qe-verify?

Comment hidden (Legacy TBPL/Treeherder Robot)

Dave Townsend [:mossop]

Assignee

Comment 31

•

10 years ago

Attached file MozReview Request: bz://1136910/Mossop (obsolete) (deleted) — Details

/r/5065 - Bug 1136910: Fix browser_tab_dragdrop.js to avoid CPOWs and correctly test clicks. Pull down this commit: hg pull review -r 6db7a8f3f8bb187ebfd11a14e1313b4ceeee921e

Attachment #8575325 - Flags: review?(mconley)

Dave Townsend [:mossop]

Assignee

Comment 32

•

10 years ago

Comment on attachment 8575325 [details] MozReview Request: bz://1136910/Mossop /r/5065 - Bug 1136910: Fix browser_tab_dragdrop.js to avoid CPOWs and correctly test clicks. Pull down this commit: hg pull review -r d05dbdde9eb028f331f2e193b83d1cc96f54b1e0

Dave Townsend [:mossop]

Assignee

Comment 33

•

10 years ago

The test here relies on bug 1128060

Depends on: 1128060

Marco Mucci [:MarcoM]

Updated

•

10 years ago

Iteration: 39.1 - 9 Mar → 39.2 - 23 Mar

Tracy Walker [:tracy]

Updated

•

10 years ago

Blocks: e10s-tests

Tracy Walker [:tracy]

Updated

•

10 years ago

tracking-e10s: ? → +

Comment hidden (Legacy TBPL/Treeherder Robot)

Mike Conley (:mconley) (:⚙️)

Comment 35

•

10 years ago

Comment on attachment 8575325 [details] MozReview Request: bz://1136910/Mossop https://reviewboard.mozilla.org/r/5063/#review4223 This looks good - just some questions. Thanks! ::: browser/base/content/test/general/browser_tab_dragdrop.js (Diff revision 2) > - function moveTabTo(a, b) { > +function moveTabTo(a, b) { moveTabTo is really not the right name here - that suggests that we're moving the tab elements around, not swap/closing. While you're here, can we call this swapAndCloseTab or something? ::: browser/base/content/test/general/browser_tab_dragdrop.js (Diff revision 2) > - var plugin = tabs[4].linkedBrowser.contentDocument.wrappedJSObject.body.firstChild; > + let plugin = tabs[4].linkedBrowser.contentDocument.wrappedJSObject.body.firstChild; Still using a CPOW here. Was the aim of this patch to eliminate all CPOW usage? Or just a large chunk of them? ::: browser/base/content/test/general/browser_tab_dragdrop.js (Diff revision 2) > + tabs[4].linkedBrowser.permanentKey = { foo: "bar" }; Can you mention that you're going to be checking for this at the end of the test?

Attachment #8575325 - Flags: review?(mconley) → review+

Comment hidden (Legacy TBPL/Treeherder Robot)

Dave Townsend [:mossop]

Assignee

Comment 37

•

10 years ago

https://reviewboard.mozilla.org/r/5063/#review4233 > Can you mention that you're going to be checking for this at the end of the test? Oh that's left over from debugging, I'll drop it. > Still using a CPOW here. Was the aim of this patch to eliminate all CPOW usage? Or just a large chunk of them? Ideally all but getting rid of the plugin bits looked harder so I left it for now. Had you read the changeset comment you would have seen that :p

Dave Townsend [:mossop]

Assignee

Comment 38

•

10 years ago

https://hg.mozilla.org/integration/fx-team/rev/49ff034c930e

Comment hidden (Legacy TBPL/Treeherder Robot)

Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)

Comment 40

•

10 years ago

Backed out for bc1 orange: https://treeherder.mozilla.org/logviewer.html#?job_id=2238309&repo=fx-team https://hg.mozilla.org/integration/fx-team/rev/3691c7f82b95

Flags: needinfo?(dtownsend)

Comment hidden (Legacy TBPL/Treeherder Robot)

:Gavin Sharp [email: gavin@gavinsharp.com]

Updated

•

10 years ago

Flags: qe-verify? → qe-verify-

Comment hidden (Legacy TBPL/Treeherder Robot)

(No longer employed by Mozilla) Aaron Klotz

Updated

•

10 years ago

Blocks: 1135130

Comment hidden (Legacy TBPL/Treeherder Robot)

Dave Townsend [:mossop]

Assignee

Comment 55

•

10 years ago

Attached patch fix browser_tabopen_reflows.js (deleted) — Details — Splinter Review

So fixing the browser_tab_dragdrop test was causing a new reflow to show up in browser_tabopen_reflows. What was happening is the click test in browser_tab_dragdrop was leaving the mouse cursor in the middle of the screen then when browser_tabopen_reflows opens about:newtab the mouse is conveniently hovering over a tile for dummy_page.html because by that point in the test run that page has been loaded enough to show up. This causes the link status display to appear causing a reflow while it figures out whether the mouse is overlapping it or not. The simplest solution I found was just to simulate a click outside of the tiled area in content before opening the new tab, try runs show this solves the problem. I also ended up converting this test to task based.

Flags: needinfo?(dtownsend)

Attachment #8578689 - Flags: review?(ttaubert)

Comment hidden (Legacy TBPL/Treeherder Robot)

Tim Taubert [:ttaubert] (inactive)

Comment 57

•

10 years ago

Comment on attachment 8578689 [details] [diff] [review] fix browser_tabopen_reflows.js Review of attachment 8578689 [details] [diff] [review]: ----------------------------------------------------------------- Great work figuring this out and thanks for updating the test! ::: browser/base/content/test/general/browser_tabopen_reflows.js @@ +107,5 @@ > + docShell.addWeakReflowObserver(observer); > + BrowserOpenTab(); > + > + // Wait until the tabopen animation has finished. > + yield waitForTransitionEnd() nit: missing semicolon

Attachment #8578689 - Flags: review?(ttaubert) → review+

Dave Townsend [:mossop]

Assignee

Comment 58

•

10 years ago

https://hg.mozilla.org/integration/fx-team/rev/2f0f28241140

Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)

Comment 59

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/2f0f28241140

Status: ASSIGNED → RESOLVED

Closed: 10 years ago

status-firefox39: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → Firefox 39

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Reporter

Updated

•

10 years ago

Blocks: 1087966

status-firefox37: --- → unaffected

status-firefox38: --- → unaffected

status-firefox-esr31: --- → unaffected

(No longer employed by Mozilla) Aaron Klotz

Updated

•

10 years ago

No longer blocks: 1135130

Comment hidden (Legacy TBPL/Treeherder Robot)

Dave Townsend [:mossop]

Assignee

Comment 66

•

9 years ago

Attachment #8575325 - Attachment is obsolete: true

Attachment #8619596 - Flags: review+

Dave Townsend [:mossop]

Assignee

Comment 67

•

9 years ago

Attached file MozReview Request: Bug 1136910: Fix browser_tab_dragdrop.js to avoid CPOWs and correctly test clicks. (deleted) — Details

MozReview Request: bz://1136910/Mossop 10 years ago Dave Townsend [:mossop] (deleted), text/x-review-board-request	mconley : review+	Details
fix browser_tabopen_reflows.js 10 years ago Dave Townsend [:mossop] (deleted), patch	ttaubert : review+	Details \| Diff \| Splinter Review
MozReview Request: Bug 1136910: Fix browser_tab_dragdrop.js to avoid CPOWs and correctly test clicks. 9 years ago Dave Townsend [:mossop] (deleted), text/x-review-board-request	mconley : review+	Details