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

Unified Diff: chrome/test/base/testing_browser_process.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 side-by-side diff with in-line comments
Download patch
Index: chrome/test/base/testing_browser_process.h
diff --git a/chrome/test/base/testing_browser_process.h b/chrome/test/base/testing_browser_process.h
index 233fdb063ff1c011a6857a70be582c0d0e31e096..8c585786e153f797de862f574c5ed6029ddbd546 100644
--- a/chrome/test/base/testing_browser_process.h
+++ b/chrome/test/base/testing_browser_process.h
@@ -58,74 +58,71 @@ class TestingBrowserProcess : public BrowserProcess {
// Convenience method to get g_browser_process as a TestingBrowserProcess*.
static TestingBrowserProcess* GetGlobal();
- virtual void ResourceDispatcherHostCreated() override;
- virtual void EndSession() override;
- virtual MetricsServicesManager* GetMetricsServicesManager() override;
- virtual metrics::MetricsService* metrics_service() override;
- virtual rappor::RapporService* rappor_service() override;
- virtual IOThread* io_thread() override;
- virtual WatchDogThread* watchdog_thread() override;
- virtual ProfileManager* profile_manager() override;
- virtual PrefService* local_state() override;
- virtual chrome_variations::VariationsService* variations_service() override;
- virtual policy::BrowserPolicyConnector* browser_policy_connector() override;
- virtual policy::PolicyService* policy_service() override;
- virtual IconManager* icon_manager() override;
- virtual GLStringManager* gl_string_manager() override;
- virtual GpuModeManager* gpu_mode_manager() override;
- virtual BackgroundModeManager* background_mode_manager() override;
- virtual void set_background_mode_manager_for_test(
+ void ResourceDispatcherHostCreated() override;
+ void EndSession() override;
+ MetricsServicesManager* GetMetricsServicesManager() override;
+ metrics::MetricsService* metrics_service() override;
+ rappor::RapporService* rappor_service() override;
+ IOThread* io_thread() override;
+ WatchDogThread* watchdog_thread() override;
+ ProfileManager* profile_manager() override;
+ PrefService* local_state() override;
+ chrome_variations::VariationsService* variations_service() override;
+ policy::BrowserPolicyConnector* browser_policy_connector() override;
+ policy::PolicyService* policy_service() override;
+ IconManager* icon_manager() override;
+ GLStringManager* gl_string_manager() override;
+ GpuModeManager* gpu_mode_manager() override;
+ BackgroundModeManager* background_mode_manager() override;
+ void set_background_mode_manager_for_test(
scoped_ptr<BackgroundModeManager> manager) override;
- virtual StatusTray* status_tray() override;
- virtual SafeBrowsingService* safe_browsing_service() override;
- virtual safe_browsing::ClientSideDetectionService*
- safe_browsing_detection_service() override;
- virtual net::URLRequestContextGetter* system_request_context() override;
- virtual BrowserProcessPlatformPart* platform_part() override;
-
- virtual extensions::EventRouterForwarder*
- extension_event_router_forwarder() override;
- virtual NotificationUIManager* notification_ui_manager() override;
- virtual message_center::MessageCenter* message_center() override;
- virtual IntranetRedirectDetector* intranet_redirect_detector() override;
- virtual void CreateDevToolsHttpProtocolHandler(
+ StatusTray* status_tray() override;
+ SafeBrowsingService* safe_browsing_service() override;
+ safe_browsing::ClientSideDetectionService* safe_browsing_detection_service()
+ override;
+ net::URLRequestContextGetter* system_request_context() override;
+ BrowserProcessPlatformPart* platform_part() override;
+
+ extensions::EventRouterForwarder* extension_event_router_forwarder() override;
+ NotificationUIManager* notification_ui_manager() override;
+ message_center::MessageCenter* message_center() override;
+ IntranetRedirectDetector* intranet_redirect_detector() override;
+ void CreateDevToolsHttpProtocolHandler(
chrome::HostDesktopType host_desktop_type,
const std::string& ip,
int port) override;
- virtual unsigned int AddRefModule() override;
- virtual unsigned int ReleaseModule() override;
- virtual bool IsShuttingDown() override;
- virtual printing::PrintJobManager* print_job_manager() override;
- virtual printing::PrintPreviewDialogController*
- print_preview_dialog_controller() override;
- virtual printing::BackgroundPrintingManager*
- background_printing_manager() override;
- virtual const std::string& GetApplicationLocale() override;
- virtual void SetApplicationLocale(const std::string& app_locale) override;
- virtual DownloadStatusUpdater* download_status_updater() override;
- virtual DownloadRequestLimiter* download_request_limiter() override;
+ unsigned int AddRefModule() override;
+ unsigned int ReleaseModule() override;
+ bool IsShuttingDown() override;
+ printing::PrintJobManager* print_job_manager() override;
+ printing::PrintPreviewDialogController* print_preview_dialog_controller()
+ override;
+ printing::BackgroundPrintingManager* background_printing_manager() override;
+ const std::string& GetApplicationLocale() override;
+ void SetApplicationLocale(const std::string& app_locale) override;
+ DownloadStatusUpdater* download_status_updater() override;
+ DownloadRequestLimiter* download_request_limiter() override;
#if (defined(OS_WIN) || defined(OS_LINUX)) && !defined(OS_CHROMEOS)
virtual void StartAutoupdateTimer() override {}
#endif
- virtual ChromeNetLog* net_log() override;
- virtual prerender::PrerenderTracker* prerender_tracker() override;
- virtual component_updater::ComponentUpdateService*
- component_updater() override;
- virtual CRLSetFetcher* crl_set_fetcher() override;
- virtual component_updater::PnaclComponentInstaller*
- pnacl_component_installer() override;
- virtual MediaFileSystemRegistry* media_file_system_registry() override;
- virtual bool created_local_state() const override;
+ ChromeNetLog* net_log() override;
+ prerender::PrerenderTracker* prerender_tracker() override;
+ component_updater::ComponentUpdateService* component_updater() override;
+ CRLSetFetcher* crl_set_fetcher() override;
+ component_updater::PnaclComponentInstaller* pnacl_component_installer()
+ override;
+ MediaFileSystemRegistry* media_file_system_registry() override;
+ bool created_local_state() const override;
#if defined(ENABLE_WEBRTC)
- virtual WebRtcLogUploader* webrtc_log_uploader() override;
+ WebRtcLogUploader* webrtc_log_uploader() override;
#endif
- virtual network_time::NetworkTimeTracker* network_time_tracker() override;
+ network_time::NetworkTimeTracker* network_time_tracker() override;
- virtual gcm::GCMDriver* gcm_driver() override;
+ gcm::GCMDriver* gcm_driver() override;
// Set the local state for tests. Consumer is responsible for cleaning it up
// afterwards (using ScopedTestingLocalState, for example).
@@ -139,7 +136,7 @@ class TestingBrowserProcess : public BrowserProcess {
private:
// See CreateInstance() and DestoryInstance() above.
TestingBrowserProcess();
- virtual ~TestingBrowserProcess();
+ ~TestingBrowserProcess() override;
scoped_ptr<content::NotificationService> notification_service_;
unsigned int module_ref_count_;
« no previous file with comments | « chrome/test/base/test_chrome_web_ui_controller_factory.h ('k') | chrome/test/base/testing_browser_process_platform_part.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698