Closed Bug 873046 Opened 11 years ago Closed 11 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)

Product:
Firefox
Component:
General
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
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: 11 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.