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

Side by Side Diff: chrome/test/base/test_chrome_web_ui_controller_factory.h

Issue 653773004: Standardize usage of virtual/override/final in chrome/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 unified diff | Download patch
« no previous file with comments | « chrome/test/base/test_browser_window.cc ('k') | chrome/test/base/testing_browser_process.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 #ifndef CHROME_TEST_BASE_TEST_CHROME_WEB_UI_CONTROLLER_FACTORY_H_ 5 #ifndef CHROME_TEST_BASE_TEST_CHROME_WEB_UI_CONTROLLER_FACTORY_H_
6 #define CHROME_TEST_BASE_TEST_CHROME_WEB_UI_CONTROLLER_FACTORY_H_ 6 #define CHROME_TEST_BASE_TEST_CHROME_WEB_UI_CONTROLLER_FACTORY_H_
7 7
8 #include <functional> 8 #include <functional>
9 #include <map> 9 #include <map>
10 #include <string> 10 #include <string>
(...skipping 14 matching lines...) Expand all
25 virtual content::WebUIController* NewWebUI(content::WebUI* web_ui, 25 virtual content::WebUIController* NewWebUI(content::WebUI* web_ui,
26 const GURL& url) = 0; 26 const GURL& url) = 0;
27 27
28 protected: 28 protected:
29 virtual ~WebUIProvider(); 29 virtual ~WebUIProvider();
30 }; 30 };
31 31
32 typedef std::map<std::string, WebUIProvider*> FactoryOverridesMap; 32 typedef std::map<std::string, WebUIProvider*> FactoryOverridesMap;
33 33
34 TestChromeWebUIControllerFactory(); 34 TestChromeWebUIControllerFactory();
35 virtual ~TestChromeWebUIControllerFactory(); 35 ~TestChromeWebUIControllerFactory() override;
36 36
37 // Override the creation for urls having |host| with |provider|. 37 // Override the creation for urls having |host| with |provider|.
38 void AddFactoryOverride(const std::string& host, WebUIProvider* provider); 38 void AddFactoryOverride(const std::string& host, WebUIProvider* provider);
39 39
40 // Remove the override for urls having |host|. 40 // Remove the override for urls having |host|.
41 void RemoveFactoryOverride(const std::string& host); 41 void RemoveFactoryOverride(const std::string& host);
42 42
43 // ChromeWebUIFactory overrides. 43 // ChromeWebUIFactory overrides.
44 virtual content::WebUI::TypeID GetWebUIType( 44 content::WebUI::TypeID GetWebUIType(content::BrowserContext* browser_context,
45 content::BrowserContext* browser_context, 45 const GURL& url) const override;
46 content::WebUIController* CreateWebUIControllerForURL(
47 content::WebUI* web_ui,
46 const GURL& url) const override; 48 const GURL& url) const override;
47 virtual content::WebUIController* CreateWebUIControllerForURL(
48 content::WebUI* web_ui, const GURL& url) const override;
49 49
50 private: 50 private:
51 // Return the WebUIProvider for the |url|'s host if it exists, otherwise NULL. 51 // Return the WebUIProvider for the |url|'s host if it exists, otherwise NULL.
52 WebUIProvider* GetWebUIProvider(Profile* profile, const GURL& url) const; 52 WebUIProvider* GetWebUIProvider(Profile* profile, const GURL& url) const;
53 53
54 // Stores the mapping of host to WebUIProvider. 54 // Stores the mapping of host to WebUIProvider.
55 FactoryOverridesMap factory_overrides_; 55 FactoryOverridesMap factory_overrides_;
56 56
57 DISALLOW_COPY_AND_ASSIGN(TestChromeWebUIControllerFactory); 57 DISALLOW_COPY_AND_ASSIGN(TestChromeWebUIControllerFactory);
58 }; 58 };
59 59
60 #endif // CHROME_TEST_BASE_TEST_CHROME_WEB_UI_CONTROLLER_FACTORY_H_ 60 #endif // CHROME_TEST_BASE_TEST_CHROME_WEB_UI_CONTROLLER_FACTORY_H_
OLDNEW
« no previous file with comments | « chrome/test/base/test_browser_window.cc ('k') | chrome/test/base/testing_browser_process.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698