Bug 219024 - WebDriver: add support for pen pointer events
Summary: WebDriver: add support for pen pointer events
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebDriver (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks: 166679
  Show dependency treegraph
 
Reported: 2020-11-17 01:35 PST by Carlos Garcia Campos
Modified: 2020-11-26 03:18 PST (History)
18 users (show)

See Also:


Attachments
WIP patch (101.74 KB, patch)
2020-11-17 01:37 PST, Carlos Garcia Campos
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
WIP patch (102.14 KB, patch)
2020-11-17 02:17 PST, Carlos Garcia Campos
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
WIP patch (102.23 KB, patch)
2020-11-17 02:29 PST, Carlos Garcia Campos
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
WIP patch (140.26 KB, patch)
2020-11-17 06:08 PST, Carlos Garcia Campos
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (149.01 KB, patch)
2020-11-23 03:07 PST, Carlos Garcia Campos
cgarcia: review?
ews-feeder: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2020-11-17 01:35:47 PST
We are not handling mouse events with pen pointer type.
Comment 1 Carlos Garcia Campos 2020-11-17 01:37:51 PST
Created attachment 414318 [details]
WIP patch

This is WIP, just to check if it builds in other ports and the result of imported/w3c/web-platform-tests/pointerevents/pointerevent_attributes_hoverable_pointers.html
Comment 2 EWS Watchlist 2020-11-17 01:38:38 PST
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API

This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Comment 3 Carlos Garcia Campos 2020-11-17 02:17:11 PST
Created attachment 414323 [details]
WIP patch
Comment 4 Carlos Garcia Campos 2020-11-17 02:29:29 PST
Created attachment 414324 [details]
WIP patch
Comment 5 Carlos Garcia Campos 2020-11-17 06:08:54 PST
Created attachment 414334 [details]
WIP patch
Comment 6 Carlos Garcia Campos 2020-11-17 06:48:05 PST
I don't understand the build failures on tv and watch platform. I need some help with those.
Comment 7 Tim Horton 2020-11-17 07:06:39 PST
(In reply to Carlos Garcia Campos from comment #6)
> I don't understand the build failures on tv and watch platform. I need some
> help with those.

It's a classic unified source 'using namespace' conflict. Might be an indirect one (WebCore -> WebKit -> top-level). I've applied your patch and started a build, I'll see if I can find the minimal change required to get things back on track.
Comment 8 Carlos Garcia Campos 2020-11-17 07:10:55 PST
(In reply to Tim Horton from comment #7)
> (In reply to Carlos Garcia Campos from comment #6)
> > I don't understand the build failures on tv and watch platform. I need some
> > help with those.
> 
> It's a classic unified source 'using namespace' conflict. Might be an
> indirect one (WebCore -> WebKit -> top-level). I've applied your patch and
> started a build, I'll see if I can find the minimal change required to get
> things back on track.

Thanks!
Comment 9 Tim Horton 2020-11-17 08:05:36 PST
I landed http://trac.webkit.org/changeset/269901/webkit and hit the retry button here (no idea if I waited long enough between doing those things, though... but it should be OK eventually).
Comment 10 Carlos Garcia Campos 2020-11-18 00:29:41 PST
It worked, thank you!
Comment 11 Carlos Garcia Campos 2020-11-23 03:07:12 PST
Created attachment 414798 [details]
Patch
Comment 12 Carlos Garcia Campos 2020-11-23 05:20:03 PST
Comment on attachment 414798 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=414798&action=review

> LayoutTests/ChangeLog:10
> +        Update expectations of
> +        imported/w3c/web-platform-tests/pointerevents/pointerevent_attributes_hoverable_pointers.html. The test is now
> +        timing out, but because tesdriver doesn't correctly handle the iframe element.

See bug #219255
Comment 13 Radar WebKit Bug Importer 2020-11-26 03:18:19 PST
<rdar://problem/71747688>