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

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: Add an inline comment 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 635 matching lines...) Expand 10 before | Expand all | Expand 10 after
646 base::Bind(&self::DispatchInstallEventOnIOThread, this, done, result), 646 base::Bind(&self::DispatchInstallEventOnIOThread, this, done, result),
647 CreateReceiver(BrowserThread::UI, done, result)); 647 CreateReceiver(BrowserThread::UI, done, result));
648 } 648 }
649 649
650 void DispatchInstallEventOnIOThread(const base::Closure& done, 650 void DispatchInstallEventOnIOThread(const base::Closure& done,
651 ServiceWorkerStatusCode* result) { 651 ServiceWorkerStatusCode* result) {
652 ASSERT_TRUE(BrowserThread::CurrentlyOn(BrowserThread::IO)); 652 ASSERT_TRUE(BrowserThread::CurrentlyOn(BrowserThread::IO));
653 int request_id = 653 int request_id =
654 version_->StartRequest(ServiceWorkerMetrics::EventType::INSTALL, 654 version_->StartRequest(ServiceWorkerMetrics::EventType::INSTALL,
655 CreateReceiver(BrowserThread::UI, done, result)); 655 CreateReceiver(BrowserThread::UI, done, result));
656 version_->DispatchEvent<ServiceWorkerHostMsg_InstallEventFinished>( 656 version_
657 request_id, ServiceWorkerMsg_InstallEvent(request_id), 657 ->RegisterRequestCallback<ServiceWorkerHostMsg_InstallEventFinished>(
658 base::Bind(&self::ReceiveInstallEventOnIOThread, this, done, result)); 658 request_id, base::Bind(&self::ReceiveInstallEventOnIOThread, this,
659 done, result));
660 version_->DispatchEvent({request_id},
661 ServiceWorkerMsg_InstallEvent(request_id));
659 } 662 }
660 663
661 void ReceiveInstallEventOnIOThread(const base::Closure& done, 664 void ReceiveInstallEventOnIOThread(const base::Closure& done,
662 ServiceWorkerStatusCode* out_result, 665 ServiceWorkerStatusCode* out_result,
663 int request_id, 666 int request_id,
664 blink::WebServiceWorkerEventResult result, 667 blink::WebServiceWorkerEventResult result,
665 bool has_fetch_handler) { 668 bool has_fetch_handler) {
666 version_->FinishRequest( 669 version_->FinishRequest(
667 request_id, result == blink::WebServiceWorkerEventResultCompleted); 670 request_id, result == blink::WebServiceWorkerEventResultCompleted);
668 version_->set_has_fetch_handler(has_fetch_handler); 671 version_->set_has_fetch_handler(has_fetch_handler);
(...skipping 1198 matching lines...) Expand 10 before | Expand all | Expand 10 after
1867 NavigateToTestPage(); 1870 NavigateToTestPage();
1868 // The V8 code cache must be stored to the CacheStorage which must be bigger 1871 // The V8 code cache must be stored to the CacheStorage which must be bigger
1869 // than 12 byte. 1872 // than 12 byte.
1870 EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize); 1873 EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize);
1871 1874
1872 NavigateToTestPage(); 1875 NavigateToTestPage();
1873 EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize); 1876 EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize);
1874 } 1877 }
1875 1878
1876 } // namespace content 1879 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698