Closed Bug 873046 Opened 6 years ago Closed 6 years ago

Intermittent browser_thumbnails_background.js | Capture timed out so thumbnail should not be cached: c:\docume~1\cltbld\locals~1\temp\tmpu05f4p\thumbnails\e82891364309427ebb2a6931429ff068.png

Categories

(Firefox :: General, defect)

x86
Windows XP
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 24

People

(Reporter: RyanVM, Assigned: adw)

References

Details

(Keywords: intermittent-failure)

https://tbpl.mozilla.org/php/getParsedLog.php?id=23007990&tree=Mozilla-Inbound

Rev3 WINNT 5.1 mozilla-inbound pgo test mochitest-browser-chrome on 2013-05-15 19:14:14 PDT for push 1e9dcad2319f
slave: talos-r3-xp-051

19:44:32     INFO -  TEST-PASS | chrome://mochitests/content/browser/toolkit/components/thumbnails/test/browser_thumbnails_background.js | onDone should be called only once
19:44:32  WARNING -  TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/toolkit/components/thumbnails/test/browser_thumbnails_background.js | Capture timed out so thumbnail should not be cached: c:\docume~1\cltbld\locals~1\temp\tmpu05f4p\thumbnails\e82891364309427ebb2a6931429ff068.png
19:44:32     INFO -  Stack trace:
19:44:32     INFO -      JS frame :: chrome://mochitests/content/browser/toolkit/components/thumbnails/test/browser_thumbnails_background.js :: onDone :: line 84
19:44:32     INFO -      JS frame :: resource://gre/modules/BackgroundPageThumbs.jsm :: callOnDoneFn :: line 263
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 120
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: then :: line 45
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 187
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 120
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: then :: line 45
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 187
19:44:32     INFO -      JS frame :: resource://gre/modules/Task.jsm :: TaskImpl_run :: line 220
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 120
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: then :: line 45
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 187
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 120
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: then :: line 45
19:44:32     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 187
19:44:32     INFO -      JS frame :: resource://gre/modules/osfile/_PromiseWorker.jsm :: onmessage :: line 134
19:44:32     INFO -      native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
The test just assumes that a capture with timeout 0 will time out before the captured page loads, but it looks like that's a bad assumption.  Makes sense.  It's a simple patch to make the page wait many seconds before finishing loading.  I'll land it once m-i reopens.
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/7d02789b81d9
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24
You need to log in before you can comment on or make changes to this bug.