From 894ba93b222d145b41636652b759cfd1d7a3db97 Mon Sep 17 00:00:00 2001 From: Tim Taubert Date: Mon, 11 Mar 2013 13:53:50 +0100 Subject: [PATCH] Bug 847100 - Make sure dummy thumbnails aren't expired too early when testing thumbnail expiration; r=yoric --- .../thumbnails/test/browser_thumbnails_expiration.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/toolkit/components/thumbnails/test/browser_thumbnails_expiration.js b/toolkit/components/thumbnails/test/browser_thumbnails_expiration.js index 9e91065e4ed..a4c1b097ab9 100644 --- a/toolkit/components/thumbnails/test/browser_thumbnails_expiration.js +++ b/toolkit/components/thumbnails/test/browser_thumbnails_expiration.js @@ -51,6 +51,14 @@ function runTests() { urls.forEach(createDummyThumbnail); ok(urls.every(thumbnailExists), "all dummy thumbnails created"); + // Make sure our dummy thumbnails aren't expired too early. + let dontExpireDummyURLs = function (cb) cb(urls); + PageThumbs.addExpirationFilter(dontExpireDummyURLs); + + registerCleanupFunction(function () { + PageThumbs.removeExpirationFilter(dontExpireDummyURLs); + }); + // Expire thumbnails and expect 10 remaining. yield expireThumbnails([]); let remainingURLs = [u for (u of urls) if (thumbnailExists(u))];