Bug 74085

Summary: Inspector: Don't translate the context when rendering the highlights on a tiled layer.
Product: WebKit Reporter: Jocelyn Turcotte <jturcotte>
Component: New BugsAssignee: Jocelyn Turcotte <jturcotte>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, joepeck, kenneth, pfeldman
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
kenneth: review+
Manual test none

Description Jocelyn Turcotte 2011-12-08 06:27:25 PST 
Inspector: Don't translate the context when rendering the highlights on a tiled layer.
Comment 1 Jocelyn Turcotte 2011-12-08 07:12:55 PST 
Created attachment 118386 [details]
Patch
Comment 2 Kenneth Rohde Christiansen 2011-12-09 01:50:15 PST 
Comment on attachment 118386 [details]
Patch

r=me
Comment 3 Jocelyn Turcotte 2011-12-09 09:38:43 PST 
Committed r102456: <http://trac.webkit.org/changeset/102456>
Comment 4 Pavel Feldman 2011-12-09 10:12:12 PST 
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.
Comment 5 Pavel Feldman 2011-12-09 10:12:46 PST 
(Btw, please use webkit.org/new-inspector-bug for the inspector bug template)
Comment 6 Joseph Pecoraro 2011-12-09 12:06:17 PST 
(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.
Comment 7 Kenneth Rohde Christiansen 2011-12-10 07:39:26 PST 
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
Comment 8 Jocelyn Turcotte 2011-12-12 08:54:39 PST 
Created attachment 118801 [details]
Manual test

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.