Bug 160371

Summary: Add an option to run the MotionMark tests with classic tile size (512x512)
Product: WebKit Reporter: Said Abou-Hallawa <sabouhallawa>
Component: AnimationsAssignee: Said Abou-Hallawa <sabouhallawa>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, dino, jonlee, rniwa, simon.fraser
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Said Abou-Hallawa 2016-07-29 18:50:22 PDT
This option allows measuring the performance of the css tests without the big tile improvement.
Comment 1 Said Abou-Hallawa 2016-07-29 19:14:37 PDT
Created attachment 284929 [details]
Patch
Comment 2 Darin Adler 2016-07-31 20:55:15 PDT
Comment on attachment 284929 [details]
Patch

Seems OK for now, but seems a bit tied to implementation details of WebKit as opposed to general web platform principles.
Comment 3 Said Abou-Hallawa 2016-08-01 11:24:48 PDT
(In reply to comment #2)
> Comment on attachment 284929 [details]
> Patch
> 
> Seems OK for now, but seems a bit tied to implementation details of WebKit
> as opposed to general web platform principles.

This is a debug option which is only visible in the MotionMark debug page. We use the debug page to test MotionMark or WebKit implementation details. We can decide which tweaks can make the measurements more accurate and how we can improve the WebKit performance.

Also I think tiling is an implementation detail and I am not aware of any specs that specify the tile implementation or size.
Comment 4 WebKit Commit Bot 2016-08-01 11:46:36 PDT
Comment on attachment 284929 [details]
Patch

Clearing flags on attachment: 284929

Committed r203978: <http://trac.webkit.org/changeset/203978>
Comment 5 WebKit Commit Bot 2016-08-01 11:46:42 PDT
All reviewed patches have been landed.  Closing bug.