Bug 906713 - Reset default capture timeout earlier to prevent oranges. r=adw, a=test-only RELEASE_BASE_20130909
authorMark Hammond <mhammond@skippinet.com.au>
Mon, 09 Sep 2013 10:16:21 +1000
changeset 148537 2cbff87e4417c4692c4e50c1b70b9965006ca249
parent 148536 f6273640c6e335861013e47e5e6164154592acc5
child 148538 1093a29a5a3e96a458d734ab710649ec29077b49
push id2848
push userryanvm@gmail.com
push dateMon, 09 Sep 2013 13:26:07 +0000
treeherdermozilla-beta@2cbff87e4417 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersadw, test-only
bugs906713
milestone24.0
Bug 906713 - Reset default capture timeout earlier to prevent oranges. r=adw, a=test-only
toolkit/components/thumbnails/test/browser_thumbnails_background.js
--- a/toolkit/components/thumbnails/test/browser_thumbnails_background.js
+++ b/toolkit/components/thumbnails/test/browser_thumbnails_background.js
@@ -167,21 +167,21 @@ let tests = [
     imports.BackgroundPageThumbs._destroyBrowserTimeout = 1000;
 
     yield capture(url1);
     ok(file1.exists(), "First file should exist after capture.");
 
     yield wait(2000);
     is(imports.BackgroundPageThumbs._thumbBrowser, undefined,
        "Thumb browser should be destroyed after timeout.");
+    imports.BackgroundPageThumbs._destroyBrowserTimeout = defaultTimeout;
 
     yield capture(url2);
     ok(file2.exists(), "Second file should exist after capture.");
 
-    imports.BackgroundPageThumbs._destroyBrowserTimeout = defaultTimeout;
     isnot(imports.BackgroundPageThumbs._thumbBrowser, undefined,
           "Thumb browser should exist immediately after capture.");
   },
 
   function privateBrowsingActive() {
     let url = "http://example.com/";
     let file = fileForURL(url);
     ok(!file.exists(), "Thumbnail file should not already exist.");