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

Side by Side Diff: chrome/browser/extensions/api/preference/chrome_direct_setting.h

Issue 2298493003: [Extensions] Convert some ChromeSyncExtensionFunctions (Closed)
Patch Set: fix Created 4 years, 3 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_EXTENSIONS_API_PREFERENCE_CHROME_DIRECT_SETTING_H__ 5 #ifndef CHROME_BROWSER_EXTENSIONS_API_PREFERENCE_CHROME_DIRECT_SETTING_H__
6 #define CHROME_BROWSER_EXTENSIONS_API_PREFERENCE_CHROME_DIRECT_SETTING_H__ 6 #define CHROME_BROWSER_EXTENSIONS_API_PREFERENCE_CHROME_DIRECT_SETTING_H__
7 7
8 #include "base/lazy_instance.h" 8 #include "base/lazy_instance.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "chrome/browser/extensions/chrome_extension_function.h" 10 #include "extensions/browser/extension_function.h"
11 11
12 class PrefService; 12 class PrefService;
13 13
14 namespace extensions { 14 namespace extensions {
15 namespace chromedirectsetting { 15 namespace chromedirectsetting {
16 16
17 // Base class to host instance method helpers. 17 // Base class to host instance method helpers.
18 class DirectSettingFunctionBase : public ChromeSyncExtensionFunction { 18 class DirectSettingFunctionBase : public UIThreadExtensionFunction {
19 protected: 19 protected:
20 DirectSettingFunctionBase(); 20 DirectSettingFunctionBase();
21 ~DirectSettingFunctionBase() override; 21 ~DirectSettingFunctionBase() override;
22 22
23 // Returns the user pref service. 23 // Returns the user pref service.
24 PrefService* GetPrefService(); 24 PrefService* GetPrefService();
25 25
26 // Returns true if the caller is a component extension. 26 // Returns true if the caller is a component extension.
27 bool IsCalledFromComponentExtension(); 27 bool IsCalledFromComponentExtension();
28 28
29 private: 29 private:
30 DISALLOW_COPY_AND_ASSIGN(DirectSettingFunctionBase); 30 DISALLOW_COPY_AND_ASSIGN(DirectSettingFunctionBase);
31 }; 31 };
32 32
33 class GetDirectSettingFunction : public DirectSettingFunctionBase { 33 class GetDirectSettingFunction : public DirectSettingFunctionBase {
34 public: 34 public:
35 DECLARE_EXTENSION_FUNCTION("types.private.ChromeDirectSetting.get", 35 DECLARE_EXTENSION_FUNCTION("types.private.ChromeDirectSetting.get",
36 TYPES_PRIVATE_CHROMEDIRECTSETTING_GET) 36 TYPES_PRIVATE_CHROMEDIRECTSETTING_GET)
37 37
38 GetDirectSettingFunction(); 38 GetDirectSettingFunction();
39 39
40 protected: 40 protected:
41 // ExtensionFunction: 41 // ExtensionFunction:
42 bool RunSync() override; 42 ResponseAction Run() override;
43 43
44 private: 44 private:
45 ~GetDirectSettingFunction() override; 45 ~GetDirectSettingFunction() override;
46 DISALLOW_COPY_AND_ASSIGN(GetDirectSettingFunction); 46 DISALLOW_COPY_AND_ASSIGN(GetDirectSettingFunction);
47 }; 47 };
48 48
49 class SetDirectSettingFunction : public DirectSettingFunctionBase { 49 class SetDirectSettingFunction : public DirectSettingFunctionBase {
50 public: 50 public:
51 DECLARE_EXTENSION_FUNCTION("types.private.ChromeDirectSetting.set", 51 DECLARE_EXTENSION_FUNCTION("types.private.ChromeDirectSetting.set",
52 TYPES_PRIVATE_CHROMEDIRECTSETTING_SET) 52 TYPES_PRIVATE_CHROMEDIRECTSETTING_SET)
53 53
54 SetDirectSettingFunction(); 54 SetDirectSettingFunction();
55 55
56 protected: 56 protected:
57 // ExtensionFunction: 57 // ExtensionFunction:
58 bool RunSync() override; 58 ResponseAction Run() override;
59 59
60 private: 60 private:
61 ~SetDirectSettingFunction() override; 61 ~SetDirectSettingFunction() override;
62 DISALLOW_COPY_AND_ASSIGN(SetDirectSettingFunction); 62 DISALLOW_COPY_AND_ASSIGN(SetDirectSettingFunction);
63 }; 63 };
64 64
65 class ClearDirectSettingFunction : public DirectSettingFunctionBase { 65 class ClearDirectSettingFunction : public DirectSettingFunctionBase {
66 public: 66 public:
67 DECLARE_EXTENSION_FUNCTION("types.private.ChromeDirectSetting.clear", 67 DECLARE_EXTENSION_FUNCTION("types.private.ChromeDirectSetting.clear",
68 TYPES_PRIVATE_CHROMEDIRECTSETTING_CLEAR) 68 TYPES_PRIVATE_CHROMEDIRECTSETTING_CLEAR)
69 69
70 ClearDirectSettingFunction(); 70 ClearDirectSettingFunction();
71 71
72 protected: 72 protected:
73 // ExtensionFunction: 73 // ExtensionFunction:
74 bool RunSync() override; 74 ResponseAction Run() override;
75 75
76 private: 76 private:
77 ~ClearDirectSettingFunction() override; 77 ~ClearDirectSettingFunction() override;
78 DISALLOW_COPY_AND_ASSIGN(ClearDirectSettingFunction); 78 DISALLOW_COPY_AND_ASSIGN(ClearDirectSettingFunction);
79 }; 79 };
80 80
81 } // namespace chromedirectsetting 81 } // namespace chromedirectsetting
82 } // namespace extensions 82 } // namespace extensions
83 83
84 #endif // CHROME_BROWSER_EXTENSIONS_API_PREFERENCE_CHROME_DIRECT_SETTING_H__ 84 #endif // CHROME_BROWSER_EXTENSIONS_API_PREFERENCE_CHROME_DIRECT_SETTING_H__
85 85
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698