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

Side by Side Diff: chrome/browser/ui/webui/sync_file_system_internals/extension_statuses_handler.h

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. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_BROWSER_UI_WEBUI_SYNC_FILE_SYSTEM_INTERNALS_EXTENSION_STATUSES_HA NDLER_H_ 5 #ifndef CHROME_BROWSER_UI_WEBUI_SYNC_FILE_SYSTEM_INTERNALS_EXTENSION_STATUSES_HA NDLER_H_
6 #define CHROME_BROWSER_UI_WEBUI_SYNC_FILE_SYSTEM_INTERNALS_EXTENSION_STATUSES_HA NDLER_H_ 6 #define CHROME_BROWSER_UI_WEBUI_SYNC_FILE_SYSTEM_INTERNALS_EXTENSION_STATUSES_HA NDLER_H_
7 7
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/memory/weak_ptr.h" 9 #include "base/memory/weak_ptr.h"
10 #include "content/public/browser/web_ui_message_handler.h" 10 #include "content/public/browser/web_ui_message_handler.h"
11 11
12 class Profile; 12 class Profile;
13 13
14 namespace syncfs_internals { 14 namespace syncfs_internals {
15 15
16 // This class handles message from WebUI page of chrome://syncfs-internals/ 16 // This class handles message from WebUI page of chrome://syncfs-internals/
17 // for the Extension Statuses tab. It corresponds to browser/resources/ 17 // for the Extension Statuses tab. It corresponds to browser/resources/
18 // sync_file_system_internals/extension_statuses.html. All methods in this class 18 // sync_file_system_internals/extension_statuses.html. All methods in this class
19 // should be called on UI thread. 19 // should be called on UI thread.
20 class ExtensionStatusesHandler : public content::WebUIMessageHandler { 20 class ExtensionStatusesHandler : public content::WebUIMessageHandler {
21 public: 21 public:
22 explicit ExtensionStatusesHandler(Profile* profile); 22 explicit ExtensionStatusesHandler(Profile* profile);
23 virtual ~ExtensionStatusesHandler(); 23 ~ExtensionStatusesHandler() override;
24 24
25 // Shared by Extension Statuses Tab and also File Metadata Tab to generate the 25 // Shared by Extension Statuses Tab and also File Metadata Tab to generate the
26 // extension drop down. 26 // extension drop down.
27 static void GetExtensionStatusesAsDictionary( 27 static void GetExtensionStatusesAsDictionary(
28 Profile* profile, 28 Profile* profile,
29 const base::Callback<void(const base::ListValue&)>& callback); 29 const base::Callback<void(const base::ListValue&)>& callback);
30 30
31 // WebUIMessageHandler implementation. 31 // WebUIMessageHandler implementation.
32 virtual void RegisterMessages() override; 32 void RegisterMessages() override;
33 33
34 private: 34 private:
35 void GetExtensionStatuses(const base::ListValue* args); 35 void GetExtensionStatuses(const base::ListValue* args);
36 void DidGetExtensionStatuses(const base::ListValue& list); 36 void DidGetExtensionStatuses(const base::ListValue& list);
37 37
38 Profile* profile_; 38 Profile* profile_;
39 base::WeakPtrFactory<ExtensionStatusesHandler> weak_ptr_factory_; 39 base::WeakPtrFactory<ExtensionStatusesHandler> weak_ptr_factory_;
40 40
41 DISALLOW_COPY_AND_ASSIGN(ExtensionStatusesHandler); 41 DISALLOW_COPY_AND_ASSIGN(ExtensionStatusesHandler);
42 }; 42 };
43 43
44 } // namespace syncfs_internals 44 } // namespace syncfs_internals
45 45
46 #endif // CHROME_BROWSER_UI_WEBUI_SYNC_FILE_SYSTEM_INTERNALS_EXTENSION_STATUSES _HANDLER_H_ 46 #endif // CHROME_BROWSER_UI_WEBUI_SYNC_FILE_SYSTEM_INTERNALS_EXTENSION_STATUSES _HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698