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

Unified Diff: chrome/test/ppapi/ppapi_filechooser_browsertest.cc

Issue 1420053005: Move code in components/safe_browsing_db and chrome/browser/s_b/ under the safe_browsing namespace. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@02_components_move
Patch Set: Remove '// namespace safe_browsing' for a small fwd decl block. Created 5 years, 1 month 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
« no previous file with comments | « chrome/test/base/testing_browser_process.cc ('k') | components/safe_browsing_db/prefix_set.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/ppapi/ppapi_filechooser_browsertest.cc
diff --git a/chrome/test/ppapi/ppapi_filechooser_browsertest.cc b/chrome/test/ppapi/ppapi_filechooser_browsertest.cc
index 99ebf1578720d216a5f7860aced4a27d237ffadf..b1a7ea2e1003d830013a7cb7f45c8994dd13f214 100644
--- a/chrome/test/ppapi/ppapi_filechooser_browsertest.cc
+++ b/chrome/test/ppapi/ppapi_filechooser_browsertest.cc
@@ -19,6 +19,8 @@
#include "chrome/browser/safe_browsing/test_database_manager.h"
#endif
+using safe_browsing::SafeBrowsingService;
+
namespace {
class TestSelectFileDialogFactory final : public ui::SelectFileDialogFactory {
@@ -122,7 +124,8 @@ class TestSelectFileDialogFactory final : public ui::SelectFileDialogFactory {
Mode mode_;
};
-class FakeDatabaseManager : public TestSafeBrowsingDatabaseManager {
+class FakeDatabaseManager
+ : public safe_browsing::TestSafeBrowsingDatabaseManager {
public:
bool IsSupported() const override { return true; }
bool MatchDownloadWhitelistUrl(const GURL& url) override {
@@ -137,20 +140,22 @@ class FakeDatabaseManager : public TestSafeBrowsingDatabaseManager {
class TestSafeBrowsingService : public SafeBrowsingService {
public:
- SafeBrowsingDatabaseManager* CreateDatabaseManager() override {
+ safe_browsing::SafeBrowsingDatabaseManager* CreateDatabaseManager() override {
return new FakeDatabaseManager();
}
protected:
~TestSafeBrowsingService() override {}
- SafeBrowsingProtocolManagerDelegate* GetProtocolManagerDelegate() override {
+ safe_browsing::SafeBrowsingProtocolManagerDelegate*
+ GetProtocolManagerDelegate() override {
// Our FakeDatabaseManager doesn't implement this delegate.
return NULL;
}
};
-class TestSafeBrowsingServiceFactory : public SafeBrowsingServiceFactory {
+class TestSafeBrowsingServiceFactory
+ : public safe_browsing::SafeBrowsingServiceFactory {
public:
SafeBrowsingService* CreateSafeBrowsingService() override {
SafeBrowsingService* service = new TestSafeBrowsingService();
« no previous file with comments | « chrome/test/base/testing_browser_process.cc ('k') | components/safe_browsing_db/prefix_set.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698