Bug 160510 - LayoutTest http/tests/fetch/fetch-in-worker-crash.html is flaky
Summary: LayoutTest http/tests/fetch/fetch-in-worker-crash.html is flaky
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-03 09:31 PDT by Ryan Haddad
Modified: 2016-08-22 00:11 PDT (History)
4 users (show)

See Also:


Attachments
Patch (3.04 KB, patch)
2016-08-04 13:19 PDT, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for landing (1.45 KB, patch)
2016-08-07 00:13 PDT, youenn fablet
no flags Details | Formatted Diff | Diff
Fixing failure flakiness (1.66 KB, patch)
2016-08-21 14:36 PDT, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2016-08-03 09:31:45 PDT
LayoutTest http/tests/fetch/fetch-in-worker-crash.html is flaky

Crash:
https://build.webkit.org/results/Apple%20El%20Capitan%20Release%20WK2%20(Tests)/r204079%20(8190)/results.html

Failure:
https://build.webkit.org/results/Apple%20El%20Capitan%20Debug%20WK1%20(Tests)/r204075%20(7134)/results.html

Flakiness dashboard:
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Ffetch%2Ffetch-in-worker-crash.html

Thread 26 Crashed:: WebCore: Worker
0   com.apple.WebCore             	0x00000001095bf3bf WebCore::WorkerThreadableLoader::MainThreadBridge::cancel() + 335 (WorkerThreadableLoader.cpp:159)
1   com.apple.WebCore             	0x000000010881a8dd WebCore::FetchResponse::stop() + 221 (FetchResponse.cpp:300)
2   com.apple.WebCore             	0x000000010925ffe7 WebCore::ScriptExecutionContext::stopActiveDOMObjects() + 343 (ScriptExecutionContext.cpp:298)
3   com.apple.WebCore             	0x00000001095be730 WTF::Function<void (WebCore::ScriptExecutionContext&)>::CallableWrapper<WebCore::WorkerThread::stop()::$_0>::call(WebCore::ScriptExecutionContext&) + 32 (Function.h:101)
4   com.apple.WebCore             	0x00000001095bb5b3 WebCore::WorkerRunLoop::runCleanupTasks(WebCore::WorkerGlobalScope*) + 211 (memory:2459)
5   com.apple.WebCore             	0x00000001095bb2bf WebCore::WorkerRunLoop::run(WebCore::WorkerGlobalScope*) + 111 (WorkerRunLoop.cpp:134)
6   com.apple.WebCore             	0x00000001095be3be WebCore::WorkerThread::workerThread() + 718 (WorkerThread.cpp:186)
7   com.apple.JavaScriptCore      	0x0000000107e19f52 WTF::threadEntryPoint(void*) + 178 (Threading.cpp:61)
8   com.apple.JavaScriptCore      	0x0000000107e1a36f WTF::wtfThreadEntryPoint(void*) + 15 (ThreadingPthreads.cpp:166)
9   libsystem_pthread.dylib       	0x00007fff8eae399d _pthread_body + 131
10  libsystem_pthread.dylib       	0x00007fff8eae391a _pthread_start + 168
11  libsystem_pthread.dylib       	0x00007fff8eae1351 thread_start + 13


--- /Volumes/Data/slave/elcapitan-debug-tests-wk1/build/layout-test-results/http/tests/fetch/fetch-in-worker-crash-expected.txt
+++ /Volumes/Data/slave/elcapitan-debug-tests-wk1/build/layout-test-results/http/tests/fetch/fetch-in-worker-crash-actual.txt
@@ -1,5 +1,5 @@
 CONSOLE MESSAGE: line 11: 0
-CONSOLE MESSAGE: line 11: 1
+CONSOLE MESSAGE: line 11: FAIL: promise 1 resolved
 Test that terminating an ongoing fetch in a worker does not cause a crash.
 
 PASS: No crash.
Comment 1 youenn fablet 2016-08-03 09:36:13 PDT

*** This bug has been marked as a duplicate of bug 160498 ***
Comment 2 youenn fablet 2016-08-04 13:14:43 PDT
Test is less flaky but still flaky. Reopening the bug
Comment 3 youenn fablet 2016-08-04 13:19:30 PDT
Created attachment 285354 [details]
Patch
Comment 4 WebKit Commit Bot 2016-08-06 01:49:55 PDT
Comment on attachment 285354 [details]
Patch

Clearing flags on attachment: 285354

Committed r204223: <http://trac.webkit.org/changeset/204223>
Comment 5 WebKit Commit Bot 2016-08-06 01:49:58 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Alexey Proskuryakov 2016-08-06 21:23:17 PDT
This test became much more flaky after this fix, and it still crashes too.

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Ffetch%2Ffetch-in-worker-crash.html

https://build.webkit.org/results/Apple%20iOS%209%20Simulator%20Release%20WK1%20(Tests)/r204235%20(7920)/DumpRenderTree-28112-crash-log.txt

Could you please take a look soon? The test is so flaky that it's likely to cause EWS false positives.
Comment 7 youenn fablet 2016-08-07 00:13:09 PDT
Created attachment 285523 [details]
Patch for landing
Comment 8 youenn fablet 2016-08-07 00:17:51 PDT
(In reply to comment #7)
> Created attachment 285523 [details]
> Patch for landing

Marked test as flaky.
I'll investigate why it is flaky in two weeks from now.
From what I see, there is a problem with the readable stream when stopping the loads, that is causing unnecessary JavaScript execution.
Comment 9 WebKit Commit Bot 2016-08-07 00:42:05 PDT
Comment on attachment 285523 [details]
Patch for landing

Clearing flags on attachment: 285523

Committed r204240: <http://trac.webkit.org/changeset/204240>
Comment 10 WebKit Commit Bot 2016-08-07 00:42:10 PDT
All reviewed patches have been landed.  Closing bug.
Comment 11 youenn fablet 2016-08-07 00:47:15 PDT
Reopened to fix issue with ReadableStream.
Comment 12 youenn fablet 2016-08-21 14:36:51 PDT
Created attachment 286573 [details]
Fixing failure flakiness
Comment 13 WebKit Commit Bot 2016-08-22 00:11:32 PDT
Comment on attachment 286573 [details]
Fixing failure flakiness

Clearing flags on attachment: 286573

Committed r204704: <http://trac.webkit.org/changeset/204704>
Comment 14 WebKit Commit Bot 2016-08-22 00:11:36 PDT
All reviewed patches have been landed.  Closing bug.