Bug 104011 - [chromium] Introduce setting for low latency rendering
Summary: [chromium] Introduce setting for low latency rendering
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sami Kyöstilä
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-04 10:00 PST by Sami Kyöstilä
Modified: 2012-12-07 07:44 PST (History)
9 users (show)

See Also:

Attachments
Patch (7.07 KB, patch)
2012-12-04 10:03 PST, Sami Kyöstilä 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sami Kyöstilä 2012-12-04 10:00:31 PST 
[chromium] Introduce setting for low latency rendering
Comment 1 Sami Kyöstilä 2012-12-04 10:03:17 PST 
Created attachment 177504 [details]
Patch
Comment 2 WebKit Review Bot 2012-12-04 10:04:42 PST 
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Comment 3 James Robinson 2012-12-06 22:54:52 PST 
Comment on attachment 177504 [details]
Patch

Name's not the best - who wouldn't want low latency mode - but good enough for now I think.
Comment 4 Eric Seidel (no email) 2012-12-06 22:58:34 PST 
What does this do?
Comment 5 James Robinson 2012-12-06 23:14:38 PST 
It's a hook for a speculative different compositor behavior.  It won't change anything else in WebKit.
Comment 6 WebKit Review Bot 2012-12-06 23:25:20 PST 
Comment on attachment 177504 [details]
Patch

Clearing flags on attachment: 177504

Committed r136926: <http://trac.webkit.org/changeset/136926>
Comment 7 WebKit Review Bot 2012-12-06 23:25:23 PST 
All reviewed patches have been landed.  Closing bug.
Comment 8 Sami Kyöstilä 2012-12-07 03:26:20 PST 
> Name's not the best - who wouldn't want low latency mode - but good enough for now I think.

Yeah, it does sound a bit like a turbo button :) I guess it could be something like platformSupportsLowLatencyRendering since it's more about what your port can handle than what you want.
Comment 9 Eric Seidel (no email) 2012-12-07 07:44:23 PST 
I would have probably used experimental in the name. But yes, it doesn't matter. So long as this is just a storage book for chromium.