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

Side by Side Diff: chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc

Issue 498513002: Respect the clipboardRead and clipboardWrite permissions in content scripts. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address more comments Created 6 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 | Annotate | Revision Log
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/renderer/extensions/chrome_extensions_dispatcher_delegate.h" 5 #include "chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/sha1.h" 8 #include "base/sha1.h"
9 #include "base/strings/string_number_conversions.h" 9 #include "base/strings/string_number_conversions.h"
10 #include "chrome/common/chrome_switches.h" 10 #include "chrome/common/chrome_switches.h"
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 } 57 }
58 58
59 ChromeExtensionsDispatcherDelegate::~ChromeExtensionsDispatcherDelegate() { 59 ChromeExtensionsDispatcherDelegate::~ChromeExtensionsDispatcherDelegate() {
60 } 60 }
61 61
62 scoped_ptr<extensions::ScriptContext> 62 scoped_ptr<extensions::ScriptContext>
63 ChromeExtensionsDispatcherDelegate::CreateScriptContext( 63 ChromeExtensionsDispatcherDelegate::CreateScriptContext(
64 const v8::Handle<v8::Context>& v8_context, 64 const v8::Handle<v8::Context>& v8_context,
65 blink::WebFrame* frame, 65 blink::WebFrame* frame,
66 const extensions::Extension* extension, 66 const extensions::Extension* extension,
67 extensions::Feature::Context context_type) { 67 extensions::Feature::Context context_type,
68 return scoped_ptr<extensions::ScriptContext>(new extensions::ChromeV8Context( 68 const extensions::Extension* effective_extension,
69 v8_context, frame, extension, context_type)); 69 extensions::Feature::Context effective_context_type) {
70 return scoped_ptr<extensions::ScriptContext>(
71 new extensions::ChromeV8Context(v8_context,
72 frame,
73 extension,
74 context_type,
75 effective_extension,
76 effective_context_type));
70 } 77 }
71 78
72 void ChromeExtensionsDispatcherDelegate::InitOriginPermissions( 79 void ChromeExtensionsDispatcherDelegate::InitOriginPermissions(
73 const extensions::Extension* extension, 80 const extensions::Extension* extension,
74 bool is_extension_active) { 81 bool is_extension_active) {
75 // TODO(jstritar): We should try to remove this special case. Also, these 82 // TODO(jstritar): We should try to remove this special case. Also, these
76 // whitelist entries need to be updated when the kManagement permission 83 // whitelist entries need to be updated when the kManagement permission
77 // changes. 84 // changes.
78 if (is_extension_active && 85 if (is_extension_active &&
79 extension->permissions_data()->HasAPIPermission( 86 extension->permissions_data()->HasAPIPermission(
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
348 new extensions::PermissionSet(extensions::APIPermissionSet(), 355 new extensions::PermissionSet(extensions::APIPermissionSet(),
349 extensions::ManifestPermissionSet(), 356 extensions::ManifestPermissionSet(),
350 origin_set, 357 origin_set,
351 extensions::URLPatternSet())); 358 extensions::URLPatternSet()));
352 } 359 }
353 360
354 void ChromeExtensionsDispatcherDelegate::HandleWebRequestAPIUsage( 361 void ChromeExtensionsDispatcherDelegate::HandleWebRequestAPIUsage(
355 bool webrequest_used) { 362 bool webrequest_used) {
356 webrequest_used_ = webrequest_used; 363 webrequest_used_ = webrequest_used;
357 } 364 }
OLDNEW
« no previous file with comments | « chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.h ('k') | chrome/renderer/extensions/chrome_v8_context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698