Attachment 361247[details] did not pass style-queue:
ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5]
Total errors found: 1 in 3 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 361269[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 361728[details] did not pass style-queue:
ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5]
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 361753[details]
Archive of layout-test-results from ews123 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Attachment 361816[details] did not pass style-queue:
ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5]
Total errors found: 1 in 42 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 361821[details]
Archive of layout-test-results from ews104 for mac-highsierra-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104 Port: mac-highsierra-wk2 Platform: Mac OS X 10.13.6
Created attachment 361823[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 361831[details]
Archive of layout-test-results from ews202 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews202 Port: win-future Platform: CYGWIN_NT-6.1-2.10.0-0.325-5-3-x86_64-64bit
Created attachment 361834[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
Created attachment 361867[details]
Archive of layout-test-results from ews101 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101 Port: mac-highsierra Platform: Mac OS X 10.13.6
Created attachment 361877[details]
Archive of layout-test-results from ews104 for mac-highsierra-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104 Port: mac-highsierra-wk2 Platform: Mac OS X 10.13.6
Created attachment 361887[details]
Archive of layout-test-results from ews126 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews126 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Created attachment 361888[details]
Archive of layout-test-results from ews202 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews202 Port: win-future Platform: CYGWIN_NT-6.1-2.10.0-0.325-5-3-x86_64-64bit
Created attachment 361960[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 361968[details]
Archive of layout-test-results from ews104 for mac-highsierra-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104 Port: mac-highsierra-wk2 Platform: Mac OS X 10.13.6
Created attachment 361975[details]
Archive of layout-test-results from ews115 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115 Port: mac-highsierra Platform: Mac OS X 10.13.6
Created attachment 361977[details]
Archive of layout-test-results from ews125 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Created attachment 362003[details]
Archive of layout-test-results from ews205 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews205 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Created attachment 362032[details]
Archive of layout-test-results from ews204 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews204 Port: win-future Platform: CYGWIN_NT-6.1-2.10.0-0.325-5-3-x86_64-64bit
Created attachment 362035[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 362082[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 362087[details]
Archive of layout-test-results from ews117 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117 Port: mac-highsierra Platform: Mac OS X 10.13.6
Created attachment 362088[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 362112[details]
Archive of layout-test-results from ews201 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews201 Port: win-future Platform: CYGWIN_NT-6.1-2.10.0-0.325-5-3-x86_64-64bit
Created attachment 362125[details]
Archive of layout-test-results from ews126 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews126 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Created attachment 362131[details]
Archive of layout-test-results from ews206 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews206 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Created attachment 362142[details]
Archive of layout-test-results from ews125 for ios-simulator-wk2
The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Created attachment 362145[details]
Archive of layout-test-results from ews205 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews205 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Created attachment 362150[details]
Archive of layout-test-results from ews202 for win-future
The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews202 Port: win-future Platform: CYGWIN_NT-6.1-2.10.0-0.325-5-3-x86_64-64bit
Comment on attachment 362151[details]
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=362151&action=review> Source/WebCore/editing/Editor.cpp:519
> +#if PLATFORM(IOS)
> + return true;
> +#else
Instead of a compilation time flag, please use the editing behavior flag.
> LayoutTests/editing/deleting/smart-delete-003-expected.txt:19
> +foo baz
This doesn't look right. We have nbsp & space between "foo" and "baz". That would show up as two spaces.
> LayoutTests/editing/deleting/smart-delete-003.html:1
> +<html>
Missing DOCTYPE.
> LayoutTests/editing/deleting/smart-delete-003.html:19
> <script src=../editing.js language="JavaScript" type="text/JavaScript" ></script>
While we're at it, we might as well as just delete language & type.
> LayoutTests/editing/deleting/smart-delete-003.html:36
> + await UIHelper.selectWordByDoubleTapOrClick(target);
selectWordByDoubleTapOrClick adds 5px padding to each position. Maybe that's too much for this test case?
> LayoutTests/editing/deleting/smart-delete-004-expected.txt:19
> +foo baz
Ditto. This has NBSP & SP.
> LayoutTests/editing/deleting/smart-delete-004.html:1
> +<html>
Missing DOCTYPE.
> LayoutTests/editing/deleting/smart-delete-004.html:19
> <script src=../editing.js language="JavaScript" type="text/JavaScript" ></script>
Ditto.
> LayoutTests/editing/selection/delete-word-granularity-text-control.html:34
> +if (!UIHelper.isIOS()) {
I think once we made the change to use the editing behavior as the way of toggling smart delete,
the right fix for this test is probably to just force the Mac editing behavior even on iOS.
Since the whole point of this test is to ensure smart delete doesn't kick in this particular circumstances,
it doesn't make much sense to test iOS behavior here.
> LayoutTests/editing/selection/delete-word-granularity-text-control.html:41
> +}
> +// However, on iOS, we have turned on smart delete in all cases, so it should delete the space
> +else {
Please put the comment inside the else clause, not between curly braces.
> LayoutTests/platform/ios/editing/deleting/delete-block-merge-contents-012-expected.txt:45
> + text run at (0,1) width 170: "This is some text."
This is a nice improvement.
Comment on attachment 362151[details]
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=362151&action=review>> LayoutTests/editing/deleting/smart-delete-003-expected.txt:19
>> +foo baz
>
> This doesn't look right. We have nbsp & space between "foo" and "baz". That would show up as two spaces.
Never mind that. I was just getting fooled by the review cool. It's actually a single NBSP.
>> LayoutTests/editing/deleting/smart-delete-004-expected.txt:19
>> +foo baz
>
> Ditto. This has NBSP & SP.
Ditto. Not an issue.
Comment on attachment 362151[details]
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=362151&action=review> LayoutTests/resources/ui-helper.js:130
> + if (this.isIOS()) {
> + await UIHelper.doubleTapAt(x, y);
No curly braces around a single line statement.
> LayoutTests/resources/ui-helper.js:133
> + } else {
> + await UIHelper.doubleClickAt(x, y);
> + }
Ditto.
> LayoutTests/resources/ui-helper.js:140
> + var rects = window.getSelection().getRangeAt(0).getClientRects();
> + var x = rects[0].left;
> + var y = rects[0].top;
Use const.
> LayoutTests/resources/ui-helper.js:149
> + } else {
> + await UIHelper.doubleClickAt(x, y);
> + }
No curly braces around a single line statement.
2019-02-05 16:46 PST, Megan Gardner
2019-02-05 19:44 PST, EWS Watchlist
2019-02-11 16:07 PST, Megan Gardner
2019-02-11 16:18 PST, Megan Gardner
2019-02-11 19:01 PST, EWS Watchlist
2019-02-12 11:49 PST, Megan Gardner
2019-02-12 12:50 PST, EWS Watchlist
2019-02-12 12:54 PST, EWS Watchlist
2019-02-12 13:45 PST, EWS Watchlist
2019-02-12 13:51 PST, EWS Watchlist
2019-02-12 15:55 PST, Megan Gardner
2019-02-12 16:52 PST, EWS Watchlist
2019-02-12 17:45 PST, EWS Watchlist
2019-02-12 18:31 PST, EWS Watchlist
2019-02-12 18:55 PST, EWS Watchlist
2019-02-13 15:52 PST, Megan Gardner
2019-02-13 16:35 PST, EWS Watchlist
2019-02-13 17:00 PST, EWS Watchlist
2019-02-13 17:40 PST, EWS Watchlist
2019-02-13 17:51 PST, EWS Watchlist
2019-02-14 01:02 PST, EWS Watchlist
2019-02-14 08:31 PST, Megan Gardner
2019-02-14 10:22 PST, EWS Watchlist
2019-02-14 10:38 PST, EWS Watchlist
2019-02-14 16:04 PST, Megan Gardner
2019-02-14 17:07 PST, EWS Watchlist
2019-02-14 17:45 PST, EWS Watchlist
2019-02-14 18:33 PST, EWS Watchlist
2019-02-15 05:35 PST, EWS Watchlist
2019-02-15 08:41 PST, Megan Gardner
2019-02-15 09:54 PST, Megan Gardner
2019-02-15 10:34 PST, EWS Watchlist
2019-02-15 10:48 PST, Megan Gardner
2019-02-15 10:52 PST, EWS Watchlist
2019-02-15 11:10 PST, Megan Gardner
2019-02-15 12:02 PST, EWS Watchlist
2019-02-15 12:24 PST, EWS Watchlist
2019-02-15 13:20 PST, EWS Watchlist
2019-02-15 13:33 PST, Megan Gardner
2019-02-15 16:48 PST, Megan Gardner
2019-02-18 08:15 PST, Megan Gardner