Chromium Code Reviews

Unified Diff: chrome/browser/ui/app_list/app_list_service_unittest.cc

Issue 671653002: Standardize usage of virtual/override/final in chrome/browser/ui/ (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.
Jump to:
View side-by-side diff with in-line comments
Index: chrome/browser/ui/app_list/app_list_service_unittest.cc
diff --git a/chrome/browser/ui/app_list/app_list_service_unittest.cc b/chrome/browser/ui/app_list/app_list_service_unittest.cc
index 3b9b0ce2852a893439a339e72823f551c69589fe..9951fd734adbcdd25bb99e38cdef9cc9910a9a02 100644
--- a/chrome/browser/ui/app_list/app_list_service_unittest.cc
+++ b/chrome/browser/ui/app_list/app_list_service_unittest.cc
@@ -42,38 +42,29 @@ class TestingAppListServiceImpl : public AppListServiceImpl {
}
// AppListService overrides:
- virtual Profile* GetCurrentAppListProfile() override {
+ Profile* GetCurrentAppListProfile() override {
// We don't return showing_for_profile_ here because that is only defined if
// the app list is visible.
return NULL;
}
- virtual void CreateForProfile(Profile* requested_profile) override {
- }
+ void CreateForProfile(Profile* requested_profile) override {}
- virtual void ShowForProfile(Profile* requested_profile) override {
+ void ShowForProfile(Profile* requested_profile) override {
showing_for_profile_ = requested_profile;
RecordAppListLaunch();
}
- virtual void DismissAppList() override {
- showing_for_profile_ = NULL;
- }
+ void DismissAppList() override { showing_for_profile_ = NULL; }
- virtual bool IsAppListVisible() const override {
- return !!showing_for_profile_;
- }
+ bool IsAppListVisible() const override { return !!showing_for_profile_; }
- virtual gfx::NativeWindow GetAppListWindow() override {
- return NULL;
- }
+ gfx::NativeWindow GetAppListWindow() override { return NULL; }
- virtual AppListControllerDelegate* GetControllerDelegate() override {
- return NULL;
- }
+ AppListControllerDelegate* GetControllerDelegate() override { return NULL; }
// AppListServiceImpl overrides:
- virtual void DestroyAppList() override { ++destroy_app_list_call_count_; }
+ void DestroyAppList() override { ++destroy_app_list_call_count_; }
private:
Profile* showing_for_profile_;

Powered by Google App Engine