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

Unified Diff: chrome/browser/safe_browsing/client_side_detection_host_unittest.cc

Issue 10068036: RefCounted types should not have public destructors, chrome/browser/ part 5 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Win fix Created 8 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/client_side_detection_host_unittest.cc
diff --git a/chrome/browser/safe_browsing/client_side_detection_host_unittest.cc b/chrome/browser/safe_browsing/client_side_detection_host_unittest.cc
index 1d68a4dcaf7305d8bb4fd4a3e62c29d4122dd2d6..b39b63b5191668a46ab85d0ee8872e57e017d4e6 100644
--- a/chrome/browser/safe_browsing/client_side_detection_host_unittest.cc
+++ b/chrome/browser/safe_browsing/client_side_detection_host_unittest.cc
@@ -101,7 +101,6 @@ class MockClientSideDetectionService : public ClientSideDetectionService {
class MockSafeBrowsingService : public SafeBrowsingService {
public:
MockSafeBrowsingService() {}
- virtual ~MockSafeBrowsingService() {}
MOCK_METHOD1(DoDisplayBlockingPage, void(const UnsafeResource& resource));
MOCK_METHOD1(MatchCsdWhitelistUrl, bool(const GURL&));
@@ -117,6 +116,9 @@ class MockSafeBrowsingService : public SafeBrowsingService {
callback.Run(false);
}
+ protected:
+ virtual ~MockSafeBrowsingService() {}
+
private:
DISALLOW_COPY_AND_ASSIGN(MockSafeBrowsingService);
};

Powered by Google App Engine
This is Rietveld 408576698