Bug 187263 - 50 failing WebDriver tests after r233417
Summary: 50 failing WebDriver tests after r233417
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebDriver (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-07-02 13:50 PDT by Michael Catanzaro
Modified: 2018-07-05 09:12 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.89 KB, patch)
2018-07-03 04:06 PDT, Carlos Garcia Campos
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews123 for ios-simulator-wk2 (2.24 MB, application/zip)
2018-07-03 06:01 PDT, EWS Watchlist
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2018-07-02 13:50:33 PDT
About 50 WebDriver tests started crashing after r233417. Would be good to investigate and resolve. I don't see how to get backtraces for WebDriver failures off the bots, unfortunately.

Note: please don't roll out this commit, since we need it to avoid introducing more similar bugs. The tests would already have been crashing if built with GCC 7 or GCC 8, so this commit only served to reveal the bug on our bots that are running GCC 6.
Comment 1 Frédéric Wang (:fredw) 2018-07-03 00:03:43 PDT
I wonder whether this could be due to the crash in the network process (bug 187243). It seems to happen frequently as "Other crashes" when running WK2 layout tests (e.g. https://build.webkit.org/results/Apple%20Sierra%20Debug%20WK2%20(Tests)/r233451%20(7099)/results.html)
Comment 2 Frédéric Wang (:fredw) 2018-07-03 01:08:02 PDT
(In reply to Frédéric Wang (:fredw) from comment #1)
> I wonder whether this could be due to the crash in the network process (bug
> 187243). It seems to happen frequently as "Other crashes" when running WK2
> layout tests (e.g.
> https://build.webkit.org/results/Apple%20Sierra%20Debug%20WK2%20(Tests)/
> r233451%20(7099)/results.html)

Running locally, I get a crash without the patch for bug 187243 and don't get any when I apply the patch. The tests still fail, but I've never run the webdriver tests before so I don't know whether my setup is correct.
Comment 3 Carlos Garcia Campos 2018-07-03 04:06:16 PDT
Created attachment 344175 [details]
Patch
Comment 4 EWS Watchlist 2018-07-03 06:01:38 PDT
Comment on attachment 344175 [details]
Patch

Attachment 344175 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/8423287

New failing tests:
animations/needs-layout.html
Comment 5 EWS Watchlist 2018-07-03 06:01:40 PDT
Created attachment 344179 [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.4
Comment 6 WebKit Commit Bot 2018-07-04 00:43:27 PDT
Comment on attachment 344175 [details]
Patch

Clearing flags on attachment: 344175

Committed r233499: <https://trac.webkit.org/changeset/233499>
Comment 7 WebKit Commit Bot 2018-07-04 00:43:28 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 Radar WebKit Bug Importer 2018-07-04 00:44:25 PDT
<rdar://problem/41810842>
Comment 9 BJ Burg 2018-07-05 09:12:55 PDT
d'oh, good catch!