Minor refactor: rename "extra_parameters" --> "params".

(The "extra" was redundant. And "params" is a well understood abbreviation of "parameters").

Review URL: http://codereview.chromium.org/2110001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47183 0039d316-1c4b-4281-b951-d872f2087c98

5 files changed, 16 insertions, 16 deletions

diff --git a/chrome/browser/net/chrome_net_log.cc b/chrome/browser/net/chrome_net_log.cc
index 34b9c12..efe801fc 100644
--- a/chrome/browser/net/chrome_net_log.cc
+++ b/chrome/browser/net/chrome_net_log.cc
@@ -27,12 +27,12 @@ void ChromeNetLog::AddEntry(EventType type,
                             const base::TimeTicks& time,
                             const Source& source,
                             EventPhase phase,
-                            EventParameters* extra_parameters) {
+                            EventParameters* params) {
   DCHECK(ChromeThread::CurrentlyOn(ChromeThread::IO));
 
   // Notify all of the log observers.
   FOR_EACH_OBSERVER(Observer, observers_,
-                    OnAddEntry(type, time, source, phase, extra_parameters));
+                    OnAddEntry(type, time, source, phase, params));
 }
 
 uint32 ChromeNetLog::NextID() {
diff --git a/chrome/browser/net/chrome_net_log.h b/chrome/browser/net/chrome_net_log.h
index b2f0d55..092be8d 100644
--- a/chrome/browser/net/chrome_net_log.h
+++ b/chrome/browser/net/chrome_net_log.h
@@ -29,7 +29,7 @@ class ChromeNetLog : public net::NetLog {
                             const base::TimeTicks& time,
                             const Source& source,
                             EventPhase phase,
-                            EventParameters* extra_parameters) = 0;
+                            EventParameters* params) = 0;
   };
 
   ChromeNetLog();
@@ -40,7 +40,7 @@ class ChromeNetLog : public net::NetLog {
                         const base::TimeTicks& time,
                         const Source& source,
                         EventPhase phase,
-                        EventParameters* extra_parameters);
+                        EventParameters* params);
   virtual uint32 NextID();
   virtual bool HasListener() const;
 
diff --git a/chrome/browser/net/passive_log_collector.cc b/chrome/browser/net/passive_log_collector.cc
index 59d848c..38f24ff 100644
--- a/chrome/browser/net/passive_log_collector.cc
+++ b/chrome/browser/net/passive_log_collector.cc
@@ -60,7 +60,7 @@ bool SortByOrderComparator(const PassiveLogCollector::Entry& a,
 void SetSubordinateSource(PassiveLogCollector::RequestInfo* info,
                           const PassiveLogCollector::Entry& entry) {
   info->subordinate_source.id = static_cast<net::NetLogIntegerParameter*>(
-      entry.extra_parameters.get())->value();
+      entry.params.get())->value();
   switch (entry.type) {
     case net::NetLog::TYPE_SOCKET_POOL_CONNECT_JOB_ID:
       info->subordinate_source.type = net::NetLog::SOURCE_CONNECT_JOB;
@@ -94,9 +94,9 @@ void PassiveLogCollector::OnAddEntry(
     const base::TimeTicks& time,
     const net::NetLog::Source& source,
     net::NetLog::EventPhase phase,
-    net::NetLog::EventParameters* extra_parameters) {
+    net::NetLog::EventParameters* params) {
   // Package the parameters into a single struct for convenience.
-  Entry entry(num_events_seen_++, type, time, source, phase, extra_parameters);
+  Entry entry(num_events_seen_++, type, time, source, phase, params);
 
   switch (entry.source.type) {
     case net::NetLog::SOURCE_URL_REQUEST:
@@ -355,7 +355,7 @@ PassiveLogCollector::SocketTracker::DoAddEntry(const Entry& entry,
   switch (entry.type) {
     case net::NetLog::TYPE_SOCKET_BYTES_SENT:
       int_arg = static_cast<net::NetLogIntegerParameter*>(
-          entry.extra_parameters.get())->value();
+          entry.params.get())->value();
       out_info->total_bytes_transmitted += int_arg;
       out_info->bytes_transmitted += int_arg;
       out_info->last_tx_rx_time = entry.time;
@@ -363,7 +363,7 @@ PassiveLogCollector::SocketTracker::DoAddEntry(const Entry& entry,
       break;
     case net::NetLog::TYPE_SOCKET_BYTES_RECEIVED:
       int_arg = static_cast<net::NetLogIntegerParameter*>(
-          entry.extra_parameters.get())->value();
+          entry.params.get())->value();
       out_info->total_bytes_received += int_arg;
       out_info->bytes_received += int_arg;
       out_info->last_tx_rx_time = entry.time;
@@ -496,11 +496,11 @@ PassiveLogCollector::RequestTracker::DoAddEntry(const Entry& entry,
   // Note: we look at the first *two* entries, since the outer REQUEST_ALIVE
   // doesn't actually contain any data.
   if (out_info->url.empty() && out_info->entries.size() <= 2 &&
-      entry.phase == net::NetLog::PHASE_BEGIN && entry.extra_parameters &&
+      entry.phase == net::NetLog::PHASE_BEGIN && entry.params &&
       (entry.type == net::NetLog::TYPE_URL_REQUEST_START ||
        entry.type == net::NetLog::TYPE_SOCKET_STREAM_CONNECT)) {
     out_info->url = static_cast<net::NetLogStringParameter*>(
-        entry.extra_parameters.get())->value();
+        entry.params.get())->value();
   }
 
   // If the request has ended, move it to the graveyard.
diff --git a/chrome/browser/net/passive_log_collector.h b/chrome/browser/net/passive_log_collector.h
index 55b7279..bbcf7ca 100644
--- a/chrome/browser/net/passive_log_collector.h
+++ b/chrome/browser/net/passive_log_collector.h
@@ -25,9 +25,9 @@ class PassiveLogCollector : public ChromeNetLog::Observer {
           const base::TimeTicks& time,
           net::NetLog::Source source,
           net::NetLog::EventPhase phase,
-          net::NetLog::EventParameters* extra_parameters)
+          net::NetLog::EventParameters* params)
         : order(order), type(type), time(time), source(source), phase(phase),
-          extra_parameters(extra_parameters) {
+          params(params) {
     }
 
     uint32 order;
@@ -35,7 +35,7 @@ class PassiveLogCollector : public ChromeNetLog::Observer {
     base::TimeTicks time;
     net::NetLog::Source source;
     net::NetLog::EventPhase phase;
-    scoped_refptr<net::NetLog::EventParameters> extra_parameters;
+    scoped_refptr<net::NetLog::EventParameters> params;
   };
 
   typedef std::vector<Entry> EntryList;
@@ -208,7 +208,7 @@ class PassiveLogCollector : public ChromeNetLog::Observer {
                           const base::TimeTicks& time,
                           const net::NetLog::Source& source,
                           net::NetLog::EventPhase phase,
-                          net::NetLog::EventParameters* extra_parameters);
+                          net::NetLog::EventParameters* params);
 
   // Clears all of the passively logged data.
   void Clear();
diff --git a/chrome/browser/net/passive_log_collector_unittest.cc b/chrome/browser/net/passive_log_collector_unittest.cc
index 1fa5b8e..077597f 100644
--- a/chrome/browser/net/passive_log_collector_unittest.cc
+++ b/chrome/browser/net/passive_log_collector_unittest.cc
@@ -61,7 +61,7 @@ void AddEndURLRequestEntries(PassiveLogCollector* collector, uint32 id) {
 
 std::string GetStringParam(const PassiveLogCollector::Entry& entry) {
   return static_cast<net::NetLogStringParameter*>(
-      entry.extra_parameters.get())->value();
+      entry.params.get())->value();
 }
 
 static const int kMaxNumLoadLogEntries = 1;