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

Side by Side Diff: extensions/renderer/dispatcher_delegate.h

Issue 442553002: Remove unneccessary arguments from extensions::DispatcherDelegate (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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
« no previous file with comments | « extensions/renderer/dispatcher.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #ifndef EXTENSIONS_RENDERER_DISPATCHER_DELEGATE_H 5 #ifndef EXTENSIONS_RENDERER_DISPATCHER_DELEGATE_H
6 #define EXTENSIONS_RENDERER_DISPATCHER_DELEGATE_H 6 #define EXTENSIONS_RENDERER_DISPATCHER_DELEGATE_H
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 10
(...skipping 24 matching lines...) Expand all
35 virtual scoped_ptr<ScriptContext> CreateScriptContext( 35 virtual scoped_ptr<ScriptContext> CreateScriptContext(
36 const v8::Handle<v8::Context>& v8_context, 36 const v8::Handle<v8::Context>& v8_context,
37 blink::WebFrame* frame, 37 blink::WebFrame* frame,
38 const Extension* extension, 38 const Extension* extension,
39 Feature::Context context_type) = 0; 39 Feature::Context context_type) = 0;
40 40
41 // Initializes origin permissions for a newly created extension context. 41 // Initializes origin permissions for a newly created extension context.
42 virtual void InitOriginPermissions(const Extension* extension, 42 virtual void InitOriginPermissions(const Extension* extension,
43 bool is_extension_active) {} 43 bool is_extension_active) {}
44 44
45 // Includes additional native handlers in a given ModuleSystem. 45 // Includes additional native handlers in a ScriptContext's ModuleSystem.
46 virtual void RegisterNativeHandlers(Dispatcher* dispatcher, 46 virtual void RegisterNativeHandlers(Dispatcher* dispatcher,
47 ModuleSystem* module_system, 47 ModuleSystem* module_system,
48 ScriptContext* context) {} 48 ScriptContext* context) {}
49 49
50 // Includes additional source resources into the resource map. 50 // Includes additional source resources into the resource map.
51 virtual void PopulateSourceMap(ResourceBundleSourceMap* source_map) {} 51 virtual void PopulateSourceMap(ResourceBundleSourceMap* source_map) {}
52 52
53 // Requires additional modules within an extension context's module system. 53 // Requires additional modules within an extension context's module system.
54 virtual void RequireAdditionalModules(ModuleSystem* module_system, 54 virtual void RequireAdditionalModules(ScriptContext* context,
55 const Extension* extension,
56 Feature::Context context_type,
57 bool is_within_platform_app) {} 55 bool is_within_platform_app) {}
58 56
59 // Allows the delegate to respond to an updated set of active extensions in 57 // Allows the delegate to respond to an updated set of active extensions in
60 // the Dispatcher. 58 // the Dispatcher.
61 virtual void OnActiveExtensionsUpdated( 59 virtual void OnActiveExtensionsUpdated(
62 const std::set<std::string>& extension_ids) {} 60 const std::set<std::string>& extension_ids) {}
63 61
64 // Sets the current Chrome channel. 62 // Sets the current Chrome channel.
65 // TODO(rockot): This doesn't belong in a generic extensions system interface. 63 // TODO(rockot): This doesn't belong in a generic extensions system interface.
66 // See http://crbug.com/368431. 64 // See http://crbug.com/368431.
(...skipping 18 matching lines...) Expand all
85 const extensions::URLPatternSet& origin_set) {} 83 const extensions::URLPatternSet& origin_set) {}
86 84
87 // Allows the delegate to respond to reports from the browser about WebRequest 85 // Allows the delegate to respond to reports from the browser about WebRequest
88 // API usage from within this process. 86 // API usage from within this process.
89 virtual void HandleWebRequestAPIUsage(bool webrequest_used) {} 87 virtual void HandleWebRequestAPIUsage(bool webrequest_used) {}
90 }; 88 };
91 89
92 } // namespace extensions 90 } // namespace extensions
93 91
94 #endif // EXTENSIONS_RENDERER_DISPATCHER_DELEGATE_H 92 #endif // EXTENSIONS_RENDERER_DISPATCHER_DELEGATE_H
OLDNEW
« no previous file with comments | « extensions/renderer/dispatcher.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698