Bug 104011

Summary: [chromium] Introduce setting for low latency rendering
Product: WebKit Reporter: Sami Kyöstilä <skyostil>
Component: New BugsAssignee: Sami Kyöstilä <skyostil>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, brianderson, dglazkov, eric, fishd, jamesr, jchaffraix, tkent+wkapi, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Sami Kyöstilä
Reported 2012-12-04 10:00:31 PST
[chromium] Introduce setting for low latency rendering
Attachments
Patch (7.07 KB, patch)
2012-12-04 10:03 PST, Sami Kyöstilä 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sami Kyöstilä
Comment 1 2012-12-04 10:03:17 PST
Created attachment 177504 [details] Patch
WebKit Review Bot
Comment 2 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.
James Robinson
Comment 3 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.
Eric Seidel (no email)
Comment 4 2012-12-06 22:58:34 PST
What does this do?
James Robinson
Comment 5 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.
WebKit Review Bot
Comment 6 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>
WebKit Review Bot
Comment 7 2012-12-06 23:25:23 PST
All reviewed patches have been landed. Closing bug.
Sami Kyöstilä
Comment 8 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.
Eric Seidel (no email)
Comment 9 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.
Note You need to log in before you can comment on or make changes to this bug.