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

Side by Side Diff: chrome/browser/net/passive_log_collector.cc

Issue 8896019: Refactor: Extract "InitProxyResolver" to "ProxyScriptDecider". (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: do another sync since commitbot failed... Created 9 years 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
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/browser/net/passive_log_collector.h" 5 #include "chrome/browser/net/passive_log_collector.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/string_util.h" 10 #include "base/string_util.h"
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 exponential_backoff_throttling_tracker_(this)), 59 exponential_backoff_throttling_tracker_(this)),
60 num_events_seen_(0) { 60 num_events_seen_(0) {
61 61
62 // Define the mapping between source types and the tracker objects. 62 // Define the mapping between source types and the tracker objects.
63 memset(&trackers_[0], 0, sizeof(trackers_)); 63 memset(&trackers_[0], 0, sizeof(trackers_));
64 trackers_[net::NetLog::SOURCE_NONE] = &global_source_tracker_; 64 trackers_[net::NetLog::SOURCE_NONE] = &global_source_tracker_;
65 trackers_[net::NetLog::SOURCE_URL_REQUEST] = &url_request_tracker_; 65 trackers_[net::NetLog::SOURCE_URL_REQUEST] = &url_request_tracker_;
66 trackers_[net::NetLog::SOURCE_SOCKET_STREAM] = &socket_stream_tracker_; 66 trackers_[net::NetLog::SOURCE_SOCKET_STREAM] = &socket_stream_tracker_;
67 trackers_[net::NetLog::SOURCE_CONNECT_JOB] = &connect_job_tracker_; 67 trackers_[net::NetLog::SOURCE_CONNECT_JOB] = &connect_job_tracker_;
68 trackers_[net::NetLog::SOURCE_SOCKET] = &socket_tracker_; 68 trackers_[net::NetLog::SOURCE_SOCKET] = &socket_tracker_;
69 trackers_[net::NetLog::SOURCE_INIT_PROXY_RESOLVER] = 69 trackers_[net::NetLog::SOURCE_PROXY_SCRIPT_DECIDER] =
70 &init_proxy_resolver_tracker_; 70 &proxy_script_decider_tracker_;
71 trackers_[net::NetLog::SOURCE_SPDY_SESSION] = &spdy_session_tracker_; 71 trackers_[net::NetLog::SOURCE_SPDY_SESSION] = &spdy_session_tracker_;
72 trackers_[net::NetLog::SOURCE_HOST_RESOLVER_IMPL_REQUEST] = 72 trackers_[net::NetLog::SOURCE_HOST_RESOLVER_IMPL_REQUEST] =
73 &dns_request_tracker_; 73 &dns_request_tracker_;
74 trackers_[net::NetLog::SOURCE_HOST_RESOLVER_IMPL_JOB] = &dns_job_tracker_; 74 trackers_[net::NetLog::SOURCE_HOST_RESOLVER_IMPL_JOB] = &dns_job_tracker_;
75 trackers_[net::NetLog::SOURCE_DISK_CACHE_ENTRY] = &disk_cache_entry_tracker_; 75 trackers_[net::NetLog::SOURCE_DISK_CACHE_ENTRY] = &disk_cache_entry_tracker_;
76 trackers_[net::NetLog::SOURCE_MEMORY_CACHE_ENTRY] = &mem_cache_entry_tracker_; 76 trackers_[net::NetLog::SOURCE_MEMORY_CACHE_ENTRY] = &mem_cache_entry_tracker_;
77 trackers_[net::NetLog::SOURCE_HTTP_STREAM_JOB] = &http_stream_job_tracker_; 77 trackers_[net::NetLog::SOURCE_HTTP_STREAM_JOB] = &http_stream_job_tracker_;
78 trackers_[net::NetLog::SOURCE_EXPONENTIAL_BACKOFF_THROTTLING] = 78 trackers_[net::NetLog::SOURCE_EXPONENTIAL_BACKOFF_THROTTLING] =
79 &exponential_backoff_throttling_tracker_; 79 &exponential_backoff_throttling_tracker_;
80 trackers_[net::NetLog::SOURCE_DNS_TRANSACTION] = &dns_transaction_tracker_; 80 trackers_[net::NetLog::SOURCE_DNS_TRANSACTION] = &dns_transaction_tracker_;
(...skipping 400 matching lines...) Expand 10 before | Expand all | Expand 10 after
481 // adds to clutter. 481 // adds to clutter.
482 return ACTION_DELETE; 482 return ACTION_DELETE;
483 } 483 }
484 return ACTION_MOVE_TO_GRAVEYARD; 484 return ACTION_MOVE_TO_GRAVEYARD;
485 } 485 }
486 486
487 return ACTION_NONE; 487 return ACTION_NONE;
488 } 488 }
489 489
490 //---------------------------------------------------------------------------- 490 //----------------------------------------------------------------------------
491 // InitProxyResolverTracker 491 // ProxyScriptDeciderTracker
492 //---------------------------------------------------------------------------- 492 //----------------------------------------------------------------------------
493 493
494 const size_t PassiveLogCollector::InitProxyResolverTracker::kMaxNumSources = 20; 494 const size_t PassiveLogCollector::ProxyScriptDeciderTracker::kMaxNumSources
495 const size_t PassiveLogCollector::InitProxyResolverTracker::kMaxGraveyardSize = 495 = 20;
496 3; 496 const size_t PassiveLogCollector::ProxyScriptDeciderTracker::kMaxGraveyardSize
497 = 3;
497 498
498 PassiveLogCollector::InitProxyResolverTracker::InitProxyResolverTracker() 499 PassiveLogCollector::ProxyScriptDeciderTracker::ProxyScriptDeciderTracker()
499 : SourceTracker(kMaxNumSources, kMaxGraveyardSize, NULL) { 500 : SourceTracker(kMaxNumSources, kMaxGraveyardSize, NULL) {
500 } 501 }
501 502
502 PassiveLogCollector::SourceTracker::Action 503 PassiveLogCollector::SourceTracker::Action
503 PassiveLogCollector::InitProxyResolverTracker::DoAddEntry( 504 PassiveLogCollector::ProxyScriptDeciderTracker::DoAddEntry(
504 const ChromeNetLog::Entry& entry, SourceInfo* out_info) { 505 const ChromeNetLog::Entry& entry, SourceInfo* out_info) {
505 AddEntryToSourceInfo(entry, out_info); 506 AddEntryToSourceInfo(entry, out_info);
506 if (entry.type == net::NetLog::TYPE_INIT_PROXY_RESOLVER && 507 if (entry.type == net::NetLog::TYPE_PROXY_SCRIPT_DECIDER &&
507 entry.phase == net::NetLog::PHASE_END) { 508 entry.phase == net::NetLog::PHASE_END) {
508 return ACTION_MOVE_TO_GRAVEYARD; 509 return ACTION_MOVE_TO_GRAVEYARD;
509 } 510 }
510 return ACTION_NONE; 511 return ACTION_NONE;
511 } 512 }
512 513
513 //---------------------------------------------------------------------------- 514 //----------------------------------------------------------------------------
514 // SpdySessionTracker 515 // SpdySessionTracker
515 //---------------------------------------------------------------------------- 516 //----------------------------------------------------------------------------
516 517
(...skipping 260 matching lines...) Expand 10 before | Expand all | Expand 10 after
777 PassiveLogCollector::CertVerifierJobTracker::DoAddEntry( 778 PassiveLogCollector::CertVerifierJobTracker::DoAddEntry(
778 const ChromeNetLog::Entry& entry, 779 const ChromeNetLog::Entry& entry,
779 SourceInfo* out_info) { 780 SourceInfo* out_info) {
780 AddEntryToSourceInfo(entry, out_info); 781 AddEntryToSourceInfo(entry, out_info);
781 if (entry.type == net::NetLog::TYPE_CERT_VERIFIER_JOB && 782 if (entry.type == net::NetLog::TYPE_CERT_VERIFIER_JOB &&
782 entry.phase == net::NetLog::PHASE_END) { 783 entry.phase == net::NetLog::PHASE_END) {
783 return ACTION_MOVE_TO_GRAVEYARD; 784 return ACTION_MOVE_TO_GRAVEYARD;
784 } 785 }
785 return ACTION_NONE; 786 return ACTION_NONE;
786 } 787 }
OLDNEW
« no previous file with comments | « chrome/browser/net/passive_log_collector.h ('k') | chrome/browser/resources/net_internals/events_view.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698