Bug 98553 – Remove "orphaned units" quirk

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Tab Atkins]

The "orphaned units" quirk allows things like "width: 20 px;" (note the space between number and unit).

This is justified as matching WinIE behavior, but neither FF nor Opera do it, and their devs don't seem to be aware of any bug reports about it.  

It also wasn't important enough to justify being put into Simon Pieters' Quirks Mode spec <http://dvcs.w3.org/hg/quirks-mode/raw-file/tip/Overview.html>, which was based on surveying browsers and seeing what quirks were needed for web compat.

So, I'm removing it.

[Tab Atkins]

Created attachment 167379 [details]
Patch

[WebKit Review Bot]

Comment on attachment 167379 [details]
Patch

Attachment 167379 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/14169925

New failing tests:
WebFrameTest.DivScrollIntoEditableTest
touchadjustment/zoom-fatfinger.html
fast/css/quirk-orphaned-units.html

[Build Bot]

Comment on attachment 167379 [details]
Patch

Attachment 167379 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14082046

New failing tests:
mathml/presentation/fractions.xhtml
fast/css/quirk-orphaned-units.html

[Tab Atkins]

Created attachment 167434 [details]
Patch

[Tab Atkins]

(In reply to comment #4)
> Created an attachment (id=167434) [details]
> Patch

I can't tell how the WebFrameTest or mathml tests were related to this quirk, so I'll let the bots cycle around again and see if the failure is reproducible.  I've fixed the other two.

[Build Bot]

Comment on attachment 167434 [details]
Patch

Attachment 167434 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14182610

New failing tests:
mathml/presentation/fractions.xhtml

[Tab Atkins]

Created attachment 167436 [details]
Patch

[Tab Atkins]

(In reply to comment #7)
> Created an attachment (id=167436) [details]
> Patch

All right, found the bug in the mathml test that was depending on this quirk accidentally.  Should hopefully go full-green now.

[Eric Seidel (no email)]

Comment on attachment 167436 [details]
Patch

SGTM.

[WebKit Review Bot]

Comment on attachment 167436 [details]
Patch

Clearing flags on attachment: 167436

Committed r130668: <http://trac.webkit.org/changeset/130668>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.