tree 6c1ce9d9b93b651e3a32f6ef8838bd8667c1ca8b
parent 7f2c6472cc3acc549eae52da5a6ef449445122ef
author beidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc> 1570564212 +0000
committer beidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc> 1570564212 +0000

Service Worker Fetch events should time out.
https://bugs.webkit.org/show_bug.cgi?id=202188

Reviewed by Alex Christensen.

Source/WebCore:

Test: http/tests/workers/service/basic-timeout.https.html

* workers/service/server/SWServer.h:
* workers/service/server/SWServerWorker.h:
(WebCore::SWServerWorker::setHasTimedOutAnyFetchTasks):
(WebCore::SWServerWorker::hasTimedOutAnyFetchTasks const):

Source/WebKit:

When we start a fetch task in the server, we also start a timeout on that fetch task.

"Time out" means the fetch task must continue to make progress at the given frequency (once every 60 seconds by default)

If any given fetch task times out in a service worker instance, that instance loses the right to handle fetches.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::setServiceWorkerFetchTimeoutForTesting):
(WebKit::NetworkProcess::resetServiceWorkerFetchTimeoutForTesting):
* NetworkProcess/NetworkProcess.h:
(WebKit::NetworkProcess::serviceWorkerFetchTimeout const):
* NetworkProcess/NetworkProcess.messages.in:

* NetworkProcess/ServiceWorker/ServiceWorkerFetchTask.cpp:
(WebKit::ServiceWorkerFetchTask::ServiceWorkerFetchTask):
(WebKit::ServiceWorkerFetchTask::didReceiveRedirectResponse):
(WebKit::ServiceWorkerFetchTask::didReceiveResponse):
(WebKit::ServiceWorkerFetchTask::didReceiveData):
(WebKit::ServiceWorkerFetchTask::didReceiveFormData):
(WebKit::ServiceWorkerFetchTask::didFinish):
(WebKit::ServiceWorkerFetchTask::didFail):
(WebKit::ServiceWorkerFetchTask::didNotHandle):
* NetworkProcess/ServiceWorker/ServiceWorkerFetchTask.h:
(WebKit::ServiceWorkerFetchTask::create):
(WebKit::ServiceWorkerFetchTask::serviceWorkerIdentifier const):
(WebKit::ServiceWorkerFetchTask::wasHandled const):
(WebKit::ServiceWorkerFetchTask::ServiceWorkerFetchTask): Deleted.

* NetworkProcess/ServiceWorker/WebSWServerConnection.cpp:
(WebKit::WebSWServerConnection::startFetch):
* NetworkProcess/ServiceWorker/WebSWServerConnection.h:

* NetworkProcess/ServiceWorker/WebSWServerToContextConnection.cpp:
(WebKit::WebSWServerToContextConnection::startFetch):
(WebKit::WebSWServerToContextConnection::fetchTaskTimedOut):
* NetworkProcess/ServiceWorker/WebSWServerToContextConnection.h:

* UIProcess/API/C/WKContext.cpp:
(WKContextSetServiceWorkerFetchTimeoutForTesting):
(WKContextResetServiceWorkerFetchTimeoutForTesting):
* UIProcess/API/C/WKContext.h:

* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::setServiceWorkerTimeoutForTesting):
(WebKit::WebProcessPool::resetServiceWorkerTimeoutForTesting):
* UIProcess/WebProcessPool.h:

Tools:

* WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
* WebKitTestRunner/InjectedBundle/TestRunner.cpp:
(WTR::TestRunner::setServiceWorkerFetchTimeout):
* WebKitTestRunner/InjectedBundle/TestRunner.h:
* WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetStateToConsistentValues):
(WTR::TestController::setServiceWorkerFetchTimeoutForTesting):
* WebKitTestRunner/TestController.h:
* WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::didReceiveSynchronousMessageFromInjectedBundle):

LayoutTests:

* http/tests/workers/service/basic-timeout.https-expected.txt: Added.
* http/tests/workers/service/basic-timeout.https.html: Added.
* http/tests/workers/service/resources/basic-timeout-worker.js: Added.
(event.event.request.url.indexOf):
* http/tests/workers/service/resources/basic-timeout.js: Added.
(async.test.finishThisTest):
(async.test.try):
(async.test.try.checkSuccessAgain):
(async.test):
* http/tests/workers/service/resources/succeed-fallback-check.php: Added.
* http/tests/workers/service/resources/timeout-fallback.html: Added.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@250852 268f45cc-cd09-0410-ab3c-d52691b4dbfc