Bug 47478

Summary: Add 'didDraw' callback to WebKit for framerate tracking
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: ASSIGNED ---    
Severity: Normal CC: abarth, bdakin, sam, simon.fraser, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: Windows 7   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description Simon Fraser (smfr) 2010-10-10 19:25:19 PDT
In order to track the rate at which the WebView is drawing to the screen, we need a 'didDraw' callback.
Comment 1 Simon Fraser (smfr) 2010-10-12 13:56:23 PDT
Created attachment 70562 [details]
Patch
Comment 2 Simon Fraser (smfr) 2010-10-12 14:07:53 PDT
http://trac.webkit.org/changeset/69600 for Mac.

Keeping open for Windows and WebKit2.
Comment 3 Eric Seidel (no email) 2010-10-14 07:19:32 PDT
Comment on attachment 70562 [details]
Patch

Cleared Sam Weinig's review+ from obsolete attachment 70562 [details] so that this bug does not appear in http://webkit.org/pending-commit.
Comment 4 Simon Fraser (smfr) 2010-10-15 15:09:07 PDT
Created attachment 70903 [details]
Patch
Comment 5 WebKit Review Bot 2010-10-15 15:12:03 PDT
Attachment 70903 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
WebKit2/UIProcess/API/C/WKPage.h:163:  Extra space between WKPageDidDraw and didDraw  [whitespace/declaration] [3]
Total errors found: 1 in 7 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 Early Warning System Bot 2010-10-15 15:16:54 PDT
Attachment 70903 [details] did not build on qt:
Build output: http://queues.webkit.org/results/4456051
Comment 7 Simon Fraser (smfr) 2010-10-15 15:20:12 PDT
Created attachment 70905 [details]
Patch
Comment 8 Simon Fraser (smfr) 2010-10-15 15:35:07 PDT
Comment on attachment 70905 [details]
Patch

http://trac.webkit.org/changeset/69892
Comment 9 Simon Fraser (smfr) 2010-10-15 15:36:16 PDT
Keep open for Windows.
Comment 10 Adam Barth 2010-10-18 15:55:50 PDT
To be clear, this is for the embedder, not for JavaScript, right?
Comment 11 Simon Fraser (smfr) 2010-10-18 16:35:19 PDT
Correct.
Comment 12 Alexey Proskuryakov 2013-10-03 11:57:13 PDT
Is this still needed?