RESOLVED WONTFIX 107834
[DumpRenderTree] Add testRunner.numberOfProgrammaticScrolls
https://bugs.webkit.org/show_bug.cgi?id=107834
Summary [DumpRenderTree] Add testRunner.numberOfProgrammaticScrolls
John Knottenbelt
Reported 2013-01-24 09:03:13 PST
[DumpRenderTree] Add testRunner.numberOfProgrammaticScrolls
Attachments
Patch (13.46 KB, patch)
2013-01-24 09:07 PST, John Knottenbelt
no flags
Patch (15.82 KB, patch)
2013-01-25 07:12 PST, John Knottenbelt
webkit.review.bot: commit-queue-
John Knottenbelt
Comment 1 2013-01-24 09:07:21 PST
WebKit Review Bot
Comment 2 2013-01-24 14:43:48 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.
jochen
Comment 3 2013-01-24 14:59:49 PST
Comment on attachment 184511 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=184511&action=review > Tools/DumpRenderTree/chromium/WebViewHost.cpp:1474 > +void WebViewHost::didProgrammaticallyScroll(const WebPoint&) could you instead add this method to WebTestProxy, and the implementation to WebTestProxyBase. WebTestProxyBase can then expose the value as a simple getter, and the WebViewHost::numberOfProgrammaticScrolls() would return proxy()->numberOfProgrammaticScrolls();
Early Warning System Bot
Comment 4 2013-01-24 15:22:11 PST
Early Warning System Bot
Comment 5 2013-01-24 16:22:41 PST
Build Bot
Comment 6 2013-01-24 18:52:24 PST
Comment on attachment 184511 [details] Patch Attachment 184511 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://queues.webkit.org/results/16120037
kov's GTK+ EWS bot
Comment 7 2013-01-24 19:05:17 PST
Build Bot
Comment 8 2013-01-24 19:51:19 PST
EFL EWS Bot
Comment 9 2013-01-25 00:18:55 PST
Build Bot
Comment 10 2013-01-25 01:31:53 PST
John Knottenbelt
Comment 11 2013-01-25 07:12:26 PST
WebKit Review Bot
Comment 12 2013-01-25 07:15:19 PST
Comment on attachment 184746 [details] Patch Attachment 184746 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/16097807
EFL EWS Bot
Comment 13 2013-01-25 07:16:08 PST
Build Bot
Comment 14 2013-01-25 07:16:19 PST
Early Warning System Bot
Comment 15 2013-01-25 07:17:51 PST
Peter Beverloo (cr-android ews)
Comment 16 2013-01-25 07:18:05 PST
Comment on attachment 184746 [details] Patch Attachment 184746 [details] did not pass cr-android-ews (chromium-android): Output: http://queues.webkit.org/results/16118380
Early Warning System Bot
Comment 17 2013-01-25 07:19:21 PST
Build Bot
Comment 18 2013-01-25 07:39:46 PST
Build Bot
Comment 19 2013-01-25 08:24:43 PST
Comment on attachment 184746 [details] Patch Attachment 184746 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://queues.webkit.org/results/16097830
Bem Jones-Bey
Comment 20 2014-04-16 17:07:46 PDT
Comment on attachment 184746 [details] Patch Cleaning up obviously stale bugs.
Note You need to log in before you can comment on or make changes to this bug.