Bug 93286 - [chromium] Add support to DumpRenderTree [EventSender] for GestureTapDown events.
Summary: [chromium] Add support to DumpRenderTree [EventSender] for GestureTapDown eve...
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: W. James MacLean
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-06 11:38 PDT by W. James MacLean
Modified: 2012-08-07 12:26 PDT (History)
3 users (show)

See Also:


Attachments
Patch (5.95 KB, patch)
2012-08-06 11:43 PDT, W. James MacLean
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description W. James MacLean 2012-08-06 11:38:58 PDT
[chromium] Add support to DumpRenderTree [EventSender] for GestureTapDown events.
Comment 1 W. James MacLean 2012-08-06 11:43:17 PDT
Created attachment 156732 [details]
Patch
Comment 2 Robert Kroeger 2012-08-07 07:55:15 PDT
Comment on attachment 156732 [details]
Patch

looks reasonable to me.
Comment 3 W. James MacLean 2012-08-07 08:37:35 PDT
I was thinking it might be useful be able to pass an additional piece of information in GestureTapDown, namely to use deltaX (or some additional field) to signal that a DRT test is in progress - this would simplify handling pixel tests for the link highlighter (which normally will be changing opacity in an unsynchronized way ...). Normally GestureTapDown would ignore deltaX.

Is this reasonable? Or should we create a new field in the gesture event object?
Comment 4 James Robinson 2012-08-07 11:17:18 PDT
I don't think that plumbing data through is necessary (or a good pattern to follow) - we have layoutTestMode, and generally speaking our test code should use as much of the same logic as our production code.
Comment 5 James Robinson 2012-08-07 11:18:02 PDT
Comment on attachment 156732 [details]
Patch

R=me
Comment 6 W. James MacLean 2012-08-07 12:19:42 PDT
(In reply to comment #4)
> I don't think that plumbing data through is necessary (or a good pattern to follow) - we have layoutTestMode, and generally speaking our test code should use as much of the same logic as our production code.

Looks like layoutTestMode() is just what the doctor ordered, so we're good!
Comment 7 WebKit Review Bot 2012-08-07 12:26:11 PDT
Comment on attachment 156732 [details]
Patch

Clearing flags on attachment: 156732

Committed r124906: <http://trac.webkit.org/changeset/124906>
Comment 8 WebKit Review Bot 2012-08-07 12:26:15 PDT
All reviewed patches have been landed.  Closing bug.