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

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

Issue 1294243004: PlzNavigate: Make ServiceWorker work with PlzNavigate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review comments Created 5 years, 2 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "content/browser/service_worker/service_worker_dispatcher_host.h" 5 #include "content/browser/service_worker/service_worker_dispatcher_host.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/profiler/scoped_tracker.h" 8 #include "base/profiler/scoped_tracker.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "base/trace_event/trace_event.h" 10 #include "base/trace_event/trace_event.h"
(...skipping 715 matching lines...) Expand 10 before | Expand all | Expand 10 after
726 "477117 ServiceWorkerDispatcherHost::OnProviderCreated")); 726 "477117 ServiceWorkerDispatcherHost::OnProviderCreated"));
727 TRACE_EVENT0("ServiceWorker", 727 TRACE_EVENT0("ServiceWorker",
728 "ServiceWorkerDispatcherHost::OnProviderCreated"); 728 "ServiceWorkerDispatcherHost::OnProviderCreated");
729 if (!GetContext()) 729 if (!GetContext())
730 return; 730 return;
731 if (GetContext()->GetProviderHost(render_process_id_, provider_id)) { 731 if (GetContext()->GetProviderHost(render_process_id_, provider_id)) {
732 bad_message::ReceivedBadMessage(this, 732 bad_message::ReceivedBadMessage(this,
733 bad_message::SWDH_PROVIDER_CREATED_NO_HOST); 733 bad_message::SWDH_PROVIDER_CREATED_NO_HOST);
734 return; 734 return;
735 } 735 }
736 scoped_ptr<ServiceWorkerProviderHost> provider_host( 736
737 new ServiceWorkerProviderHost(render_process_id_, route_id, provider_id, 737 scoped_ptr<ServiceWorkerProviderHost> provider_host;
738 provider_type, GetContext()->AsWeakPtr(), 738 if (ServiceWorkerUtils::IsBrowserAssignedProviderId(provider_id)) {
739 this)); 739 // PlzNavigate
740 // Retrieve the provider host previously created for navigation requests.
741 provider_host = GetContext()->TakeNavigationProviderHost(provider_id);
742 if (provider_host == nullptr) {
743 bad_message::ReceivedBadMessage(
744 this, bad_message::SWDH_PROVIDER_CREATED_NO_HOST);
745 return;
746 }
747 DCHECK_EQ(SERVICE_WORKER_PROVIDER_FOR_WINDOW, provider_type);
748 provider_host->CompleteNavigationInitialized(render_process_id_, route_id,
749 this);
750 } else {
751 provider_host =
752 scoped_ptr<ServiceWorkerProviderHost>(new ServiceWorkerProviderHost(
753 render_process_id_, route_id, provider_id, provider_type,
754 GetContext()->AsWeakPtr(), this));
755 }
740 GetContext()->AddProviderHost(provider_host.Pass()); 756 GetContext()->AddProviderHost(provider_host.Pass());
741 } 757 }
742 758
743 void ServiceWorkerDispatcherHost::OnProviderDestroyed(int provider_id) { 759 void ServiceWorkerDispatcherHost::OnProviderDestroyed(int provider_id) {
744 TRACE_EVENT0("ServiceWorker", 760 TRACE_EVENT0("ServiceWorker",
745 "ServiceWorkerDispatcherHost::OnProviderDestroyed"); 761 "ServiceWorkerDispatcherHost::OnProviderDestroyed");
746 if (!GetContext()) 762 if (!GetContext())
747 return; 763 return;
748 if (!GetContext()->GetProviderHost(render_process_id_, provider_id)) { 764 if (!GetContext()->GetProviderHost(render_process_id_, provider_id)) {
749 bad_message::ReceivedBadMessage( 765 bad_message::ReceivedBadMessage(
(...skipping 542 matching lines...) Expand 10 before | Expand all | Expand 10 after
1292 if (!handle) { 1308 if (!handle) {
1293 bad_message::ReceivedBadMessage(this, 1309 bad_message::ReceivedBadMessage(this,
1294 bad_message::SWDH_TERMINATE_BAD_HANDLE); 1310 bad_message::SWDH_TERMINATE_BAD_HANDLE);
1295 return; 1311 return;
1296 } 1312 }
1297 handle->version()->StopWorker( 1313 handle->version()->StopWorker(
1298 base::Bind(&ServiceWorkerUtils::NoOpStatusCallback)); 1314 base::Bind(&ServiceWorkerUtils::NoOpStatusCallback));
1299 } 1315 }
1300 1316
1301 } // namespace content 1317 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698