+++ This bug was initially created as a clone of Bug #730837 +++ New SeaMonkey failures on mochitest-browser-chrome: { TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/common/places/tests/browser_457473_no_copy_guid.js | Exception thrown - [Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsITransactionManager.undoTransaction]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)" location: "JS frame :: resource:///modules/PlacesUIUtils.jsm :: <TOP_LEVEL> :: line 1396" data: no] TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/common/places/tests/browser_library_views_liveupdate.js | Node is recognized as a livemark TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/common/places/tests/browser_library_views_liveupdate.js | Node is recognized as a livemark TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/common/places/tests/browser_library_views_liveupdate.js | Node is recognized as a livemark } There may be more on other suites. *** Need to port: *Tests: https://hg.mozilla.org/mozilla-central/rev/4814a8d64b8d In this case, I'm going to port tests before code (bug 730837).

Whoever reviews it first... I did not update http://mxr.mozilla.org/comm-central/find?string=test_bookmarks_html.js yet, as that file seems to need other sync' before this one.

Comment on attachment 600955 [details] [diff] [review] (Av1) Update tests now that livemarks are asynchronous [Checked in: Comment 5] PS: This is 99% copy+paste (with 2+ nits), nothing SM specific.

(In reply to Serge Gautherie (:sgautherie) from comment #1) > http://mxr.mozilla.org/comm-central/find?string=test_bookmarks_html.js > yet, as that file seems to need other sync' before this one. I filed bug 731663.

Comment on attachment 600955 [details] [diff] [review] (Av1) Update tests now that livemarks are asynchronous [Checked in: Comment 5] seems right to me r=me

Comment on attachment 600955 [details] [diff] [review] (Av1) Update tests now that livemarks are asynchronous [Checked in: Comment 5] http://hg.mozilla.org/comm-central/rev/949f8b97b255

(In reply to Serge Gautherie (:sgautherie) from comment #5) > http://hg.mozilla.org/comm-central/rev/949f8b97b255 http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1330742797.1330747356.24459.gz Linux comm-central-trunk debug test mochitest-other on 2012/03/02 18:46:37 + http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1330744452.1330748114.25476.gz Linux comm-central-trunk debug test xpcshell on 2012/03/02 19:14:12 V.Fixed wrt this patch (only).

Trivial copy+paste. (Untested.)

Comment on attachment 607037 [details] [diff] [review] (Bv1) Update test_bookmarks_html.js now that livemarks are asynchronous [Checked in: Comment 8] http://hg.mozilla.org/comm-central/rev/58b62bf5edf3

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1332414995.1332418388.9262.gz WINNT 5.2 comm-central-trunk debug test xpcshell on 2012/03/22 04:16:35 V.Fixed

Comment on attachment 607037 [details] [diff] [review] (Bv1) Update test_bookmarks_html.js now that livemarks are asynchronous [Checked in: Comment 8] Test Only. a=me

Is there a dependancy on this test I'm not aware of? $ hg qpush applying 730849-Bv1_test_bookmarks_html.diff patching file suite/common/places/tests/unit/test_bookmarks_html.js Hunk #1 FAILED at 112 Hunk #2 FAILED at 134 Hunk #3 FAILED at 160 Hunk #4 FAILED at 218 Hunk #5 FAILED at 252 Hunk #6 FAILED at 337 Hunk #7 FAILED at 365 7 out of 7 hunks FAILED -- saving rejects to file suite/common/places/tests/unit/test_bookmarks_html .js.rej patch failed, unable to continue (try -v) patch failed, rejects left in working dir errors during apply, please fix and refresh 730849-Bv1_test_bookmarks_html.diff

patching file suite/common/places/tests/unit/test_bookmarks_html.js Hunk #3 FAILED at 163 Hunk #5 FAILED at 259 2 out of 7 hunks FAILED Removing c-n request for the moment.

After bug 731663 Cv1-SM-a210 landed: patching file suite/common/places/tests/unit/test_bookmarks_html.js Hunk #3 FAILED at 163 1 out of 7 hunks FAILED

(In reply to Jens Hatlak (:InvisibleSmiley) from comment #13) > patching file suite/common/places/tests/unit/test_bookmarks_html.js > Hunk #3 FAILED at 163 > 1 out of 7 hunks FAILED That's because bug 736547 landed on c-c out of order...

Comment on attachment 617229 [details] [diff] [review] (Bv1-a210) Update test_bookmarks_html.js now that livemarks are asynchronous [Checkin: Comment 15] http://hg.mozilla.org/releases/comm-aurora/rev/a86f9d4fc2a7

(No idea about whiteboard and flags here, please check and handle yourself.)

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey-Aurora/1335230145.1335233640.17596.gz OS X 10.5 comm-aurora debug test xpcshell on 2012/04/23 18:15:45 s: cb-sea-miniosx01 seamonkey2.10: verified.