Bug 140297 - [ThreadedCompositor] Prevent excessive rendering call.
Summary: [ThreadedCompositor] Prevent excessive rendering call.
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: Gwang Yoon Hwang
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-09 10:37 PST by Gwang Yoon Hwang
Modified: 2015-04-13 02:27 PDT (History)
3 users (show)

See Also:

Attachments
Patch (5.63 KB, patch)
2015-01-09 10:38 PST, Gwang Yoon Hwang 		zan: review+
Details | Formatted Diff | Diff
Patch (5.56 KB, patch)
2015-01-10 03:17 PST, Gwang Yoon Hwang 		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 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.