Skip to content

Commit

Permalink
Fix exception on timeout in SanityTest.js, and increase the timeout t…
Browse files Browse the repository at this point in the history
…ime. (bug 1205819, r=mchang)
  • Loading branch information
dvander committed Sep 19, 2015
1 parent 731f585 commit 5e18aa3
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions toolkit/components/gfx/SanityTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ const DEVICE_PREF="sanity-test.device-id";
const VERSION_PREF="sanity-test.version";
const DISABLE_VIDEO_PREF="media.hardware-video-decoding.failed";
const RUNNING_PREF="sanity-test.running";
const TIMEOUT_SEC=6;
const TIMEOUT_SEC=20;

// GRAPHICS_SANITY_TEST histogram enumeration values
const TEST_PASSED=0;
Expand Down Expand Up @@ -220,11 +220,14 @@ var listener = {
this.utils = null;
this.canvas = null;

this.messages.forEach((msgName) => {
this.mm.removeMessageListener(msgName, this);
});
if (this.mm) {
// We don't have a MessageManager if onWindowLoaded never fired.
this.messages.forEach((msgName) => {
this.mm.removeMessageListener(msgName, this);
});

this.mm = null;
this.mm = null;
}

// Remove the annotation after we've cleaned everything up, to catch any
// incidental crashes from having performed the sanity test.
Expand Down

0 comments on commit 5e18aa3

Please sign in to comment.