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

Unified Diff: chrome/browser/extensions/api/permissions/permissions_api.cc

Issue 9309044: Supporting more APIs with json_schema_compiler (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: support for choices Created 8 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/permissions/permissions_api.cc
diff --git a/chrome/browser/extensions/api/permissions/permissions_api.cc b/chrome/browser/extensions/api/permissions/permissions_api.cc
index 40b31bb3c12699767aebeb35625706f760376f9f..acbde79d011e527e5d48e47c3e3b3848eeca6182 100644
--- a/chrome/browser/extensions/api/permissions/permissions_api.cc
+++ b/chrome/browser/extensions/api/permissions/permissions_api.cc
@@ -44,15 +44,17 @@ bool ignore_user_gesture_for_tests = false;
bool ContainsPermissionsFunction::RunImpl() {
scoped_ptr<Contains::Params> params(Contains::Params::Create(*args_));
- EXTENSION_FUNCTION_VALIDATE(params.get());
scoped_refptr<ExtensionPermissionSet> permissions =
UnpackPermissionSet(params->permissions, &error_);
+ LOG(WARNING) << "IN HERE\n\n\n";
if (!permissions.get())
return false;
+ LOG(WARNING) << "derp HERE\n\n\n";
result_.reset(Contains::Result::Create(
GetExtension()->GetActivePermissions()->Contains(*permissions)));
+ LOG(WARNING) << "OUT HERE\n\n\n";
return true;
}

Powered by Google App Engine
This is Rietveld 408576698