Inspector: Don't translate the context when rendering the highlights on a tiled layer.
Created attachment 118386 [details]
Comment on attachment 118386 [details]
Committed r102456: <http://trac.webkit.org/changeset/102456>
Alexander, Joe, are you cool with these changes? I know there has been a number of incremental improvements to this code related to scrollers and iframes that you worked on.
(Btw, please use webkit.org/new-inspector-bug for the inspector bug template)
(In reply to comment #4)
> Alexander, Joe, are you cool with these changes? I know there has been a
> number of incremental improvements to this code related to scrollers and
> iframes that you worked on.
Looks good to me, Qt guys would need to know best here since "delegatesScrolling"
appears to only be used by them! That function name is not a very useful name
in this context but the comment helps clarify that.
It would also be good to attach a simple .html test case to this bugzilla bug so that
a test of this since there was no manual test or LayoutTest included in the patch.
This might fix Qt but other ports might be broken for such a case.
delegatesScrolling basically means that webcore doesnt handle scrolling but the ui does (ie tiled backing store). If you have a better name that would be great. I guess you need something similar for iOS
Created attachment 118801 [details]
Here's a manual test file, tell me if you would prefer me to push it too or if it needs adjustments.
Thanks for looking at the patch.