Add the client for accessing Subresource Filter only list.

chrome/browser/safe_browsing/test_safe_browsing_service.cc

 // Copyright (c) 2016 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/safe_browsing/test_safe_browsing_service.h"
 
 #include "base/strings/string_util.h"
 #include "chrome/browser/safe_browsing/download_protection_service.h"
 #include "chrome/browser/safe_browsing/ping_manager.h"
 #include "chrome/browser/safe_browsing/ui_manager.h"
 #include "components/safe_browsing_db/database_manager.h"
 #include "components/safe_browsing_db/test_database_manager.h"
+#include "components/safe_browsing_db/v4_feature_list.h"

[vakh (use Gerrit instead), 2017/02/10 03:00:11]

#include not needed?

[melandory, 2017/02/14 17:40:33]

Done.

 
 namespace safe_browsing {
 
 // TestSafeBrowsingService functions:
 TestSafeBrowsingService::TestSafeBrowsingService()
     : protocol_manager_delegate_disabled_(false),
       serialized_download_report_(base::EmptyString()) {}
 
 TestSafeBrowsingService::~TestSafeBrowsingService() {}
 
@@ skipping 25 matching lines @@
   // protocol_manager_delegate.
   protocol_manager_delegate_disabled_ = true;
 }
 
 void TestSafeBrowsingService::SetUIManager(
     TestSafeBrowsingUIManager* ui_manager) {
   ui_manager->SetSafeBrowsingService(this);
   ui_manager_ = ui_manager;
 }
 
+const scoped_refptr<SafeBrowsingDatabaseManager>&
+TestSafeBrowsingService::v4_local_database_manager() const {
+  return database_manager_;

[vakh (use Gerrit instead), 2017/02/10 03:00:12]

this looks like a hack we should avoid.

[melandory, 2017/02/14 17:40:33]

Yep, I'll piggy back on changes in your CL 2675063002.

WDYT, about extracting Fake V4 databases and factories you've introduced in CL
2675063002 in separate file, so it's easy to reuse them?

Here I've drafted the CL: https://codereview.chromium.org/2690413002

And in the last patchset of this CL you can see how it will be used.

+}
+
 SafeBrowsingDatabaseManager* TestSafeBrowsingService::CreateDatabaseManager() {
   if (database_manager_)
     return database_manager_.get();
   return SafeBrowsingService::CreateDatabaseManager();
 }
 
 SafeBrowsingUIManager* TestSafeBrowsingService::CreateUIManager() {
   if (ui_manager_)
     return ui_manager_.get();
   return SafeBrowsingService::CreateUIManager();
@@ skipping 80 matching lines @@
     const std::string& serialized) {
   details_.push_back(serialized);
 }
 
 std::list<std::string>* TestSafeBrowsingUIManager::GetThreatDetails() {
   return &details_;
 }
 
 TestSafeBrowsingUIManager::~TestSafeBrowsingUIManager() {}
 }  // namespace safe_browsing