Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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_DECLARATIVE_DECLARATIVE_API_H_ | 5 #ifndef CHROME_BROWSER_EXTENSIONS_API_DECLARATIVE_DECLARATIVE_API_H_ |
| 6 #define CHROME_BROWSER_EXTENSIONS_API_DECLARATIVE_DECLARATIVE_API_H_ | 6 #define CHROME_BROWSER_EXTENSIONS_API_DECLARATIVE_DECLARATIVE_API_H_ |
| 7 | 7 |
| 8 #include "base/compiler_specific.h" | 8 #include "base/compiler_specific.h" |
| 9 #include "base/memory/ref_counted.h" | 9 #include "base/memory/ref_counted.h" |
| 10 #include "chrome/browser/extensions/api/declarative/rules_registry.h" | 10 #include "chrome/browser/extensions/api/declarative/rules_registry.h" |
| (...skipping 11 matching lines...) Expand all Loading... | |
| 22 // ExtensionFunction: | 22 // ExtensionFunction: |
| 23 virtual bool HasPermission() OVERRIDE; | 23 virtual bool HasPermission() OVERRIDE; |
| 24 virtual bool RunImpl() OVERRIDE; | 24 virtual bool RunImpl() OVERRIDE; |
| 25 | 25 |
| 26 // Concrete implementation of the RulesFunction that is being called | 26 // Concrete implementation of the RulesFunction that is being called |
| 27 // on the thread on which the respective RulesRegistry lives. | 27 // on the thread on which the respective RulesRegistry lives. |
| 28 // Returns false in case of errors. | 28 // Returns false in case of errors. |
| 29 virtual bool RunImplOnCorrectThread() = 0; | 29 virtual bool RunImplOnCorrectThread() = 0; |
| 30 | 30 |
| 31 scoped_refptr<RulesRegistry> rules_registry_; | 31 scoped_refptr<RulesRegistry> rules_registry_; |
| 32 int webview_instance_id_; | |
|
Jeffrey Yasskin
2013/11/09 02:47:22
Please comment what value this has when there's no
Fady Samuel
2013/11/10 03:39:56
Looks like it's stale unused code! Thanks for catc
| |
| 32 }; | 33 }; |
| 33 | 34 |
| 34 class EventsEventAddRulesFunction : public RulesFunction { | 35 class EventsEventAddRulesFunction : public RulesFunction { |
| 35 public: | 36 public: |
| 36 DECLARE_EXTENSION_FUNCTION("events.addRules", EVENTS_ADDRULES) | 37 DECLARE_EXTENSION_FUNCTION("events.addRules", EVENTS_ADDRULES) |
| 37 | 38 |
| 38 protected: | 39 protected: |
| 39 virtual ~EventsEventAddRulesFunction() {} | 40 virtual ~EventsEventAddRulesFunction() {} |
| 40 | 41 |
| 41 // RulesFunction: | 42 // RulesFunction: |
| (...skipping 18 matching lines...) Expand all Loading... | |
| 60 protected: | 61 protected: |
| 61 virtual ~EventsEventGetRulesFunction() {} | 62 virtual ~EventsEventGetRulesFunction() {} |
| 62 | 63 |
| 63 // RulesFunction: | 64 // RulesFunction: |
| 64 virtual bool RunImplOnCorrectThread() OVERRIDE; | 65 virtual bool RunImplOnCorrectThread() OVERRIDE; |
| 65 }; | 66 }; |
| 66 | 67 |
| 67 } // namespace extensions | 68 } // namespace extensions |
| 68 | 69 |
| 69 #endif // CHROME_BROWSER_EXTENSIONS_API_DECLARATIVE_DECLARATIVE_API_H_ | 70 #endif // CHROME_BROWSER_EXTENSIONS_API_DECLARATIVE_DECLARATIVE_API_H_ |
| OLD | NEW |