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

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

Issue 1410853003: [Safe Browsing] Only check main frame and iframe URLs on Mobile, for speed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use a comma-separated list of types Created 5 years, 2 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/test_database_manager.cc
diff --git a/chrome/browser/safe_browsing/test_database_manager.cc b/chrome/browser/safe_browsing/test_database_manager.cc
index 4f89047f63511b32b81e32fa135f4d1b0c19d70a..6ef026e1eb1f50f8ae469bfea82c65e1b9c33b76 100644
--- a/chrome/browser/safe_browsing/test_database_manager.cc
+++ b/chrome/browser/safe_browsing/test_database_manager.cc
@@ -15,6 +15,17 @@ bool TestSafeBrowsingDatabaseManager::IsSupported() const {
return false;
}
+bool TestSafeBrowsingDatabaseManager::ChecksAreAlwaysAsync() const {
+ NOTIMPLEMENTED();
+ return false;
+}
+
+bool TestSafeBrowsingDatabaseManager::CanCheckResourceType(
+ content::ResourceType resource_type) const {
+ NOTIMPLEMENTED();
mattm 2015/10/19 20:19:40 The tests that use TestSafeBrowsingDatabaseManager
Nathan Parker 2015/10/19 21:03:22 Yup, just noticed I missed that. Done.
+ return false;
+}
+
bool TestSafeBrowsingDatabaseManager::CanCheckUrl(const GURL& url) const {
NOTIMPLEMENTED();
return false;

Powered by Google App Engine
This is Rietveld 408576698