RESOLVED FIXED Bug 113524
[Chromium] Don't create SolidColorLayer for full transparent background 
https://bugs.webkit.org/show_bug.cgi?id=113524
Summary [Chromium] Don't create SolidColorLayer for full transparent background
Xianzhu Wang
Reported 2013-03-28 11:55:36 PDT
http://trac.webkit.org/changeset/146826 which enabled solidColorLayer on Chromium breaks Chromium-Android. I'd like to disable it temporarily until we support it for impl-side-painting.
Attachments
Patch (3.26 KB, patch)
2013-03-28 15:29 PDT, Xianzhu Wang 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
James Robinson
Comment 1 2013-03-28 12:16:01 PDT
It's supported today, isn't it?
James Robinson
Comment 2 2013-03-28 12:23:30 PDT
http://trac.webkit.org/export/147140/trunk/LayoutTests/compositing/geometry/fixed-in-composited.html triggers the solid color layer path and works fine with impl-side painting on my linux box.
Xianzhu Wang
Comment 3 2013-03-28 14:50:54 PDT
The scrolling problem on Android is caused by a full-transparent SolidColorLayer over a scrollable layer.
James Robinson
Comment 4 2013-03-28 14:57:10 PDT
Ah, I see - is the fully transparent layer eating input events or something like that? If something's fully transparent then we definitely shouldn't make any layer at all.
Xianzhu Wang
Comment 5 2013-03-28 14:59:44 PDT
(In reply to comment #4) > Ah, I see - is the fully transparent layer eating input events or something like that? Yes. The compositor selects the transparent solid color layer instead of the scrollable layer under it as the potentially scrolling layer.
Xianzhu Wang
Comment 6 2013-03-28 15:29:18 PDT
Created attachment 195658 [details] Patch
James Robinson
Comment 7 2013-03-28 15:37:11 PDT
Comment on attachment 195658 [details] Patch Nice.
WebKit Review Bot
Comment 8 2013-03-28 16:06:59 PDT
Comment on attachment 195658 [details] Patch Clearing flags on attachment: 195658 Committed r147178: <http://trac.webkit.org/changeset/147178>
WebKit Review Bot
Comment 9 2013-03-28 16:07:03 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.