531227 - mochitest-browser-chrome: browser_bookmarksProperties.js times out intermittently

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, defect)

Description

[Timothy Nikkel (:tnikkel)]

Linux mozilla-1.9.2 test everythingelse
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.6-Unittest/1259197158.1259200635.4415.gz

TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | About to focus the autocomplete results tree
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Tag on node has not changed
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | End of test: Bug 462662 - Pressing Enter to select tag from autocomplete closes bookmarks properties dialog
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Start of test: Bug 475529 - Add button in new folder dialog not default anymore
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'zoom'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 90}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Error in parsing value for 'letter-spacing'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 347}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 406}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 44}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 49}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-top-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 53}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property '-moz-border-top-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 54}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-bottom-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 57}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property '-moz-border-bottom-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 58}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Error: "gEditItemOverlay is null" {file: "chrome://browser/content/places/bookmarkProperties.xul" line: 1}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Error: "gEditItemOverlay is null" {file: "chrome://browser/content/places/bookmarkProperties.xul" line: 1}]
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Sidebar tree has been loaded
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | We have a places node selected: Unsorted Bookmarks
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js |  command 'placesCmd_new:folder' on current selected node is enabled
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | EditItemOverlay is initialized
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | About to focus the namePicker field
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Folder name has been edited
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | End of test: Bug 475529 - Add button in new folder dialog not default anymore
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Start of test: Bug 476020 - Pressing Esc while having the tag autocomplete open closes the bookmarks panel
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Correctly added a bookmark
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Correctly added a tag
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Sidebar tree has been loaded
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Bookmark has been selected
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | We have a places node selected: bookmark/http://www.mozilla.org/
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js |  command 'placesCmd_show:info' on current selected node is enabled
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | EditItemOverlay is initialized
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | About to focus the tagsField
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Autocomplete results tree exists
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | We have 1 autocomplete result
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | We have selected a tag from the autocomplete popup
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | About to focus the autocomplete results tree
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Tag on node has not changed
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | End of test: Bug 476020 - Pressing Esc while having the tag autocomplete open closes the bookmarks panel
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Start of test:  Bug 491269 - Test that editing folder name in bookmarks properties dialog does not accept the dialog
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | http://www.mozilla.org/ is a visited url.
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'zoom'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 90}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Error in parsing value for 'letter-spacing'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 347}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 406}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 44}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 49}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-top-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 53}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property '-moz-border-top-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 54}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-bottom-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 57}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property '-moz-border-bottom-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 58}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Error: "gEditItemOverlay is null" {file: "chrome://browser/content/places/bookmarkProperties.xul" line: 1}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Error: "gEditItemOverlay is null" {file: "chrome://browser/content/places/bookmarkProperties.xul" line: 1}]
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Sidebar tree has been loaded
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | The correct visit has been selected
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | The selected node is not bookmarked
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | We have a places node selected: www.mozilla.org
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js |  command 'placesCmd_createBookmark' on current selected node is enabled
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | EditItemOverlay is initialized
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | We are editing new folder name in folder tree
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | We have finished editing folder name in folder tree
TEST-PASS | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Dialog window should not be closed by pressing ESC in folder name textbox
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | End of test:  Bug 491269 - Test that editing folder name in bookmarks properties dialog does not accept the dialog
Waiting for window activation...
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'zoom'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 90}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Error in parsing value for 'letter-spacing'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 347}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/screen.css" line: 406}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 44}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 49}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-top-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 53}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property '-moz-border-top-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 54}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property 'border-bottom-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 57}]
TEST-INFO | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Console message: [JavaScript Warning: "Unknown property '-moz-border-bottom-right-radius'.  Declaration dropped." {file: "http://www.mozilla.org/style/enhanced.css" line: 58}]

command timed out: 1200 seconds without output, killing pid 4381

Comment 1

[Timothy Nikkel (:tnikkel)]

Related to bug 496058?

Comment 2

[Marco Bonardo [:mak]]

(In reply to comment #1)
> Related to bug 496058?

doesn't look like

I see:
"Waiting for window activation..."

this means all the tests are complete, and browser-test is waiting for the browser windows for the next test. that is bug 521282

Comment 3

[Marco Bonardo [:mak]]

notice that this error could appear for any browser-chrome test, and will be wrongly reported as an error in the previous test. this is uncool.

Comment 4

[Dão Gottwald [:dao]]

Since browser_bookmarksProperties.js opens modal dialogs and uses the window watcher, I suspect it's a bug in that test. See also bug 530906.

Comment 5

[Marco Bonardo [:mak]]

when we are here:

| End of test:  Bug 491269 - Test that editing folder name in bookmarks
properties dialog does not accept the dialog

All tests have been executed, and we must have received "domwindowclosed" otherwise the test would timeout before, that means the dialog is effectively closed. Before replacing ww usage, i'd like to understand what happens, if i received domwindowclosed and i call finish, there should be no leftover.

Comment 6

[Marco Bonardo [:mak]]

could be due to the fact we call finish() in domwindowclosed instead of enqueuing the call?

Comment 7

[Dão Gottwald [:dao]]

Attachment: fix window watcher use

getting domwindowclosed doesn't mean you got it for the window you were interested in

Comment 8

[Marco Bonardo [:mak]]

could you also
replace runNextTest() with executeSoon(runNextTest); just to be sure we don't finish or start next test while receiving an event.

Comment 9

[Dão Gottwald [:dao]]

Attachment: fix window watcher use

executeSoon added

Comment 10

[Marco Bonardo [:mak]]

Comment on attachment 414710 [details] [diff] [review]
fix window watcher use

yay, thanks!

Comment 11

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/8ae42c1b8d83

Comment 12

[Dão Gottwald [:dao]]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/124a37409f56

Comment 13

[Gervase Markham [:gerv]]

Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv

Comment 14

[Dão Gottwald [:dao]]

I guess this is fixed...

Comment 15

[Legacy TBPL/Treeherder Robot]

jorendorff%mozilla.com
http://tinderbox.mozilla.org/showlog.cgi?log=TraceMonkey/1275635415.1275636132.31248.gz
Linux tracemonkey opt test mochitest-other on 2010/06/04 00:10:15

s: moz2-linux-slave29
7444 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/toolkit/content/tests/chrome/test_bug418874.xul | undo correctly enabled when placeholder was not changed through property - got false, expected true
7445 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/toolkit/content/tests/chrome/test_bug418874.xul | undo correctly enabled when placeholder explicitly changed through property - got false, expected true
PROCESS-CRASH | automation.py | application crashed (minidump found)
Thread 0 (crashed)
PROCESS-CRASH | automation.py | application crashed (minidump found)
Thread 0 (crashed)
TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | Timed out

Comment 16

[Legacy TBPL/Treeherder Robot]

mbrubeck%mozilla.com
https://tbpl.mozilla.org/php/getParsedLog.php?id=7958691&tree=Firefox
Rev3 MacOSX Leopard 10.5.8 mozilla-central opt test mochitest-other on 2011-12-15 10:51:57

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/places/tests/browser/browser_bookmarksProperties.js | an unexpected uncaught JS exception reported through window.onerror - result is null at chrome://browser/content/places/tree.xml:619