956202 - Intermittent TEST-UNEXPECTED-FAIL | browser_UITour.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort.

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

Reporter

Description

•

11 years ago

https://tbpl.mozilla.org/php/getParsedLog.php?id=32490453&tree=Mozilla-Inbound slave: talos-r4-snow-127 14:27:10 INFO - ++DOCSHELL 0x1733bc0b0 == 46 [pid = 913] [id = 6798] 14:27:10 INFO - ++DOMWINDOW == 374 (0x14fdbfa58) [pid = 913] [serial = 17199] [outer = 0x0] 14:27:10 INFO - ++DOMWINDOW == 375 (0x149b672d8) [pid = 913] [serial = 17200] [outer = 0x14fdbfa58] 14:27:10 INFO - [Parent 913] WARNING: NS_ENSURE_TRUE(mMutable) failed: file ../../../../netwerk/base/src/nsSimpleURI.cpp, line 265 14:27:10 INFO - [Parent 913] WARNING: NS_ENSURE_TRUE(mMutable) failed: file ../../../../netwerk/base/src/nsSimpleURI.cpp, line 265 14:27:10 INFO - ++DOMWINDOW == 376 (0x154f910d8) [pid = 913] [serial = 17201] [outer = 0x14fdbfa58] 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | Menu should initially be closed 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | Menu should be shown after showMenu() 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | Menu should be closed after hideMenu() 14:27:11 INFO - [Parent 913] WARNING: NS_ENSURE_TRUE(mMutable) failed: file ../../../../netwerk/base/src/nsSimpleURI.cpp, line 265 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | Element should not be null, when checking visibility 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | Highlight should be closed/hidden after UITour tab is closed 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | Element should not be null, when checking visibility 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | Tooltip should be closed/hidden after UITour tab is closed 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | @noautohide on the menu panel should have been cleaned up 14:27:11 INFO - TEST-PASS | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | Any pinned tab should be closed after UITour tab is closed 14:27:14 INFO - --DOCSHELL 0x171117c60 == 45 [pid = 913] [id = 6787] 14:27:14 INFO - --DOCSHELL 0x16ce120c0 == 44 [pid = 913] [id = 6786] 14:27:14 INFO - --DOCSHELL 0x1726a8ba0 == 43 [pid = 913] [id = 6785] 14:27:14 INFO - --DOCSHELL 0x16ee30f10 == 42 [pid = 913] [id = 6784] 14:27:14 INFO - --DOCSHELL 0x171f93f20 == 41 [pid = 913] [id = 6772] 14:27:14 INFO - --DOCSHELL 0x1728d80a0 == 40 [pid = 913] [id = 6771] 14:27:14 INFO - --DOCSHELL 0x171980bd0 == 39 [pid = 913] [id = 6770] 14:27:14 INFO - --DOCSHELL 0x1757eaaf0 == 38 [pid = 913] [id = 6782] 14:27:14 INFO - --DOCSHELL 0x16e800cd0 == 37 [pid = 913] [id = 6781] 14:27:14 INFO - --DOCSHELL 0x1734b3780 == 36 [pid = 913] [id = 6780] 14:27:14 INFO - --DOCSHELL 0x170d7ef00 == 35 [pid = 913] [id = 6779] 14:27:14 INFO - --DOCSHELL 0x175a60850 == 34 [pid = 913] [id = 6778] 14:27:14 INFO - --DOCSHELL 0x16e6c49c0 == 33 [pid = 913] [id = 6777] 14:27:14 INFO - --DOCSHELL 0x1747918e0 == 32 [pid = 913] [id = 6776] 14:27:14 INFO - --DOCSHELL 0x1715ceb00 == 31 [pid = 913] [id = 6775] 14:27:14 INFO - --DOCSHELL 0x1711309f0 == 30 [pid = 913] [id = 6774] 14:27:14 INFO - --DOCSHELL 0x17285d640 == 29 [pid = 913] [id = 6773] 14:27:14 INFO - --DOCSHELL 0x16e7395c0 == 28 [pid = 913] [id = 6769] 14:27:14 WARNING - TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. 14:27:14 INFO - TEST-INFO | MEMORY STAT vsize after test: 4418306048 14:27:14 INFO - TEST-INFO | MEMORY STAT residentFast after test: 957030400 14:27:14 INFO - INFO TEST-END | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | finished in 32805ms

Comment hidden (Legacy TBPL/Treeherder Robot)

Ed Morley [:emorley]

Comment 8

•

11 years ago

Blair, please may you take a look at this? (Started ~3rd Jan)

Assignee: nobody → bmcbride

Blocks: fx-UITour

Flags: needinfo?(bmcbride)

Comment hidden (Legacy TBPL/Treeherder Robot)

RyanVM https://tbpl.mozilla.org/php/getParsedLog.php?id=32591828&tree=Mozilla-Inbound Rev3 Fedora 12x64 mozilla-inbound debug test mochitest-browser-chrome on 2014-01-06 09:48:25 revision: 5568609fa907 slave: talos-r3-fed64-049 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_946320_tabs_from_other_computers.js | Uncaught exception - Subview (PanelUI-history) did not show within 20 seconds. TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify starts with 0 logins - Got 2, expected 0 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 3, expected 1 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 4, expected 2 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 5, expected 3 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 6, expected 4 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 7, expected 5 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login count - Got 7, expected 5 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login modified - Got mysecretpw, expected newpassword0 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login removed - Got 6, expected 4 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login order

Blair McBride [:Unfocused] (UNAVAILABLE)

Comment 23

•

11 years ago

Huh. We must have been right at the threshold on OSX and Linux, as the only thing that we landed before that was on the 10th December.

Status: NEW → ASSIGNED

Flags: needinfo?(bmcbride)

Comment hidden (Legacy TBPL/Treeherder Robot)

RyanVM https://tbpl.mozilla.org/php/getParsedLog.php?id=32593784&tree=Fx-Team Rev3 Fedora 12x64 fx-team debug test mochitest-browser-chrome on 2014-01-06 10:30:12 revision: 1a5a6071b1a3 slave: talos-r3-fed64-040 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/customizableui/test/browser_946320_tabs_from_other_computers.js | Uncaught exception - Subview (PanelUI-history) did not show within 20 seconds. TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/modules/test/browser_UITour.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify starts with 0 logins - Got 2, expected 0 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 3, expected 1 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 4, expected 2 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 5, expected 3 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 6, expected 4 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login added - Got 7, expected 5 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login count - Got 7, expected 5 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login modified - Got mysecretpw, expected newpassword0 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login removed - Got 6, expected 4 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/passwordmgr/test/browser/browser_passwordmgr_observers.js | Verify login order

Comment hidden (Legacy TBPL/Treeherder Robot)

Carsten Book [:Tomcat]

Comment 59

•

11 years ago

also backedout bug 935815 again, since after landing this bug occured again

Comment hidden (Legacy TBPL/Treeherder Robot)

:Gijs (he/him)

Assignee

Comment 108

•

11 years ago

Attached patch split off UITour checking functions, — Details — Splinter Review

So let's start off by making splitting up easier...

Attachment #8358002 - Flags: review?(bmcbride)

:Gijs (he/him)

Assignee

Updated

•

11 years ago

Assignee: bmcbride → gijskruitbosch+bugs

Blair McBride [:Unfocused] (UNAVAILABLE)

Comment 109

•

11 years ago

Comment on attachment 8358002 [details] [diff] [review] split off UITour checking functions, Review of attachment 8358002 [details] [diff] [review]: ----------------------------------------------------------------- <3

Attachment #8358002 - Flags: review?(bmcbride) → review+

:Gijs (he/him)

Assignee

Comment 110

•

11 years ago

Attached patch split off some of the UITour tests, (obsolete) — Details — Splinter Review

... and then split them up.

Attachment #8358007 - Flags: review?(bmcbride)

:Gijs (he/him)

Assignee

Comment 111

•

11 years ago

Attached patch split off some of the UITour tests, — Details — Splinter Review

Slightly too fast, this time with the requisite manifest addition. And, I checked this locally, and it runs the tests and passes here.

Attachment #8358010 - Flags: review?(bmcbride)

:Gijs (he/him)

Assignee

Updated

•

11 years ago

Attachment #8358007 - Attachment is obsolete: true

Attachment #8358007 - Flags: review?(bmcbride)

Comment hidden (Legacy TBPL/Treeherder Robot)

Blair McBride [:Unfocused] (UNAVAILABLE)

Comment 113

•

11 years ago

Comment on attachment 8358010 [details] [diff] [review] split off some of the UITour tests, Review of attachment 8358010 [details] [diff] [review]: ----------------------------------------------------------------- <3

Attachment #8358010 - Flags: review?(bmcbride) → review+

:Gijs (he/him)

Assignee

Comment 114

•

11 years ago

remote: https://hg.mozilla.org/integration/fx-team/rev/610bb7362337 remote: https://hg.mozilla.org/integration/fx-team/rev/e9c395d0aec5

Comment hidden (Legacy TBPL/Treeherder Robot)

:Gijs (he/him)

Assignee

Updated

•

11 years ago

Attachment #8358002 - Flags: checkin+

:Gijs (he/him)

Assignee

Updated

•

11 years ago

Attachment #8358010 - Flags: checkin+

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Comment 137

•

11 years ago

https://hg.mozilla.org/mozilla-central/rev/610bb7362337 https://hg.mozilla.org/mozilla-central/rev/e9c395d0aec5 Let's leave this open for a couple days before resolving.

Comment hidden (Legacy TBPL/Treeherder Robot)

Matthew N. [:MattN]

Updated

•

11 years ago

Status: ASSIGNED → RESOLVED

Closed: 11 years ago

Resolution: --- → FIXED

Target Milestone: --- → Firefox 29

Ryan VanderMeulen [:RyanVM]

Updated

•

11 years ago

status-firefox27: --- → unaffected

status-firefox28: --- → unaffected

status-firefox29: --- → fixed

status-firefox-esr24: --- → unaffected

Matthew N. [:MattN]

Updated

•

11 years ago

Depends on: 967370

split off UITour checking functions, 11 years ago :Gijs (he/him) 4.43 KB, patch	Unfocused : review+ Gijs : checkin+	Details \| Diff \| Splinter Review
split off some of the UITour tests, 11 years ago :Gijs (he/him) 16.36 KB, patch		Details \| Diff \| Splinter Review
split off some of the UITour tests, 11 years ago :Gijs (he/him) 16.78 KB, patch	Unfocused : review+ Gijs : checkin+	Details \| Diff \| Splinter Review