Closed Bug 811287 Opened 12 years ago Closed 11 years ago

Intermittent test_bookmarks_html_corrupt.js | null != null

Categories

(Toolkit :: Places, defect)

x86
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla27
Tracking Status
firefox25 --- fixed
firefox26 --- fixed
firefox27 --- fixed
firefox-esr24 --- fixed

People

(Reporter: emorley, Assigned: raymondlee)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

Rev3 WINNT 6.1 mozilla-central pgo test xpcshell on 2012-11-13 03:02:59 PST for push 1b0226622e94

slave: talos-r3-w7-028

https://tbpl.mozilla.org/php/getParsedLog.php?id=16988011&tree=Firefox

{
TEST-UNEXPECTED-FAIL | c:/talos-slave/test/build/xpcshell/tests/toolkit/components/places/tests/unit/test_bookmarks_html_corrupt.js | null != null - See following stack:
JS frame :: c:\talos-slave\test\build\xpcshell\head.js :: do_throw :: line 451
JS frame :: c:\talos-slave\test\build\xpcshell\head.js :: _do_check_neq :: line 507
JS frame :: c:\talos-slave\test\build\xpcshell\head.js :: do_check_neq :: line 528
JS frame :: c:/talos-slave/test/build/xpcshell/tests/toolkit/components/places/tests/unit/test_bookmarks_html_corrupt.js :: DC_onComplete :: line 200
native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
}
Whiteboard: [orange]
Raymond, any chance you could take a look at this?
Flags: needinfo?(raymond)
Attached patch v1 (obsolete) — Splinter Review
I think it's probably a timing issue.  I have used {Promise}s to wait for the callbacks of PlacesUtils.livemarks.getLivemark and icos.getFaviconDataForPage before carrying on.  Hope this fixes the issue.
Attachment #810401 - Flags: review?(mak77)
Flags: needinfo?(raymond)
Comment on attachment 810401 [details] [diff] [review]
v1

Review of attachment 810401 [details] [diff] [review]:
-----------------------------------------------------------------

it's worth a try
Attachment #810401 - Flags: review?(mak77) → review+
Pushed to try
https://tbpl.mozilla.org/?tree=Try&rev=c086e0e33883

Hope this fixes the issue.
Assignee: nobody → raymond
Attachment #810401 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c28d6f3f4d84
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: