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

chrome/browser/profiles/profile_impl_io_data.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/profiles/profile_impl_io_data.h"
 
 #include <set>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 180 matching lines @@
       IOThread::ShouldEnableQuicForDataReductionProxy();
 
   io_data_->set_data_reduction_proxy_io_data(
       CreateDataReductionProxyChromeIOData(
           g_browser_process->io_thread()->net_log(), profile_->GetPrefs(),
           BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO),
           BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI),
           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*
     ProfileImplIOData::Handle::GetResourceContext() const {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   LazyInitialize();
   return GetResourceContextNoInit();
 }
 
 content::ResourceContext*
@@ skipping 588 matching lines @@
     const base::Closure& completion) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   DCHECK(initialized());
 
   DCHECK(transport_security_state());
   // Completes synchronously.
   transport_security_state()->DeleteAllDynamicDataSince(time);
   DCHECK(http_server_properties_manager_);
   http_server_properties_manager_->Clear(completion);
 }