RESOLVED FIXED 111818
create runtime flags for CSS Compositing
https://bugs.webkit.org/show_bug.cgi?id=111818
Summary create runtime flags for CSS Compositing
Rik Cabanier
Reported 2013-03-07 20:59:44 PST
This patch will add flags to WebCore to turn CSS compositing and blending features off and on.
Attachments
not for review (11.65 KB, text/plain)
2013-03-07 21:11 PST, Rik Cabanier
no flags
not for review (11.65 KB, patch)
2013-03-07 21:41 PST, Rik Cabanier
no flags
patch (12.46 KB, patch)
2013-03-08 15:41 PST, Rik Cabanier
no flags
Patch (22.57 KB, patch)
2013-03-13 15:00 PDT, Rik Cabanier
no flags
Patch (22.57 KB, text/plain)
2013-03-13 15:55 PDT, Rik Cabanier
no flags
Patch (22.46 KB, patch)
2013-03-13 16:15 PDT, Rik Cabanier
no flags
Fixed style issue (22.46 KB, patch)
2013-03-13 20:07 PDT, Rik Cabanier
no flags
Rik Cabanier
Comment 1 2013-03-07 21:11:08 PST
Created attachment 192140 [details] not for review
WebKit Review Bot
Comment 2 2013-03-07 21:13:00 PST
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Early Warning System Bot
Comment 3 2013-03-07 21:23:43 PST
Early Warning System Bot
Comment 4 2013-03-07 21:27:09 PST
Rik Cabanier
Comment 5 2013-03-07 21:41:07 PST
Created attachment 192141 [details] not for review
Early Warning System Bot
Comment 6 2013-03-07 21:50:06 PST
Early Warning System Bot
Comment 7 2013-03-07 21:51:33 PST
Build Bot
Comment 8 2013-03-07 22:37:17 PST
Comment on attachment 192141 [details] not for review Attachment 192141 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-commit-queue.appspot.com/results/16992546
Build Bot
Comment 9 2013-03-08 05:46:19 PST
Build Bot
Comment 10 2013-03-08 08:32:05 PST
Rik Cabanier
Comment 11 2013-03-08 15:41:40 PST
Rik Cabanier
Comment 12 2013-03-13 15:00:30 PDT
Simon Fraser (smfr)
Comment 13 2013-03-13 15:32:31 PDT
Comment on attachment 193002 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=193002&action=review > Source/WebCore/bindings/generic/RuntimeEnabledFeatures.cpp:60 > +bool RuntimeEnabledFeatures::isSSCompositingEnabled = false; Typo.
Rik Cabanier
Comment 14 2013-03-13 15:36:40 PDT
(In reply to comment #13) > (From update of attachment 193002 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=193002&action=review > > > Source/WebCore/bindings/generic/RuntimeEnabledFeatures.cpp:60 > > +bool RuntimeEnabledFeatures::isSSCompositingEnabled = false; > > Typo. Good catch!
Rik Cabanier
Comment 15 2013-03-13 15:55:34 PDT
Simon Fraser (smfr)
Comment 16 2013-03-13 16:02:25 PDT
Comment on attachment 193014 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=193014&action=review > Source/WebCore/ChangeLog:3 > + create runtime flags for CSS Compositing Create > Source/WebCore/ChangeLog:10 > + Added a runtime flags to enable CSS compositing as a runtime flag for chromium. Is this really chromium-specific?
Rik Cabanier
Comment 17 2013-03-13 16:09:10 PDT
(In reply to comment #16) > (From update of attachment 193014 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=193014&action=review > > > Source/WebCore/ChangeLog:3 > > + create runtime flags for CSS Compositing > > Create > > > Source/WebCore/ChangeLog:10 > > + Added a runtime flags to enable CSS compositing as a runtime flag for chromium. > > Is this really chromium-specific? Well, that's how the bug started out :-) I guess it is now also a preference in Safari but it's always on by default.
Rik Cabanier
Comment 18 2013-03-13 16:10:26 PDT
(In reply to comment #17) > (In reply to comment #16) > > (From update of attachment 193014 [details] [details]) > > View in context: https://bugs.webkit.org/attachment.cgi?id=193014&action=review > > > > > Source/WebCore/ChangeLog:3 > > > + create runtime flags for CSS Compositing > > > > Create > > > > > Source/WebCore/ChangeLog:10 > > > + Added a runtime flags to enable CSS compositing as a runtime flag for chromium. > > > > Is this really chromium-specific? > > Well, that's how the bug started out :-) > I guess it is now also a preference in Safari but it's always on by default. I will update the changelog
Rik Cabanier
Comment 19 2013-03-13 16:15:37 PDT
Ryosuke Niwa
Comment 20 2013-03-13 20:04:12 PDT
Comment on attachment 193018 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=193018&action=review rs=me. > Source/WebCore/css/CSSParser.cpp:4322 > + if (cssCompositingEnabled() &&(val->id == CSSValueNormal || val->id == CSSValueMultiply Nit: space needed after &&.
Ryosuke Niwa
Comment 21 2013-03-13 20:04:56 PDT
Comment on attachment 193018 [details] Patch cq- due to the nit.
Rik Cabanier
Comment 22 2013-03-13 20:07:20 PDT
Created attachment 193049 [details] Fixed style issue
WebKit Review Bot
Comment 23 2013-03-13 20:55:11 PDT
Comment on attachment 193049 [details] Fixed style issue Clearing flags on attachment: 193049 Committed r145784: <http://trac.webkit.org/changeset/145784>
WebKit Review Bot
Comment 24 2013-03-13 20:55:17 PDT
All reviewed patches have been landed. Closing bug.
Takahiro Ichihashi
Comment 25 2014-02-24 18:30:06 PST
What version is expected to be landed with this patch?
Note You need to log in before you can comment on or make changes to this bug.