RESOLVED FIXED Bug 82941
We should use CSSPropertyID rather than integers when manipulating CSS property ids. 
https://bugs.webkit.org/show_bug.cgi?id=82941
Summary We should use CSSPropertyID rather than integers when manipulating CSS proper...
Alexis Menard (darktears)
Reported 2012-04-02 13:38:00 PDT
We should use CSSPropertyID rather than integers when manipulating CSS property ids.
Attachments
Patch (22.23 KB, patch)
2012-04-02 13:56 PDT, Alexis Menard (darktears) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexis Menard (darktears)
Comment 1 2012-04-02 13:56:26 PDT
Created attachment 135177 [details] Patch
Alexis Menard (darktears)
Comment 2 2012-04-02 13:57:35 PDT
(In reply to comment #1) > Created an attachment (id=135177) [details] > Patch It's quite a big change at the end, I'm willing to it but I want to make sure people agree that it will make the code nicer.
Alexis Menard (darktears)
Comment 3 2012-04-02 13:58:05 PDT
(In reply to comment #2) > (In reply to comment #1) > > Created an attachment (id=135177) [details] [details] > > Patch > > It's quite a big change at the end, I'm willing to it but I want to make sure people agree that it will make the code nicer. big change I mean because there will be more patches.
WebKit Review Bot
Comment 4 2012-04-02 15:30:22 PDT
Comment on attachment 135177 [details] Patch Clearing flags on attachment: 135177 Committed r112952: <http://trac.webkit.org/changeset/112952>
WebKit Review Bot
Comment 5 2012-04-02 15:30:27 PDT
All reviewed patches have been landed. Closing bug.
Alexis Menard (darktears)
Comment 6 2012-04-04 06:56:16 PDT
(In reply to comment #5) > All reviewed patches have been landed. Closing bug. https://bugs.webkit.org/show_bug.cgi?id=83151 fixes variable names removed too aggressively in some .h files.
Note You need to log in before you can comment on or make changes to this bug.