Add the client for accessing Subresource Filter only list.

chrome/browser/safe_browsing/test_safe_browsing_service.cc

Index: chrome/browser/safe_browsing/test_safe_browsing_service.cc
diff --git a/chrome/browser/safe_browsing/test_safe_browsing_service.cc b/chrome/browser/safe_browsing/test_safe_browsing_service.cc
index b836b18dec5c154330073c3c92b0fd9b87f05410..7dc36e0bffb08347ddeb1acd12a483382de787cc 100644
--- a/chrome/browser/safe_browsing/test_safe_browsing_service.cc
+++ b/chrome/browser/safe_browsing/test_safe_browsing_service.cc
@@ -10,6 +10,7 @@
 #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 {
 
@@ -55,6 +56,11 @@ void TestSafeBrowsingService::SetUIManager(
   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();