Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(124)

Side by Side Diff: chrome/browser/safe_browsing/client_side_detection_host_unittest.cc

Issue 11419224: Add missing (and remove superfluous) 'explicit' from constructors. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + remove non-straightforward changes Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/file_path.h" 5 #include "base/file_path.h"
6 #include "base/memory/ref_counted.h" 6 #include "base/memory/ref_counted.h"
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "base/synchronization/waitable_event.h" 8 #include "base/synchronization/waitable_event.h"
9 #include "chrome/browser/safe_browsing/browser_feature_extractor.h" 9 #include "chrome/browser/safe_browsing/browser_feature_extractor.h"
10 #include "chrome/browser/safe_browsing/client_side_detection_host.h" 10 #include "chrome/browser/safe_browsing/client_side_detection_host.h"
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 class MockTestingProfile : public TestingProfile { 140 class MockTestingProfile : public TestingProfile {
141 public: 141 public:
142 MockTestingProfile() {} 142 MockTestingProfile() {}
143 virtual ~MockTestingProfile() {} 143 virtual ~MockTestingProfile() {}
144 144
145 MOCK_CONST_METHOD0(IsOffTheRecord, bool()); 145 MOCK_CONST_METHOD0(IsOffTheRecord, bool());
146 }; 146 };
147 147
148 class MockBrowserFeatureExtractor : public BrowserFeatureExtractor { 148 class MockBrowserFeatureExtractor : public BrowserFeatureExtractor {
149 public: 149 public:
150 explicit MockBrowserFeatureExtractor( 150 MockBrowserFeatureExtractor(
151 WebContents* tab, 151 WebContents* tab,
152 ClientSideDetectionService* service) 152 ClientSideDetectionService* service)
153 : BrowserFeatureExtractor(tab, service) {} 153 : BrowserFeatureExtractor(tab, service) {}
154 virtual ~MockBrowserFeatureExtractor() {} 154 virtual ~MockBrowserFeatureExtractor() {}
155 155
156 MOCK_METHOD3(ExtractFeatures, 156 MOCK_METHOD3(ExtractFeatures,
157 void(const BrowseInfo* info, 157 void(const BrowseInfo* info,
158 ClientPhishingRequest*, 158 ClientPhishingRequest*,
159 const BrowserFeatureExtractor::DoneCallback&)); 159 const BrowserFeatureExtractor::DoneCallback&));
160 }; 160 };
(...skipping 623 matching lines...) Expand 10 before | Expand all | Expand 10 after
784 EXPECT_TRUE(Mock::VerifyAndClear(ui_manager_.get())); 784 EXPECT_TRUE(Mock::VerifyAndClear(ui_manager_.get()));
785 EXPECT_EQ(url, resource.url); 785 EXPECT_EQ(url, resource.url);
786 EXPECT_EQ(url, resource.original_url); 786 EXPECT_EQ(url, resource.original_url);
787 resource.callback.Reset(); 787 resource.callback.Reset();
788 msg = process()->sink().GetFirstMessageMatching( 788 msg = process()->sink().GetFirstMessageMatching(
789 SafeBrowsingMsg_StartPhishingDetection::ID); 789 SafeBrowsingMsg_StartPhishingDetection::ID);
790 ASSERT_FALSE(msg); 790 ASSERT_FALSE(msg);
791 } 791 }
792 792
793 } // namespace safe_browsing 793 } // namespace safe_browsing
OLDNEW
« no previous file with comments | « chrome/browser/prerender/prerender_unittest.cc ('k') | chrome/browser/sessions/session_service_test_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698