Convert SupervisedUserResourceThrottle to a NavigationThrottle.

chrome/browser/supervised_user/supervised_user_service_factory.h

Index: chrome/browser/supervised_user/supervised_user_service_factory.h
diff --git a/chrome/browser/supervised_user/supervised_user_service_factory.h b/chrome/browser/supervised_user/supervised_user_service_factory.h
index 51067006332339c314d2b221e0e40eaf38ee1fb2..5ec7ff28910f6788c89fc293fd0db4f6650e2830 100644
--- a/chrome/browser/supervised_user/supervised_user_service_factory.h
+++ b/chrome/browser/supervised_user/supervised_user_service_factory.h
@@ -9,12 +9,17 @@
 #include "chrome/browser/supervised_user/supervised_users.h"
 #include "components/keyed_service/content/browser_context_keyed_service_factory.h"
 
+namespace content {
+class BrowserContext;
+}
+
 class Profile;
 class SupervisedUserService;
 
 class SupervisedUserServiceFactory : public BrowserContextKeyedServiceFactory {
  public:
-  static SupervisedUserService* GetForProfile(Profile* profile);
+  static SupervisedUserService* GetForProfile(
+      content::BrowserContext* browser_context);

[Marc Treib, 2017/03/28 10:22:58]

Is this change actually required? If so, the method should probably be renamed
to GetForBrowserContext, but updating all callers won't be fun... maybe just
have both?

[mmenke, 2017/03/28 17:55:05]

It's not really needed, I just was horrified by:

SupervisedUserServiceFactory::GetForProfile(navigation_handle->GetWebContents()->GetBrowserContext())->GetURLFilter()

And thought throwing in a Profile::FromBrowserContext in there made the line
even worse, particularly as SupervisedUserServiceFactory::GetForProfile actually
uses a method that just needs a BrowserContext, not a profile.  I've gone ahead
and added Profile::FromBrowserContext to the aforementioned horrifying line
(Prefer that to adding a method to SupervisedUserServiceFactory).

 
   static SupervisedUserServiceFactory* GetInstance();