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

Side by Side Diff: content/browser/frame_host/navigation_handle_impl.cc

Issue 1721813002: Adding DRP specfic UMA for FirstContentfulPaint (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits Created 4 years, 7 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/frame_host/navigation_handle_impl.h" 5 #include "content/browser/frame_host/navigation_handle_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "content/browser/frame_host/ancestor_throttle.h" 9 #include "content/browser/frame_host/ancestor_throttle.h"
10 #include "content/browser/frame_host/frame_tree_node.h" 10 #include "content/browser/frame_host/frame_tree_node.h"
(...skipping 251 matching lines...) Expand 10 before | Expand all | Expand 10 after
262 WillRedirectRequest(new_url, new_method_is_post ? "POST" : "GET", 262 WillRedirectRequest(new_url, new_method_is_post ? "POST" : "GET",
263 new_referrer_url, new_is_external_protocol, 263 new_referrer_url, new_is_external_protocol,
264 scoped_refptr<net::HttpResponseHeaders>(), 264 scoped_refptr<net::HttpResponseHeaders>(),
265 base::Bind(&UpdateThrottleCheckResult, &result)); 265 base::Bind(&UpdateThrottleCheckResult, &result));
266 266
267 // Reset the callback to ensure it will not be called later. 267 // Reset the callback to ensure it will not be called later.
268 complete_callback_.Reset(); 268 complete_callback_.Reset();
269 return result; 269 return result;
270 } 270 }
271 271
272 NavigationData* NavigationHandleImpl::GetNavigationData() {
273 return navigation_data_.get();
274 }
275
272 void NavigationHandleImpl::InitServiceWorkerHandle( 276 void NavigationHandleImpl::InitServiceWorkerHandle(
273 ServiceWorkerContextWrapper* service_worker_context) { 277 ServiceWorkerContextWrapper* service_worker_context) {
274 DCHECK(IsBrowserSideNavigationEnabled()); 278 DCHECK(IsBrowserSideNavigationEnabled());
275 service_worker_handle_.reset( 279 service_worker_handle_.reset(
276 new ServiceWorkerNavigationHandle(service_worker_context)); 280 new ServiceWorkerNavigationHandle(service_worker_context));
277 } 281 }
278 282
279 void NavigationHandleImpl::WillStartRequest( 283 void NavigationHandleImpl::WillStartRequest(
280 const std::string& method, 284 const std::string& method,
281 const Referrer& sanitized_referrer, 285 const Referrer& sanitized_referrer,
(...skipping 216 matching lines...) Expand 10 before | Expand all | Expand 10 after
498 complete_callback_.Reset(); 502 complete_callback_.Reset();
499 503
500 if (!callback.is_null()) 504 if (!callback.is_null())
501 callback.Run(result); 505 callback.Run(result);
502 506
503 // No code after running the callback, as it might have resulted in our 507 // No code after running the callback, as it might have resulted in our
504 // destruction. 508 // destruction.
505 } 509 }
506 510
507 } // namespace content 511 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/frame_host/navigation_handle_impl.h ('k') | content/browser/frame_host/navigation_request.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698