SDCH: track NetworkBytesSpent in SdchOwner

To do this, add an extra parameter to SdchDictionaryFetcher::OnDictionaryFetchedCallback and SdchOwner::OnDictionaryFetched,
which indicates whether the fetched dictionary was served from cache or not. If this parameter indicates a network load, have SdchOwner
histogram the size of the received dictionary.

Also, add unit tests covering this behavior.

BUG=465873

Review URL: https://codereview.chromium.org/1050193002

Cr-Commit-Position: refs/heads/master@{#323745}

1 files changed, 5 insertions, 5 deletions

diff --git a/net/sdch/sdch_owner.h b/net/sdch/sdch_owner.h
index 1655275..053eecd 100644
--- a/net/sdch/sdch_owner.h
+++ b/net/sdch/sdch_owner.h
@@ -68,16 +68,16 @@ class NET_EXPORT SdchOwner : public net::SdchObserver,
   void OnPrefValueChanged(const std::string& key) override;
   void OnInitializationCompleted(bool succeeded) override;
 
-  // Implementation detail--this is the function callback by the callback
-  // passed to the fetcher through which the fetcher informs the SdchOwner
-  // that it's gotten the dictionary.  The first two arguments are
-  // bound locally.
+  // Implementation detail--this is the function callback by the callback passed
+  // to the fetcher through which the fetcher informs the SdchOwner that it's
+  // gotten the dictionary.  The first two arguments are bound locally.
   // Public for testing.
   void OnDictionaryFetched(base::Time last_used,
                            int use_count,
                            const std::string& dictionary_text,
                            const GURL& dictionary_url,
-                           const net::BoundNetLog& net_log);
+                           const net::BoundNetLog& net_log,
+                           bool was_from_cache);
 
   void SetClockForTesting(scoped_ptr<base::Clock> clock);