1303099 - LenientFloat should finish argument processing before bailing out

Status: RESOLVED FIXED

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

Description

[Boris Zbarsky [:bzbarsky]]

See https://github.com/whatwg/html/issues/1785

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: LenientFloat handling should not prevent all arguments being processed; instead it should just cause an early return right after argument processing

Comment 2

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Comment on attachment 8791733 [details] [diff] [review]
LenientFloat handling should not prevent all arguments being processed; instead it should just cause an early return right after argument processing

Review of attachment 8791733 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm. Since this syncs us with everyone else, is this turning into a wpt?

Comment 3

[Boris Zbarsky [:bzbarsky]]

Yes, I'll add a test as soon as I make sure we don't have one already. ;) https://treeherder.mozilla.org/#/jobs?repo=try&revision=eb9802a1e662 is checking that.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Attachment: With a web platform test

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f60f494e7d57
LenientFloat handling should not prevent all arguments being processed; instead it should just cause an early return right after argument processing.  r=qdot

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f60f494e7d57