Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(119)

Side by Side Diff: content/browser/service_worker/service_worker_browsertest.cc

Issue 2034663002: ServiceWorker: Keep the worker alive until FetchEvent.waitUntil settles (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stddef.h> 5 #include <stddef.h>
6 #include <stdint.h> 6 #include <stdint.h>
7 7
8 #include <map> 8 #include <map>
9 #include <memory> 9 #include <memory>
10 #include <utility> 10 #include <utility>
(...skipping 634 matching lines...) Expand 10 before | Expand all | Expand 10 after
645 base::Bind(&self::DispatchInstallEventOnIOThread, this, done, result), 645 base::Bind(&self::DispatchInstallEventOnIOThread, this, done, result),
646 CreateReceiver(BrowserThread::UI, done, result)); 646 CreateReceiver(BrowserThread::UI, done, result));
647 } 647 }
648 648
649 void DispatchInstallEventOnIOThread(const base::Closure& done, 649 void DispatchInstallEventOnIOThread(const base::Closure& done,
650 ServiceWorkerStatusCode* result) { 650 ServiceWorkerStatusCode* result) {
651 ASSERT_TRUE(BrowserThread::CurrentlyOn(BrowserThread::IO)); 651 ASSERT_TRUE(BrowserThread::CurrentlyOn(BrowserThread::IO));
652 int request_id = 652 int request_id =
653 version_->StartRequest(ServiceWorkerMetrics::EventType::INSTALL, 653 version_->StartRequest(ServiceWorkerMetrics::EventType::INSTALL,
654 CreateReceiver(BrowserThread::UI, done, result)); 654 CreateReceiver(BrowserThread::UI, done, result));
655 version_->DispatchEvent<ServiceWorkerHostMsg_InstallEventFinished>( 655 version_
656 request_id, ServiceWorkerMsg_InstallEvent(request_id), 656 ->RegisterRequestCallback<ServiceWorkerHostMsg_InstallEventFinished>(
657 base::Bind(&self::ReceiveInstallEventOnIOThread, this, done, result)); 657 request_id, base::Bind(&self::ReceiveInstallEventOnIOThread, this,
658 done, result));
659 version_->DispatchEvent({request_id},
660 ServiceWorkerMsg_InstallEvent(request_id));
658 } 661 }
659 662
660 void ReceiveInstallEventOnIOThread(const base::Closure& done, 663 void ReceiveInstallEventOnIOThread(const base::Closure& done,
661 ServiceWorkerStatusCode* out_result, 664 ServiceWorkerStatusCode* out_result,
662 int request_id, 665 int request_id,
663 blink::WebServiceWorkerEventResult result, 666 blink::WebServiceWorkerEventResult result,
664 bool has_fetch_handler) { 667 bool has_fetch_handler) {
665 version_->FinishRequest( 668 version_->FinishRequest(
666 request_id, result == blink::WebServiceWorkerEventResultCompleted); 669 request_id, result == blink::WebServiceWorkerEventResultCompleted);
667 version_->set_has_fetch_handler(has_fetch_handler); 670 version_->set_has_fetch_handler(has_fetch_handler);
(...skipping 1198 matching lines...) Expand 10 before | Expand all | Expand 10 after
1866 NavigateToTestPage(); 1869 NavigateToTestPage();
1867 // The V8 code cache must be stored to the CacheStorage which must be bigger 1870 // The V8 code cache must be stored to the CacheStorage which must be bigger
1868 // than 12 byte. 1871 // than 12 byte.
1869 EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize); 1872 EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize);
1870 1873
1871 NavigateToTestPage(); 1874 NavigateToTestPage();
1872 EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize); 1875 EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize);
1873 } 1876 }
1874 1877
1875 } // namespace content 1878 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698