Seen on an iPad running iOS 12 with a hardware keyboard attached.
All modifier keys are false in a non-programmatically-generated mouse event regardless of whether a modifier key was held when the mouse event was created. I noticed this when using the Contact app on iCloud.com. You can observe this issue yourself by performing the following:
1. Visit <https://icloud.com> and sign in.
2. Tap Contacts.
3. Tap on some contact.
4. Hold down the Shift key on the keyboard and tap another contact.
Then all contacts between the first contact tapped in (3) and the last contact tapped in (4) should be selected. But only the last tapped contact in (4) is selected.
(In reply to Daniel Bates from comment #0)
> Seen on an iPad running iOS 12 with a hardware keyboard attached.
>
> All modifier keys are false in a non-programmatically-generated mouse event
> regardless of whether a modifier key was held when the mouse event was
> created. I noticed this when using the Contact app on iCloud.com. You can
> observe this issue yourself by performing the following:
>
> 1. Visit <https://icloud.com> and sign in.
> 2. Tap Contacts.
> 3. Tap on some contact.
> 4. Hold down the Shift key on the keyboard and tap another contact.
>
> Then all contacts between the first contact tapped in (3) and the last
> contact tapped in (4) should be selected. But only the last tapped contact
> in (4) is selected.
iCloud seems to use touch events on iOS and does not check for modifier flags. iCloud will address this in <rdar://problem/47856410>. Re-purposing this bug to wire up modifier keys for mouse, touch/pointer events.
Attachment 361342[details] did not pass style-queue:
ERROR: Source/WebKit/Shared/ios/NativeWebTouchEventIOS.mm:139: Comma should be at the beginning of the line in a member initialization list. [whitespace/init] [4]
Total errors found: 1 in 21 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Attachment 361421[details] did not pass style-queue:
ERROR: Source/WebKit/Shared/ios/NativeWebTouchEventIOS.mm:139: Comma should be at the beginning of the line in a member initialization list. [whitespace/init] [4]
Total errors found: 1 in 24 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Attachment 361427[details] did not pass style-queue:
ERROR: Source/WebKit/Shared/ios/NativeWebTouchEventIOS.mm:125: Comma should be at the beginning of the line in a member initialization list. [whitespace/init] [4]
Total errors found: 1 in 25 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Attachment 361435[details] did not pass style-queue:
ERROR: Source/WebKit/Shared/ios/NativeWebTouchEventIOS.mm:124: Comma should be at the beginning of the line in a member initialization list. [whitespace/init] [4]
Total errors found: 1 in 25 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 361447[details]
Archive of layout-test-results from ews103 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103 Port: mac-highsierra Platform: Mac OS X 10.13.6
Created attachment 361455[details]
Archive of layout-test-results from ews112 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112 Port: mac-highsierra Platform: Mac OS X 10.13.6
Created attachment 361456[details]
Archive of layout-test-results from ews121 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Created attachment 361458[details]
Archive of layout-test-results from ews107 for mac-highsierra-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107 Port: mac-highsierra-wk2 Platform: Mac OS X 10.13.6
Attachment 361538[details] did not pass style-queue:
ERROR: Source/WebKit/Shared/ios/NativeWebTouchEventIOS.mm:124: Comma should be at the beginning of the line in a member initialization list. [whitespace/init] [4]
Total errors found: 1 in 34 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 361551[details]
Archive of layout-test-results from ews100 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100 Port: mac-highsierra Platform: Mac OS X 10.13.6
Created attachment 361553[details]
Archive of layout-test-results from ews107 for mac-highsierra-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107 Port: mac-highsierra-wk2 Platform: Mac OS X 10.13.6
Created attachment 361556[details]
Archive of layout-test-results from ews121 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Created attachment 361575[details]
Archive of layout-test-results from ews113 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews113 Port: mac-highsierra Platform: Mac OS X 10.13.6
Attachment 361637[details] did not pass style-queue:
ERROR: Source/WebKit/Shared/ios/NativeWebTouchEventIOS.mm:124: Comma should be at the beginning of the line in a member initialization list. [whitespace/init] [4]
Total errors found: 1 in 35 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Attachment 361639[details] did not pass style-queue:
ERROR: Source/WebKit/Shared/ios/NativeWebTouchEventIOS.mm:124: Comma should be at the beginning of the line in a member initialization list. [whitespace/init] [4]
Total errors found: 1 in 35 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 361641[details]
Archive of layout-test-results from ews107 for mac-highsierra-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107 Port: mac-highsierra-wk2 Platform: Mac OS X 10.13.6
Created attachment 361642[details]
Archive of layout-test-results from ews124 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Attachment 361646[details] did not pass style-queue:
ERROR: Source/WebKit/Shared/ios/NativeWebTouchEventIOS.mm:124: Comma should be at the beginning of the line in a member initialization list. [whitespace/init] [4]
Total errors found: 1 in 38 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 361646[details]
Patch and layout tests
View in context: https://bugs.webkit.org/attachment.cgi?id=361646&action=review> LayoutTests/http/tests/adClickAttribution/anchor-tag-attributes-validation-expected.txt:-2
> -CONSOLE MESSAGE: line 107: adcampaignid must have a non-negative value less than 64 for Ad Click Attribution.
We should really fix these
2019-02-06 16:08 PST, Daniel Bates
2019-02-07 11:16 PST, Daniel Bates
2019-02-07 12:23 PST, Daniel Bates
2019-02-07 13:31 PST, Daniel Bates
2019-02-07 14:31 PST, EWS Watchlist
2019-02-07 15:01 PST, EWS Watchlist
2019-02-07 15:02 PST, EWS Watchlist
2019-02-07 15:11 PST, EWS Watchlist
2019-02-08 14:51 PST, Daniel Bates
2019-02-08 15:56 PST, EWS Watchlist
2019-02-08 16:02 PST, EWS Watchlist
2019-02-08 16:23 PST, EWS Watchlist
2019-02-08 18:15 PST, EWS Watchlist
2019-02-10 12:36 PST, Daniel Bates
2019-02-10 12:47 PST, Daniel Bates
2019-02-10 14:05 PST, EWS Watchlist
2019-02-10 14:47 PST, EWS Watchlist
2019-02-10 15:09 PST, Daniel Bates
2019-02-11 10:55 PST, Daniel Bates