Bug 198275 - [Pointer Events] Test at pointerevents/ios/touch-action-none-in-overflow-scrolling-touch.html times out
Summary: [Pointer Events] Test at pointerevents/ios/touch-action-none-in-overflow-scro...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: UI Events (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Antoine Quint
URL:
Keywords: InRadar
Depends on: 198310
Blocks:
  Show dependency tree / graph
 
Reported: 2019-05-27 08:31 PDT by Antoine Quint
Modified: 2019-05-28 17:06 PDT (History)
5 users (show)

See Also:

Attachments
Patch (2.04 KB, patch)
2019-05-27 08:34 PDT, Antoine Quint 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Antoine Quint 2019-05-27 08:31:29 PDT 
The test at pointerevents/ios/touch-action-none-in-overflow-scrolling-touch.html is timing out since we switched to the new touch-action implementation with paint-time data.
Comment 1 Antoine Quint 2019-05-27 08:31:41 PDT 
<rdar://problem/50618496>
Comment 2 Antoine Quint 2019-05-27 08:34:36 PDT 
Created attachment 370692 [details]
Patch
Comment 3 Antti Koivisto 2019-05-27 08:40:33 PDT 
Comment on attachment 370692 [details]
Patch

Test?
Comment 4 Antoine Quint 2019-05-27 08:50:14 PDT 
(In reply to Antti Koivisto from comment #3)
> Comment on attachment 370692 [details]
> Patch
> 
> Test?

This fixes an existing test :)
Comment 5 WebKit Commit Bot 2019-05-27 09:19:12 PDT 
Comment on attachment 370692 [details]
Patch

Clearing flags on attachment: 370692

Committed r245795: <https://trac.webkit.org/changeset/245795>
Comment 6 WebKit Commit Bot 2019-05-27 09:19:14 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 7 Antti Koivisto 2019-05-27 10:59:23 PDT 
> This fixes an existing test :)

Was expecting removal from TestExpectations or something.
Comment 8 Antoine Quint 2019-05-28 00:46:21 PDT 
(In reply to Antti Koivisto from comment #7)
> > This fixes an existing test :)
> 
> Was expecting removal from TestExpectations or something.

It was timing out but this had never been marked in TestExpectations which were still expecting a consistent PASS. This is addressed now.
Comment 9 WebKit Commit Bot 2019-05-28 16:21:49 PDT 
Re-opened since this is blocked by bug 198310
Comment 10 Tim Horton 2019-05-28 17:06:36 PDT 
I stopped the rollout