commit c5ef21adc76a76b5aa3bbeeb843f829ab3d1bd4f
author engedy <engedy@chromium.org> Fri Jun 17 20:22:27 2016
committer Commit bot <commit-bot@chromium.org> Fri Jun 17 20:23:58 2016
tree 6e5b632bdbbd5ec34133af7a8761e87c6c1fa51d
parent d9b1b341d89d7bd0025b1268ecb772f4c3eac425

Minimal implementation for the subresource_filter::RulesetService.

This CL contains the minimal implementation for the Safe Browsing subresource
filter ruleset service, providing versioned persistent storage for subresource
filtering rules.

As of this CL, no rules are persisted or used in production code yet.

The following aspects will be implemented in subsequent CLs:
  - Deleting obsoleted versions of rules at an appropriate time.
  - Recovery path from unlikely, yet not impossible, error scenarios.
  - Support for `pulling` new versions from the component updater.

BUG=609747

Review-Url: https://codereview.chromium.org/2072033002
Cr-Commit-Position: refs/heads/master@{#400486}

chrome/browser/browser_process_impl.cc

diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 48e6dd0..f6b642d2 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -98,6 +98,8 @@
 #include "components/prefs/pref_service.h"
 #include "components/safe_json/safe_json_parser.h"
 #include "components/signin/core/common/profile_management_switches.h"
+#include "components/subresource_filter/core/browser/ruleset_service.h"
+#include "components/subresource_filter/core/browser/subresource_filter_constants.h"
 #include "components/translate/core/browser/translate_download_manager.h"
 #include "components/update_client/update_query_params.h"
 #include "components/web_resource/web_resource_pref_names.h"
@@ -195,6 +197,7 @@
       created_notification_ui_manager_(false),
       created_notification_bridge_(false),
       created_safe_browsing_service_(false),
+      created_subresource_filter_ruleset_service_(false),
       shutting_down_(false),
       tearing_down_(false),
       download_status_updater_(new DownloadStatusUpdater),
@@ -856,6 +859,14 @@
   return NULL;
 }
 
+subresource_filter::RulesetService*
+BrowserProcessImpl::subresource_filter_ruleset_service() {
+  DCHECK(CalledOnValidThread());
+  if (!created_subresource_filter_ruleset_service_)
+    CreateSubresourceFilterRulesetService();
+  return subresource_filter_ruleset_service_.get();
+}
+
 #if (defined(OS_WIN) || defined(OS_LINUX)) && !defined(OS_CHROMEOS)
 void BrowserProcessImpl::StartAutoupdateTimer() {
   autoupdate_timer_.Start(FROM_HERE,
@@ -1127,6 +1138,25 @@
   safe_browsing_service_->Initialize();
 }
 
+void BrowserProcessImpl::CreateSubresourceFilterRulesetService() {
+  DCHECK(!subresource_filter_ruleset_service_);
+  created_subresource_filter_ruleset_service_ = true;
+
+  base::SequencedWorkerPool* blocking_pool =
+      content::BrowserThread::GetBlockingPool();
+  scoped_refptr<base::SequencedTaskRunner> blocking_task_runner(
+      blocking_pool->GetSequencedTaskRunnerWithShutdownBehavior(
+          blocking_pool->GetSequenceToken(),
+          base::SequencedWorkerPool::SKIP_ON_SHUTDOWN));
+
+  base::FilePath user_data_dir;
+  PathService::Get(chrome::DIR_USER_DATA, &user_data_dir);
+  subresource_filter_ruleset_service_.reset(
+      new subresource_filter::RulesetService(
+          local_state(), blocking_task_runner,
+          user_data_dir.Append(subresource_filter::kRulesetBaseDirectoryName)));
+}
+
 void BrowserProcessImpl::CreateGCMDriver() {
   DCHECK(!gcm_driver_);