If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

test_bug490949.html calls "SimpleTest.finish()" twice

NEW
Unassigned

Status

()

Core
ImageLib
6 years ago
6 years ago

People

(Reporter: dholbert, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Created attachment 534340 [details]
screenshot

While writing a mochitest for another imagelib bug, I happened to notice that the mochitest modules/libpr0n/test/mochitest/test_bug490949.html triggers **two** calls to SimpleTest.finish(), resulting in two test-results printouts (with different numbers of successes reported).

See attached screenshot.

The finish() calls are ultimately triggered by the main method "checkLoad", which is called in two places:
> <body onload="checkFirst()">
[...]
> remoteCanvas.load(checkFirst);

I don't know which of those we want to keep or remove (maybe we need both, but then the test needs tweaking to avoid duplicate finish calls...?)

Reference to source:
http://mxr.mozilla.org/mozilla-central/source/modules/libpr0n/test/mochitest/test_bug490949.html?force=1
(Reporter)

Updated

6 years ago
OS: Linux → All
Hardware: x86_64 → All
You need to log in before you can comment on or make changes to this bug.