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

Side by Side Diff: content/browser/loader/resource_loader.cc

Issue 1721813002: Adding DRP specfic UMA for FirstContentfulPaint (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changing lifetime of NavigationSupportsUserData for Draft Created 4 years, 9 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/loader/resource_loader.h" 5 #include "content/browser/loader/resource_loader.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/location.h" 10 #include "base/location.h"
11 #include "base/metrics/histogram.h" 11 #include "base/metrics/histogram.h"
12 #include "base/profiler/scoped_tracker.h" 12 #include "base/profiler/scoped_tracker.h"
13 #include "base/single_thread_task_runner.h" 13 #include "base/single_thread_task_runner.h"
14 #include "base/thread_task_runner_handle.h" 14 #include "base/thread_task_runner_handle.h"
15 #include "base/time/time.h" 15 #include "base/time/time.h"
16 #include "content/browser/appcache/appcache_interceptor.h" 16 #include "content/browser/appcache/appcache_interceptor.h"
17 #include "content/browser/child_process_security_policy_impl.h" 17 #include "content/browser/child_process_security_policy_impl.h"
18 #include "content/browser/frame_host/navigation_user_data.h"
18 #include "content/browser/loader/cross_site_resource_handler.h" 19 #include "content/browser/loader/cross_site_resource_handler.h"
19 #include "content/browser/loader/detachable_resource_handler.h" 20 #include "content/browser/loader/detachable_resource_handler.h"
20 #include "content/browser/loader/resource_loader_delegate.h" 21 #include "content/browser/loader/resource_loader_delegate.h"
21 #include "content/browser/loader/resource_request_info_impl.h" 22 #include "content/browser/loader/resource_request_info_impl.h"
22 #include "content/browser/service_worker/service_worker_request_handler.h" 23 #include "content/browser/service_worker/service_worker_request_handler.h"
23 #include "content/browser/ssl/ssl_client_auth_handler.h" 24 #include "content/browser/ssl/ssl_client_auth_handler.h"
24 #include "content/browser/ssl/ssl_manager.h" 25 #include "content/browser/ssl/ssl_manager.h"
25 #include "content/browser/ssl/ssl_policy.h" 26 #include "content/browser/ssl/ssl_policy.h"
26 #include "content/common/ssl_status_serialization.h" 27 #include "content/common/ssl_status_serialization.h"
27 #include "content/public/browser/cert_store.h" 28 #include "content/public/browser/cert_store.h"
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
121 net::IsCertStatusError(ssl_status.cert_status) && 122 net::IsCertStatusError(ssl_status.cert_status) &&
122 !net::IsCertStatusMinorError(ssl_status.cert_status); 123 !net::IsCertStatusMinorError(ssl_status.cert_status);
123 } else { 124 } else {
124 // We should not have any SSL state. 125 // We should not have any SSL state.
125 DCHECK(!request->ssl_info().cert_status); 126 DCHECK(!request->ssl_info().cert_status);
126 DCHECK_EQ(request->ssl_info().security_bits, -1); 127 DCHECK_EQ(request->ssl_info().security_bits, -1);
127 DCHECK_EQ(request->ssl_info().key_exchange_info, 0); 128 DCHECK_EQ(request->ssl_info().key_exchange_info, 0);
128 DCHECK(!request->ssl_info().connection_status); 129 DCHECK(!request->ssl_info().connection_status);
129 } 130 }
130 } 131 }
131
132 } // namespace 132 } // namespace
133 133
134 ResourceLoader::ResourceLoader(scoped_ptr<net::URLRequest> request, 134 ResourceLoader::ResourceLoader(scoped_ptr<net::URLRequest> request,
135 scoped_ptr<ResourceHandler> handler, 135 scoped_ptr<ResourceHandler> handler,
136 ResourceLoaderDelegate* delegate) 136 ResourceLoaderDelegate* delegate)
137 : deferred_stage_(DEFERRED_NONE), 137 : deferred_stage_(DEFERRED_NONE),
138 request_(std::move(request)), 138 request_(std::move(request)),
139 handler_(std::move(handler)), 139 handler_(std::move(handler)),
140 delegate_(delegate), 140 delegate_(delegate),
141 is_transferring_(false), 141 is_transferring_(false),
(...skipping 396 matching lines...) Expand 10 before | Expand all | Expand 10 after
538 void ResourceLoader::CompleteResponseStarted() { 538 void ResourceLoader::CompleteResponseStarted() {
539 ResourceRequestInfoImpl* info = GetRequestInfo(); 539 ResourceRequestInfoImpl* info = GetRequestInfo();
540 scoped_refptr<ResourceResponse> response(new ResourceResponse()); 540 scoped_refptr<ResourceResponse> response(new ResourceResponse());
541 PopulateResourceResponse(info, request_.get(), response.get()); 541 PopulateResourceResponse(info, request_.get(), response.get());
542 542
543 delegate_->DidReceiveResponse(this); 543 delegate_->DidReceiveResponse(this);
544 544
545 // TODO(darin): Remove ScopedTracker below once crbug.com/475761 is fixed. 545 // TODO(darin): Remove ScopedTracker below once crbug.com/475761 is fixed.
546 tracked_objects::ScopedTracker tracking_profile( 546 tracked_objects::ScopedTracker tracking_profile(
547 FROM_HERE_WITH_EXPLICIT_FUNCTION("475761 OnResponseStarted()")); 547 FROM_HERE_WITH_EXPLICIT_FUNCTION("475761 OnResponseStarted()"));
548
549 bool defer = false; 548 bool defer = false;
550 if (!handler_->OnResponseStarted(response.get(), &defer)) { 549 if (!handler_->OnResponseStarted(response.get(), &defer)) {
551 Cancel(); 550 Cancel();
552 } else if (defer) { 551 } else if (defer) {
553 read_deferral_start_time_ = base::TimeTicks::Now(); 552 read_deferral_start_time_ = base::TimeTicks::Now();
554 deferred_stage_ = DEFERRED_READ; // Read first chunk when resumed. 553 deferred_stage_ = DEFERRED_READ; // Read first chunk when resumed.
555 } 554 }
556 } 555 }
557 556
558 void ResourceLoader::StartReading(bool is_continuation) { 557 void ResourceLoader::StartReading(bool is_continuation) {
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
710 case net::URLRequestStatus::FAILED: 709 case net::URLRequestStatus::FAILED:
711 status = STATUS_UNDEFINED; 710 status = STATUS_UNDEFINED;
712 break; 711 break;
713 } 712 }
714 713
715 UMA_HISTOGRAM_ENUMERATION("Net.Prefetch.Pattern", status, STATUS_MAX); 714 UMA_HISTOGRAM_ENUMERATION("Net.Prefetch.Pattern", status, STATUS_MAX);
716 } 715 }
717 } 716 }
718 717
719 } // namespace content 718 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698