Index: components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc |
diff --git a/components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc b/components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc |
index cb08eefd8395921510033df511472b131fe93bee..d45a265136f96c93752aac8476e3ee24874a2619 100644 |
--- a/components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc |
+++ b/components/subresource_filter/content/browser/content_subresource_filter_driver_factory_unittest.cc |
@@ -236,9 +236,10 @@ class ContentSubresourceFilterDriverFactoryTest |
base::MessageLoop::current()->task_runner()); |
ruleset_dealer_->SetRulesetFile( |
testing::TestRuleset::Open(test_ruleset_pair_.indexed)); |
- client_ = new MockSubresourceFilterClient(ruleset_dealer_.get()); |
+ client_ = |
+ base::MakeUnique<MockSubresourceFilterClient>(ruleset_dealer_.get()); |
ContentSubresourceFilterDriverFactory::CreateForWebContents( |
- RenderViewHostTestHarness::web_contents(), base::WrapUnique(client())); |
+ RenderViewHostTestHarness::web_contents(), client()); |
ResetConfigurationToEnableFilteringOnSocialEngineeringSites(); |
// Add a subframe. |
@@ -276,7 +277,7 @@ class ContentSubresourceFilterDriverFactoryTest |
RenderViewHostTestHarness::web_contents()); |
} |
- MockSubresourceFilterClient* client() { return client_; } |
+ MockSubresourceFilterClient* client() { return client_.get(); } |
content::RenderFrameHost* GetSubframeRFH() { |
for (content::RenderFrameHost* rfh : |
@@ -493,9 +494,7 @@ class ContentSubresourceFilterDriverFactoryTest |
testing::TestRulesetCreator test_ruleset_creator_; |
testing::TestRulesetPair test_ruleset_pair_; |
- // Owned by the factory. |
- MockSubresourceFilterClient* client_; |
- |
+ std::unique_ptr<MockSubresourceFilterClient> client_; |
std::unique_ptr<VerifiedRulesetDealer::Handle> ruleset_dealer_; |
DISALLOW_COPY_AND_ASSIGN(ContentSubresourceFilterDriverFactoryTest); |