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

Side by Side Diff: chrome/browser/sync/sync_error_notifier_ash.cc

Issue 686563003: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #include "chrome/browser/sync/sync_error_notifier_ash.h" 5 #include "chrome/browser/sync/sync_error_notifier_ash.h"
6 6
7 #include "ash/shell.h" 7 #include "ash/shell.h"
8 #include "ash/shell_delegate.h" 8 #include "ash/shell_delegate.h"
9 #include "ash/system/system_notifier.h" 9 #include "ash/system/system_notifier.h"
10 #include "base/strings/string16.h" 10 #include "base/strings/string16.h"
(...skipping 27 matching lines...) Expand all
38 38
39 const char kProfileSyncNotificationId[] = "chrome://settings/sync/"; 39 const char kProfileSyncNotificationId[] = "chrome://settings/sync/";
40 40
41 // A simple notification delegate for the sync setup button. 41 // A simple notification delegate for the sync setup button.
42 class SyncNotificationDelegate : public NotificationDelegate { 42 class SyncNotificationDelegate : public NotificationDelegate {
43 public: 43 public:
44 SyncNotificationDelegate(const std::string& id, 44 SyncNotificationDelegate(const std::string& id,
45 Profile* profile); 45 Profile* profile);
46 46
47 // NotificationDelegate: 47 // NotificationDelegate:
48 virtual void Display() override; 48 void Display() override;
49 virtual void Error() override; 49 void Error() override;
50 virtual void Close(bool by_user) override; 50 void Close(bool by_user) override;
51 virtual bool HasClickedListener() override; 51 bool HasClickedListener() override;
52 virtual void Click() override; 52 void Click() override;
53 virtual void ButtonClick(int button_index) override; 53 void ButtonClick(int button_index) override;
54 virtual std::string id() const override; 54 std::string id() const override;
55 55
56 protected: 56 protected:
57 virtual ~SyncNotificationDelegate(); 57 ~SyncNotificationDelegate() override;
58 58
59 private: 59 private:
60 void ShowSyncSetup(); 60 void ShowSyncSetup();
61 61
62 // Unique id of the notification. 62 // Unique id of the notification.
63 const std::string id_; 63 const std::string id_;
64 64
65 Profile* profile_; 65 Profile* profile_;
66 66
67 DISALLOW_COPY_AND_ASSIGN(SyncNotificationDelegate); 67 DISALLOW_COPY_AND_ASSIGN(SyncNotificationDelegate);
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
194 ui::ResourceBundle::GetSharedInstance().GetImageNamed( 194 ui::ResourceBundle::GetSharedInstance().GetImageNamed(
195 IDR_NOTIFICATION_ALERT), 195 IDR_NOTIFICATION_ALERT),
196 blink::WebTextDirectionDefault, 196 blink::WebTextDirectionDefault,
197 notifier_id, 197 notifier_id,
198 base::string16(), // display_source 198 base::string16(), // display_source
199 base::ASCIIToUTF16(notification_id_), 199 base::ASCIIToUTF16(notification_id_),
200 data, 200 data,
201 delegate); 201 delegate);
202 notification_ui_manager->Add(notification, profile_); 202 notification_ui_manager->Add(notification, profile_);
203 } 203 }
OLDNEW
« no previous file with comments | « chrome/browser/sync/sync_error_notifier_ash.h ('k') | chrome/browser/sync/sync_error_notifier_ash_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698