Bug 50472 – Reflected unsigned attributes should be in the range [0, 2^31)

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Andy Estes]

Reflected unsigned attributes should be in the range [0, 2^31)

[Andy Estes]

Created attachment 75573 [details]
Patch

[Andy Estes]

This patch implements the algorithm HTML5 describes for getting reflected unsigned attributes. From section 2.8.1:

"If a reflecting IDL attribute is an unsigned integer type (unsigned long) then, on getting, the content attribute must be parsed according to the rules for parsing non-negative integers, and if that is successful, and the value is in the range 0 to 2147483647 inclusive, the resulting value must be returned. If, on the other hand, it fails or returns an out of range value, or if the attribute is absent, the default value must be returned instead, or 0 if there is no default value."

[Andy Estes]

Before anyone reviews this, it might be nice to let all the EWS bots do their thing before changing the review flag since I've only tested the code generation on the Mac.

[Darin Adler]

Comment on attachment 75573 [details]
Patch

The patch looks great; we also need a test that shows the actual behavior change in running JavaScript. Something in the LayoutTests directory. I’m going to say review- because that’s missing.

[Andy Estes]

Created attachment 75749 [details]
Patch

[Andy Estes]

(In reply to comment #4)
> (From update of attachment 75573 [details])
> The patch looks great; we also need a test that shows the actual behavior change in running JavaScript. Something in the LayoutTests directory. I’m going to say review- because that’s missing.

Thanks Darin. The new patch adds test cases for marquee's hspace and vspace properties, which are both reflected and unsigned.

[WebKit Review Bot]

Attachment 75749 [details] did not pass style-queue:

Failed to run "[u'git', u'reset', u'--hard', u'HEAD']" exit_code: 128
error: Could not write new index file.
fatal: Could not reset index file to revision 'HEAD'.


If any of these errors are false positives, please file a bug against check-webkit-style.

[Andy Estes]

Committed r73564: <http://trac.webkit.org/changeset/73564>

[WebKit Review Bot]

http://trac.webkit.org/changeset/73564 might have broken GTK Linux 32-bit Release
The following tests are not passing:
fast/css/input-search-padding.html
fast/css/text-input-with-webkit-border-radius.html
fast/forms/box-shadow-override.html
fast/forms/control-restrict-line-height.html
fast/forms/input-appearance-height.html
fast/forms/placeholder-pseudo-style.html
fast/forms/placeholder-set-value.html
fast/forms/search-cancel-button-style-sharing.html
fast/forms/search-placeholder-value-changed.html
fast/forms/search-rtl.html
fast/forms/search-styled.html
fast/forms/search-transformed.html
fast/forms/search-vertical-alignment.html
fast/forms/search-zoomed.html
fast/forms/searchfield-heights.html