[subresource_filter] s/ShouldSuppressActivation/OnPageActivationComputed

components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc

 // Copyright 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 "components/subresource_filter/content/browser/content_subresource_filter_driver_factory.h"
 
 #include <set>
 #include <string>
 
 #include "base/macros.h"
@@ skipping 171 matching lines @@
     {ActivationDecision::ACTIVATION_DISABLED, ActivationLevel::DISABLED},
 };
 
 class MockSubresourceFilterClient : public SubresourceFilterClient {
  public:
   MockSubresourceFilterClient(VerifiedRulesetDealer::Handle* ruleset_dealer)
       : ruleset_dealer_(ruleset_dealer) {}
 
   ~MockSubresourceFilterClient() override = default;
 
-  bool ShouldSuppressActivation(content::NavigationHandle* handle) override {
-    return handle->IsInMainFrame() &&
-           whitelisted_hosts_.find(handle->GetURL().host()) !=
-               whitelisted_hosts_.end();
+  bool OnPageActivationComputed(content::NavigationHandle* handle,
+                                bool activated) override {
+    DCHECK(handle->IsInMainFrame());
+    return activated && whitelisted_hosts_.find(handle->GetURL().host()) !=
+                            whitelisted_hosts_.end();

[shivanisha, 2017/05/10 18:43:18]

Did you mean to move back to count()?

[Charlie Harrison, 2017/05/10 18:55:21]

Oh, I just forgot to update the test only implementation of the Mock client.
I'll update this one to use count() too.

[Charlie Harrison, 2017/05/10 18:57:32]

Done. OK landing now :)

   }
 
   void WhitelistByContentSettings(const GURL& url) override {}
   void WhitelistInCurrentWebContents(const GURL& url) override {
     if (url.SchemeIsHTTPOrHTTPS())
       whitelisted_hosts_.insert(url.host());
   }
 
   VerifiedRulesetDealer::Handle* GetRulesetDealer() override {
     return ruleset_dealer_;
@@ skipping 685 matching lines @@
     ActivationScopeTest,
     ContentSubresourceFilterDriverFactoryActivationScopeTest,
     ::testing::ValuesIn(kActivationScopeTestData));
 
 INSTANTIATE_TEST_CASE_P(
     ActivationLevelTest,
     ContentSubresourceFilterDriverFactoryActivationLevelTest,
     ::testing::ValuesIn(kActivationLevelTestData));
 
 }  // namespace subresource_filter