Bug 1362410 - Make the setTimeout run on the same target as HistoryTracker. r?smaug draft
authorSamael Wang <freesamael@gmail.com>
Tue, 28 Nov 2017 15:50:27 +0800
changeset 704070 767a95a3aad7aaa2c8637ba5acd4f92d8c484aa0
parent 703941 f5f03ee9e6abf77964f8dc1b9d69c6ccd3f655fd
child 741990 c369e1d23d5238f1d620aee113e427752668cf70
push id91066
push userbmo:sawang@mozilla.com
push dateTue, 28 Nov 2017 07:54:04 +0000
reviewerssmaug
bugs1362410
milestone59.0a1
Bug 1362410 - Make the setTimeout run on the same target as HistoryTracker. r?smaug MozReview-Commit-ID: KvcFxG7RMNZ
docshell/test/browser/browser_bug1347823.js
--- a/docshell/test/browser/browser_bug1347823.js
+++ b/docshell/test/browser/browser_bug1347823.js
@@ -47,18 +47,20 @@ add_task(async function testExpiredCache
         content.document.body.textContent = "modified";
       });
 
       // Load a random page.
       BrowserTestUtils.loadURI(browser, "data:text/html;charset=utf-8,page2");
       await BrowserTestUtils.browserLoaded(browser);
 
       // Wait for 3 times of expiration timeout, hopefully it's evicted...
-      await new Promise(resolve => {
-        setTimeout(resolve, 3000);
+      await ContentTask.spawn(browser, null, () => {
+        return new Promise(resolve => {
+          content.setTimeout(resolve, 3000);
+        });
       });
 
       // Go back and verify text content.
       let awaitPageShow = BrowserTestUtils.waitForContentEvent(browser, "pageshow");
       browser.goBack();
       await awaitPageShow;
       await ContentTask.spawn(browser, null, () => {
         is(content.document.body.textContent, "page1");