Bug 94600

Summary: [BlackBerry] requestAnimationFrame: Unscheduled display link frames need to be sent to main thread
Product: WebKit Reporter: Andrew Lo <anlo>
Component: WebKit BlackBerryAssignee: Andrew Lo <anlo>
Status: RESOLVED FIXED    
Severity: Normal CC: mifenton, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Andrew Lo
Reported 2012-08-21 07:42:55 PDT
http://trac.webkit.org/changeset/116792 implements destroying DisplayRefreshMonitor after the number of unscheduled frames exceeds 10. Part of that change involves dispatching to the main thread whether the frame was scheduled or not. Make the corresponding change for our port that was made for DisplayRefreshMonitorMac.cpp.
Attachments
Patch (1.89 KB, patch)
2012-08-21 08:00 PDT, Andrew Lo 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andrew Lo
Comment 1 2012-08-21 08:00:50 PDT
Created attachment 159691 [details] Patch
Rob Buis
Comment 2 2012-08-21 08:02:48 PDT
Comment on attachment 159691 [details] Patch LGTM.
WebKit Review Bot
Comment 3 2012-08-21 17:07:32 PDT
Comment on attachment 159691 [details] Patch Clearing flags on attachment: 159691 Committed r126227: <http://trac.webkit.org/changeset/126227>
WebKit Review Bot
Comment 4 2012-08-21 17:07:34 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.