Bug 119712

Summary: Remove support for 'pixel' and 'pos' CSSOM prefixes
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: CSSAssignee: Sam Weinig <sam>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, cdumez, commit-queue, darin, dino, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, hyatt, joepeck, jonlee, macpherson, menard, mkwst, sam, simon.fraser, syoichi, webkit-bug-importer
Priority: P2 Keywords: BlinkMergeCandidate, InRadar, WebExposed
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=218158
Bug Depends on: 218525    
Bug Blocks:    
Attachments:
Description Flags
Merges the patch
none
Patch
ews-feeder: commit-queue-
Patch
none
Patch
none
Patch
none
Patch
ews-feeder: commit-queue-
Patch
none
Patch
none
Patch none

Ryosuke Niwa
Reported 2013-08-12 18:12:10 PDT
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[1]. 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. [1]: http://msdn.microsoft.com/en-us/library/ie/ms531135(v=vs.85).aspx
Attachments
Merges the patch (8.99 KB, patch)
2013-08-12 19:16 PDT, Ryosuke Niwa
no flags
Patch (12.33 KB, patch)
2020-10-23 13:43 PDT, Sam Weinig
ews-feeder: commit-queue-
Patch (13.13 KB, patch)
2020-10-23 13:53 PDT, Sam Weinig
no flags
Patch (16.06 KB, patch)
2020-10-24 10:55 PDT, Sam Weinig
no flags
Patch (23.40 KB, patch)
2021-02-26 16:54 PST, Sam Weinig
no flags
Patch (12.60 KB, patch)
2021-02-26 16:57 PST, Sam Weinig
ews-feeder: commit-queue-
Patch (12.64 KB, patch)
2021-02-26 18:32 PST, Sam Weinig
no flags
Patch (12.75 KB, patch)
2021-02-26 18:34 PST, Sam Weinig
no flags
Patch (12.75 KB, patch)
2021-02-27 09:50 PST, Sam Weinig
no flags
Ryosuke Niwa
Comment 1 2013-08-12 19:16:50 PDT
Created attachment 208584 [details] Merges the patch
Simon Fraser (smfr)
Comment 2 2013-08-12 20:36:22 PDT
Hmm, would be nice to know if these are used in the wild.
Ryosuke Niwa
Comment 3 2013-08-12 20:42:12 PDT
(In reply to comment #2) > Hmm, would be nice to know if these are used in the wild. Indeed.
Ryosuke Niwa
Comment 4 2013-08-12 20:44:45 PDT
Joseph Pecoraro
Comment 5 2015-11-12 15:29:17 PST
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.
Sam Weinig
Comment 6 2020-10-23 13:43:35 PDT
Sam Weinig
Comment 7 2020-10-23 13:53:19 PDT
Sam Weinig
Comment 8 2020-10-24 10:55:10 PDT
EWS
Comment 9 2020-10-25 09:34:51 PDT
Committed r268957: <https://trac.webkit.org/changeset/268957> All reviewed patches have been landed. Closing bug and clearing flags on attachment 412241 [details].
Radar WebKit Bug Importer
Comment 10 2020-10-25 09:35:21 PDT
WebKit Commit Bot
Comment 11 2020-11-03 10:43:22 PST
Re-opened since this is blocked by bug 218525
Sam Weinig
Comment 12 2021-02-26 16:54:15 PST Comment hidden (obsolete)
Sam Weinig
Comment 13 2021-02-26 16:57:30 PST Comment hidden (obsolete)
Sam Weinig
Comment 14 2021-02-26 18:32:04 PST Comment hidden (obsolete)
Sam Weinig
Comment 15 2021-02-26 18:34:51 PST Comment hidden (obsolete)
Sam Weinig
Comment 16 2021-02-27 09:50:01 PST
EWS
Comment 17 2021-02-27 11:51:51 PST
Committed r273627: <https://commits.webkit.org/r273627> All reviewed patches have been landed. Closing bug and clearing flags on attachment 421748 [details].
Note You need to log in before you can comment on or make changes to this bug.