Bug 140297

Summary: [ThreadedCompositor] Prevent excessive rendering call.
Product: WebKit Reporter: Gwang Yoon Hwang <yoon>
Component: New BugsAssignee: Gwang Yoon Hwang <yoon>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, mrobinson, zan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
zan: review+
Patch none

Description Gwang Yoon Hwang 2015-01-09 10:37:41 PST 
[ThreadedCompositor] Prevent excessive rendering call.
Comment 1 Gwang Yoon Hwang 2015-01-09 10:38:29 PST 
Created attachment 244349 [details]
Patch
Comment 2 Zan Dobersek 2015-01-10 01:25:15 PST 
Comment on attachment 244349 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=244349&action=review

> Source/WebKit2/Shared/CoordinatedGraphics/threadedcompositor/ThreadedCompositor.cpp:78
> +        const double targetFPS = 60;

This variable can be static.
Comment 3 Gwang Yoon Hwang 2015-01-10 03:17:09 PST 
Created attachment 244407 [details]
Patch
Comment 4 Gwang Yoon Hwang 2015-01-10 03:21:00 PST 
Comment on attachment 244349 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=244349&action=review

>> Source/WebKit2/Shared/CoordinatedGraphics/threadedcompositor/ThreadedCompositor.cpp:78
>> +        const double targetFPS = 60;
> 
> This variable can be static.

Good point. I'll modify
Comment 5 WebKit Commit Bot 2015-01-10 04:07:28 PST 
Comment on attachment 244407 [details]
Patch

Clearing flags on attachment: 244407

Committed r178238: <http://trac.webkit.org/changeset/178238>
Comment 6 Zan Dobersek 2015-04-13 02:27:39 PDT 
All done here, closing.