Bug 114119

Summary: Remove feature: CSS variables
Product: WebKit Reporter: Andreas Kling <kling>
Component: CSSAssignee: Andreas Kling <kling>
Status: RESOLVED FIXED    
Severity: Normal CC: abecsi, allan.jensen, beidson, benjamin, buildbot, cmarcelo, dbates, divya, eric, esprehn+autocc, gtk-ews, gyuyoung.kim, jon.rimmer, kling, koivisto, macpherson, menard, ojan.autocc, rakuco, rego+ews, rniwa, syoichi, vestbo, webkit.review.bot, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Remove CSS variables (for EWS)
none
Remove CSS variables (for EWS)
webkit-ews: commit-queue-
I can't brain on Sundays
webkit-ews: commit-queue-
rebased patch kling: review+

Description Andreas Kling 2013-04-07 09:34:59 PDT
The CSS variables feature was only shipped by Chromium and we should remove it, as it'll just start to bitrot.
Comment 2 Andreas Kling 2013-04-07 09:56:17 PDT
Created attachment 196791 [details]
Remove CSS variables (for EWS)
Comment 3 Andreas Kling 2013-04-07 09:57:13 PDT
Created attachment 196792 [details]
Remove CSS variables (for EWS)
Comment 4 Early Warning System Bot 2013-04-07 10:05:22 PDT
Comment on attachment 196792 [details]
Remove CSS variables (for EWS)

Attachment 196792 [details] did not pass qt-ews (qt):
Output: http://webkit-commit-queue.appspot.com/results/17525742
Comment 5 Early Warning System Bot 2013-04-07 10:07:19 PDT
Comment on attachment 196792 [details]
Remove CSS variables (for EWS)

Attachment 196792 [details] did not pass qt-wk2-ews (qt):
Output: http://webkit-commit-queue.appspot.com/results/17540338
Comment 6 EFL EWS Bot 2013-04-07 10:25:26 PDT
Comment on attachment 196792 [details]
Remove CSS variables (for EWS)

Attachment 196792 [details] did not pass efl-ews (efl):
Output: http://webkit-commit-queue.appspot.com/results/17528774
Comment 7 Build Bot 2013-04-07 10:34:46 PDT
Comment on attachment 196792 [details]
Remove CSS variables (for EWS)

Attachment 196792 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-commit-queue.appspot.com/results/17446744
Comment 8 kov's GTK+ EWS bot 2013-04-07 10:44:21 PDT
Comment on attachment 196792 [details]
Remove CSS variables (for EWS)

Attachment 196792 [details] did not pass gtk-ews (gtk):
Output: http://webkit-commit-queue.appspot.com/results/17524750
Comment 9 Build Bot 2013-04-07 10:45:39 PDT
Comment on attachment 196792 [details]
Remove CSS variables (for EWS)

Attachment 196792 [details] did not pass mac-ews (mac):
Output: http://webkit-commit-queue.appspot.com/results/17486841
Comment 10 Andreas Kling 2013-04-07 12:36:36 PDT
Created attachment 196801 [details]
I can't brain on Sundays
Comment 11 Early Warning System Bot 2013-04-07 12:56:32 PDT
Comment on attachment 196801 [details]
I can't brain on Sundays

Attachment 196801 [details] did not pass qt-ews (qt):
Output: http://webkit-commit-queue.appspot.com/results/17423102
Comment 12 Early Warning System Bot 2013-04-07 13:02:00 PDT
Comment on attachment 196801 [details]
I can't brain on Sundays

Attachment 196801 [details] did not pass qt-wk2-ews (qt):
Output: http://webkit-commit-queue.appspot.com/results/17547013
Comment 13 EFL EWS Bot 2013-04-07 13:21:00 PDT
Comment on attachment 196801 [details]
I can't brain on Sundays

Attachment 196801 [details] did not pass efl-ews (efl):
Output: http://webkit-commit-queue.appspot.com/results/17504007
Comment 14 Jon Rimmer 2013-04-07 13:25:08 PDT
As a web developer, I have to say that this is an extremely disappointing decision. Apple should be turning this feature on in their browsers, not removing it.
Comment 15 Build Bot 2013-04-07 13:30:15 PDT
Comment on attachment 196801 [details]
I can't brain on Sundays

Attachment 196801 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-commit-queue.appspot.com/results/17522156
Comment 16 Build Bot 2013-04-07 14:00:48 PDT
Comment on attachment 196801 [details]
I can't brain on Sundays

Attachment 196801 [details] did not pass mac-ews (mac):
Output: http://webkit-commit-queue.appspot.com/results/17508024
Comment 17 Allan Sandfeld Jensen 2013-04-07 14:22:30 PDT
Comment on attachment 196801 [details]
I can't brain on Sundays

It would probably be a good idea to repeat the rationalle in the Changelog for future reference.
Comment 18 Andreas Kling 2013-04-07 14:30:47 PDT
(In reply to comment #17)
> (From update of attachment 196801 [details])
> It would probably be a good idea to repeat the rationalle in the Changelog for future reference.

Oh definitely. These are just my (lame) attempts at getting a green EWS before making the final patch.

(In reply to comment #14)
> As a web developer, I have to say that this is an extremely disappointing decision. Apple should be turning this feature on in their browsers, not removing it.

Please voice your concerns on the webkit-dev mailing list by responding to this thread: <https://lists.webkit.org/pipermail/webkit-dev/2013-April/024476.html>

Whether or not the feature is removed is still up for discussion. I'm just trying to be proactive here.
Comment 19 Build Bot 2013-04-07 15:13:58 PDT
Comment on attachment 196801 [details]
I can't brain on Sundays

Attachment 196801 [details] did not pass mac-ews (mac):
Output: http://webkit-commit-queue.appspot.com/results/17553004
Comment 20 Antti Koivisto 2013-11-28 07:24:05 PST
Created attachment 218002 [details]
rebased patch
Comment 21 Andreas Kling 2013-11-28 07:31:58 PST
Comment on attachment 218002 [details]
rebased patch

r=me.

Thank you for moving this forward. The code was nowhere near WebKit quality standards, and it's effectively unmaintained.
There should be something about that in the ChangeLog.
Comment 22 Antti Koivisto 2013-11-28 07:44:55 PST
ttps://trac.webkit.org/r159842