LevelDB storage for data reduction proxy to store data usage stats.

chrome/browser/profiles/profile_impl_io_data.cc

Index: chrome/browser/profiles/profile_impl_io_data.cc
diff --git a/chrome/browser/profiles/profile_impl_io_data.cc b/chrome/browser/profiles/profile_impl_io_data.cc
index fc465a72334d80212a6694fb14bc45d768fa3e95..799c8c87db69575e075ed74ed653a91e6feceb20 100644
--- a/chrome/browser/profiles/profile_impl_io_data.cc
+++ b/chrome/browser/profiles/profile_impl_io_data.cc
@@ -198,11 +198,12 @@ void ProfileImplIOData::Handle::Init(
           enable_quic_for_data_reduction_proxy)
           .Pass());
 
-  DataReductionProxyChromeSettingsFactory::GetForBrowserContext(profile_)->
-      InitDataReductionProxySettings(
+  DataReductionProxyChromeSettingsFactory::GetForBrowserContext(profile_)

[jeremyim, 2015/06/30 01:59:18]

nit: leave -> on the previous line since git cl format isn't enforced in this
directory

[Not at Google. Contact bengr, 2015/07/01 17:13:49]

"->" goes back to new line everytime I run git cl format. Seems better to leave
as is rather than manually changing back each time?

+      ->InitDataReductionProxySettings(
           io_data_->data_reduction_proxy_io_data(), profile_->GetPrefs(),
-          profile_->GetRequestContext(),
-          BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI));
+          profile_->GetRequestContext(), profile_path,
+          BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI),
+          BrowserThread::GetMessageLoopProxyForThread(BrowserThread::DB));

[michaeln, 2015/06/29 23:19:30]

Please use the default blocking pool setup by the BrowserThread class for these
new  tasks that do fileio. We'd like to eventually remove the named DB and FILE
threads and to use the blocking pool for the existing callers of those threads.
Since you're adding new code, it'd be good to use the blocking pool straight
away, unless there's a reason not to.

pool = BrowserThread::GetBlockingPool();
runner = pool->GetSequencedTaskRunnerWithShutdownBehavior(
    pool->GetSequenceToken(), SKIP_ON_SHUTDOWN);

There's nothing heavy weight about doing so.

[Not at Google. Contact bengr, 2015/07/01 17:13:49]

Done. Thanks for the explanation. I had misunderstood your suggestion.

 }
 
 content::ResourceContext*