471285 - Workers: Spec for runtime error behavior has changed

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Attachment: Patch,v1

Small changes to runtime errors have been introduced into the spec, see http://www.whatwg.org/specs/web-workers/current-work/#runtime

Patch attached.

Comment 1

[Johnny Stenback (:jst)]

Comment on attachment 354571 [details] [diff] [review]
Patch,v1

Add the missing newline at the end of nsDOMWorkerEvents.cpp.

r+sr=jst

Comment 2

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Pushed changeset 1c68eb288c85 to mozilla-central.

Comment 3

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Had to back this out for a mochitest that was timing out (test_scopeOnerror.html).

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1230676718.1230679237.7642.gz&fulltext=1

Comment 4

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Fixed again, changeset 068d562872dc on mozilla-central. Needed to tweak the tests to prevent timeouts on linux, see changeset 25ba82075a6d.

Also found bug 471920.

Comment 5

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Pushed changeset d27cd2907bea to mozilla-1.9.1.