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

Unified Diff: content/renderer/service_worker/service_worker_context_client.cc

Issue 2654433012: Revert of Show service worker navigation preload requests in DevTools Network tab (Closed)
Patch Set: rebase again Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: content/renderer/service_worker/service_worker_context_client.cc
diff --git a/content/renderer/service_worker/service_worker_context_client.cc b/content/renderer/service_worker/service_worker_context_client.cc
index 2e79d0dc687d3d40b5d2253f940107f7acb61dfe..d639084d7a823a00fade6ec3ef01279821575357 100644
--- a/content/renderer/service_worker/service_worker_context_client.cc
+++ b/content/renderer/service_worker/service_worker_context_client.cc
@@ -286,7 +286,7 @@ class ServiceWorkerContextClient::NavigationPreloadRequest final
const ResourceResponseHead& response_head) override {
// This will delete |this|.
ReportErrorToClient(
- "Service Worker navigation preload doesn't support redirects.");
+ "Service Worker navigation preload doesn't suport redirect.");
}
void OnDataDownloaded(int64_t data_length,
@@ -303,7 +303,7 @@ class ServiceWorkerContextClient::NavigationPreloadRequest final
void OnReceiveCachedMetadata(const std::vector<uint8_t>& data) override {}
void OnTransferSizeUpdated(int32_t transfer_size_diff) override {
- // TODO(horo): Send this transfer size update notification to DevTools.
+ NOTREACHED();
}
void OnStartLoadingResponseBody(
@@ -325,8 +325,7 @@ class ServiceWorkerContextClient::NavigationPreloadRequest final
if (!client)
return;
// This will delete |this|.
- client->OnNavigationPreloadComplete(fetch_event_id_,
- status.encoded_data_length);
+ client->OnNavigationPreloadComplete(fetch_event_id_);
}
private:
@@ -959,12 +958,11 @@ void ServiceWorkerContextClient::DispatchFetchEvent(
const ServiceWorkerFetchRequest& request,
mojom::FetchEventPreloadHandlePtr preload_handle,
const DispatchFetchEventCallback& callback) {
- std::unique_ptr<NavigationPreloadRequest> preload_request;
- if (preload_handle) {
- proxy_->onNavigationPreloadSent(fetch_event_id, request.url);
- preload_request = base::MakeUnique<NavigationPreloadRequest>(
- fetch_event_id, request.url, std::move(preload_handle));
- }
+ std::unique_ptr<NavigationPreloadRequest> preload_request =
+ preload_handle
+ ? base::MakeUnique<NavigationPreloadRequest>(
+ fetch_event_id, request.url, std::move(preload_handle))
+ : nullptr;
const bool navigation_preload_sent = !!preload_request;
blink::WebServiceWorkerRequest webRequest;
TRACE_EVENT0("ServiceWorker",
@@ -1253,9 +1251,7 @@ void ServiceWorkerContextClient::OnNavigationPreloadError(
}
void ServiceWorkerContextClient::OnNavigationPreloadComplete(
- int fetch_event_id,
- int64_t encoded_data_length) {
- proxy_->onNavigationPreloadCompleted(fetch_event_id, encoded_data_length);
+ int fetch_event_id) {
context_->preload_requests.Remove(fetch_event_id);
}

Powered by Google App Engine
This is Rietveld 408576698