Consider merging https://chromium.googlesource.com/chromium/blink/+/7cb1d002fd79604c9e8d0bd810ba6f8f21ca78af
These prefixes are:
1. Non-standard: the CSSOM standard doesn't mention them.
2. Broken: 'pixelWidth' should automagically convert from whatever unit the
stylesheet specified into CSS pixels. We're currently not doing that at
all: if an element has a 'width' of 100em, pixelWidth is '100', which is
probably wildly inaccurate.
3. Not covered by any tests ('fast/dom/CSSStyleDeclaration/css-properties-case-sensitive.html'
is the only test mentioning these properties, and it only checks
case-sensitivity: it doesn't validate behavior).
This patch drops support completely.
Created attachment 208584 [details]
Merges the patch
Hmm, would be nice to know if these are used in the wild.
(In reply to comment #2)
> Hmm, would be nice to know if these are used in the wild.
Here's blink-dev thread: https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/pixel$20pos/blink-dev/JS4nH_d1YkA/xd99lydDTfcJ
We might want to discuss this on webkit-dev.
Comment on attachment 208584 [details]
Merges the patch
I think this is a great idea. I just tested and both Firefox / Chrome no longer support this, so it doesn't seem like too controversial a change. Clearing flags to rebaseline.
Created attachment 412209 [details]
Created attachment 412210 [details]
Created attachment 412241 [details]
Committed r268957: <https://trac.webkit.org/changeset/268957>
All reviewed patches have been landed. Closing bug and clearing flags on attachment 412241 [details].
Re-opened since this is blocked by bug 218525
Created attachment 421728 [details]
Created attachment 421729 [details]
Created attachment 421732 [details]
Created attachment 421733 [details]
Created attachment 421748 [details]
Committed r273627: <https://commits.webkit.org/r273627>
All reviewed patches have been landed. Closing bug and clearing flags on attachment 421748 [details].