Bug 97287

Summary: [mac] WKTR should use tiled drawing/threaded scrolling on Mountain Lion
Product: WebKit Reporter: Tim Horton <thorton>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: andersca, bdakin, mitz, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 98065, 98077, 98064, 98074    
Bug Blocks:    
Attachments:
Description Flags
wktr switch
none
patch to enable tiled drawing on ML none

Tim Horton
Reported 2012-09-20 20:47:26 PDT
It'd be nice if we could run the tests with a TiledCoreAnimationDrawingArea and scrolling thread, etc. It'd be really nice if we could swap into and out of tiled drawing via internals, but running all the tests that way would be a start. This patch will just add support to WKTR, but no way to use it yet.
Attachments
wktr switch (4.22 KB, patch)
2012-09-20 23:04 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
patch to enable tiled drawing on ML (2.17 KB, patch)
2012-09-25 13:20 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Tim Horton
Comment 1 2012-09-20 20:59:07 PDT
On second thought, there are no other features enabled by WKTR command line flags, so maybe we should just do the right thing and support it via internals? I don't know how much work it is to get swapping-out-the-drawing-area working again. Anders?
Tim Horton
Comment 2 2012-09-20 23:04:26 PDT
Created attachment 165055 [details] wktr switch Attaching a patch to add a --use-tiled-drawing switch to WKTR. I don't think this is the approach we want to take, but it's probably easier than the alternative.
Tim Horton
Comment 3 2012-09-21 12:05:29 PDT
Related to <rdar://problem/11778900>
Tim Horton
Comment 4 2012-09-25 13:20:30 PDT
Created attachment 165658 [details] patch to enable tiled drawing on ML Attaching a patch to turn on tiled drawing in WKTR on ML. Will wait to land until I see what effect it has on layout test results.
Note You need to log in before you can comment on or make changes to this bug.