[subresource_filter] Implement the "Smart" UI on Android

chrome/browser/subresource_filter/subresource_filter_content_settings_manager_factory.cc

Index: chrome/browser/subresource_filter/subresource_filter_content_settings_manager_factory.cc
diff --git a/chrome/browser/subresource_filter/subresource_filter_content_settings_manager_factory.cc b/chrome/browser/subresource_filter/subresource_filter_content_settings_manager_factory.cc
index a710f41cacb472b911e9056b98e16b62d28e6af9..2e6c6230db52a1ae1e899fc36a34d8622ff37eab 100644
--- a/chrome/browser/subresource_filter/subresource_filter_content_settings_manager_factory.cc
+++ b/chrome/browser/subresource_filter/subresource_filter_content_settings_manager_factory.cc
@@ -9,11 +9,14 @@
 #include "chrome/browser/subresource_filter/subresource_filter_content_settings_manager.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "components/keyed_service/core/keyed_service.h"
+#include "content/public/browser/browser_context.h"
 
 // static
-void SubresourceFilterContentSettingsManagerFactory::EnsureForProfile(
+SubresourceFilterContentSettingsManager*
+SubresourceFilterContentSettingsManagerFactory::EnsureForProfile(

[msramek, 2017/04/13 16:08:27]

IIRC you named EnsureForProfile() because it returned void. Should we name it
GetForProfile() now?

[Charlie Harrison, 2017/04/13 17:54:01]

Done.

     Profile* profile) {
-  GetInstance()->GetServiceForBrowserContext(profile, true /* create */);
+  return static_cast<SubresourceFilterContentSettingsManager*>(
+      GetInstance()->GetServiceForBrowserContext(profile, true /* create */));
 }
 
 // static
@@ -34,3 +37,9 @@ SubresourceFilterContentSettingsManagerFactory::BuildServiceInstanceFor(
   return new SubresourceFilterContentSettingsManager(
       static_cast<Profile*>(profile));
 }
+
+content::BrowserContext*
+SubresourceFilterContentSettingsManagerFactory::GetBrowserContextToUse(
+    content::BrowserContext* context) const {
+  return context;
+}