Bug 87058 - Add PaintBehaviorDontClipOverflow for Android
Summary: Add PaintBehaviorDontClipOverflow for Android
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adam Barth
URL:
Keywords:
Depends on:
Blocks: 66687
  Show dependency treegraph
 
Reported: 2012-05-21 16:46 PDT by Adam Barth
Modified: 2012-05-22 09:54 PDT (History)
5 users (show)

See Also:


Attachments
work in progress (6.76 KB, patch)
2012-05-21 16:47 PDT, Adam Barth
no flags Details | Formatted Diff | Diff
Android changes to RenderLayerBacking.cpp (14.01 KB, patch)
2012-05-21 16:55 PDT, Adam Barth
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2012-05-21 16:46:23 PDT
Add PaintBehaviorDontClipOverflow for Android
Comment 1 Adam Barth 2012-05-21 16:47:22 PDT
Created attachment 143134 [details]
work in progress
Comment 2 Adam Barth 2012-05-21 16:55:59 PDT
Created attachment 143136 [details]
Android changes to RenderLayerBacking.cpp

@skyostil: I'm having trouble finishing this patch because it's related to changes to RenderLayerBacking.cpp, which I've attached.  In particular, I'm having trouble creating a patch out of the changes to RenderLayerBacking::paintContents.  Would you be willing to help me with that part?  Thanks!
Comment 3 Adam Barth 2012-05-21 16:56:22 PDT
+jamesr because this is getting close to the compositor.
Comment 4 Eric Seidel (no email) 2012-05-21 17:01:44 PDT
OK.  We just need to know why you'd ever want this behavior. :)
Comment 5 Sami Kyostila 2012-05-22 03:05:11 PDT
I have a WIP patch that completely reworks the way clipping is handled for compositor scrollable layers and fixes several bugs in the process. I was planning to land it as a part of https://bugs.webkit.org/show_bug.cgi?id=78862. I suggest we don't try merging in anything scrolling related for now from the Android diffs because all of that has changed quite a bit.
Comment 6 Adam Barth 2012-05-22 09:54:58 PDT
Thanks Sami.