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

Side by Side Diff: chrome/renderer/page_load_histograms.cc

Issue 392823002: remove PLT.BeginToFinish{,Doc}_AfterPreconnectRequest (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: set kouhei@ as the histogram owner Created 6 years, 5 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "chrome/renderer/page_load_histograms.h" 5 #include "chrome/renderer/page_load_histograms.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 610 matching lines...) Expand 10 before | Expand all | Expand 10 after
621 begin_to_finish_doc); 621 begin_to_finish_doc);
622 PLT_HISTOGRAM("PLT.BeginToFinish_ContentPrefetcher", 622 PLT_HISTOGRAM("PLT.BeginToFinish_ContentPrefetcher",
623 begin_to_finish_all_loads); 623 begin_to_finish_all_loads);
624 } 624 }
625 if (document_state->was_referred_by_prefetcher()) { 625 if (document_state->was_referred_by_prefetcher()) {
626 PLT_HISTOGRAM("PLT.BeginToFinishDoc_ContentPrefetcherReferrer", 626 PLT_HISTOGRAM("PLT.BeginToFinishDoc_ContentPrefetcherReferrer",
627 begin_to_finish_doc); 627 begin_to_finish_doc);
628 PLT_HISTOGRAM("PLT.BeginToFinish_ContentPrefetcherReferrer", 628 PLT_HISTOGRAM("PLT.BeginToFinish_ContentPrefetcherReferrer",
629 begin_to_finish_all_loads); 629 begin_to_finish_all_loads);
630 } 630 }
631 if (document_state->was_after_preconnect_request()) {
632 PLT_HISTOGRAM("PLT.BeginToFinishDoc_AfterPreconnectRequest",
633 begin_to_finish_doc);
634 PLT_HISTOGRAM("PLT.BeginToFinish_AfterPreconnectRequest",
635 begin_to_finish_all_loads);
636 }
637 631
638 const bool use_webrequest_histogram = 632 const bool use_webrequest_histogram =
639 ChromeContentRendererClient::WasWebRequestUsedBySomeExtensions(); 633 ChromeContentRendererClient::WasWebRequestUsedBySomeExtensions();
640 if (use_webrequest_histogram) { 634 if (use_webrequest_histogram) {
641 switch (load_type) { 635 switch (load_type) {
642 case DocumentState::NORMAL_LOAD: 636 case DocumentState::NORMAL_LOAD:
643 PLT_HISTOGRAM( 637 PLT_HISTOGRAM(
644 "PLT.BeginToFinish_NormalLoad_ExtensionWebRequest", 638 "PLT.BeginToFinish_NormalLoad_ExtensionWebRequest",
645 begin_to_finish_all_loads); 639 begin_to_finish_all_loads);
646 break; 640 break;
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
754 748
755 DCHECK(document_state); 749 DCHECK(document_state);
756 DCHECK(ds); 750 DCHECK(ds);
757 GURL url(ds->request().url()); 751 GURL url(ds->request().url());
758 Time start = document_state->start_load_time(); 752 Time start = document_state->start_load_time();
759 Time finish = document_state->finish_load_time(); 753 Time finish = document_state->finish_load_time();
760 // TODO(mbelshe): should we log more stats? 754 // TODO(mbelshe): should we log more stats?
761 VLOG(1) << "PLT: " << (finish - start).InMilliseconds() << "ms " 755 VLOG(1) << "PLT: " << (finish - start).InMilliseconds() << "ms "
762 << url.spec(); 756 << url.spec();
763 } 757 }
OLDNEW
« no previous file with comments | « no previous file | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698