SupervisedUserNavigationThrottle: create only for supervised profiles

chrome/browser/supervised_user/supervised_user_navigation_throttle.cc

 // Copyright 2014 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/supervised_user/supervised_user_navigation_throttle.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
@@ skipping 102 matching lines @@
   else
     UMA_HISTOGRAM_SPARSE_SLOWLY("ManagedUsers.FilteringResult", value);
 }
 
 }  // namespace
 
 // static
 std::unique_ptr<SupervisedUserNavigationThrottle>
 SupervisedUserNavigationThrottle::MaybeCreateThrottleFor(
     content::NavigationHandle* navigation_handle) {
-  if (!navigation_handle->IsInMainFrame())
+  Profile* profile = Profile::FromBrowserContext(
+      navigation_handle->GetWebContents()->GetBrowserContext());
+  if (!profile->IsSupervised() || !navigation_handle->IsInMainFrame())

[Bernhard Bauer, 2017/04/05 09:31:12]

I would split these up into two separate checks.

[Marc Treib, 2017/04/05 11:43:33]

Done.

     return nullptr;
   // Can't use base::MakeUnique because the constructor is private.
   return base::WrapUnique(
       new SupervisedUserNavigationThrottle(navigation_handle));
 }
 
 SupervisedUserNavigationThrottle::SupervisedUserNavigationThrottle(
     content::NavigationHandle* navigation_handle)
     : NavigationThrottle(navigation_handle),
       url_filter_(
@@ skipping 102 matching lines @@
   DCHECK(deferred_);
   deferred_ = false;
   navigation_handle()->Resume();
 }
 
 void SupervisedUserNavigationThrottle::Cancel() {
   DCHECK(deferred_);
   deferred_ = false;
   navigation_handle()->CancelDeferredNavigation(CANCEL);
 }