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

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

Issue 9317072: Allow omitting optional parameters for Extensions API functions (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Synced and merged. Created 8 years, 9 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 (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 #include "chrome/renderer/extensions/custom_bindings_util.h" 5 #include "chrome/renderer/extensions/custom_bindings_util.h"
6 6
7 #include <map> 7 #include <map>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/string_util.h" 10 #include "base/string_util.h"
(...skipping 26 matching lines...) Expand all
37 "extensions/browser_action_custom_bindings.js", 37 "extensions/browser_action_custom_bindings.js",
38 "extensions/content_settings_custom_bindings.js", 38 "extensions/content_settings_custom_bindings.js",
39 "extensions/experimental.declarative_custom_bindings.js", 39 "extensions/experimental.declarative_custom_bindings.js",
40 "extensions/devtools_custom_bindings.js", 40 "extensions/devtools_custom_bindings.js",
41 "extensions/input.ime_custom_bindings.js", 41 "extensions/input.ime_custom_bindings.js",
42 "extensions/omnibox_custom_bindings.js", 42 "extensions/omnibox_custom_bindings.js",
43 "extensions/page_action_custom_bindings.js", 43 "extensions/page_action_custom_bindings.js",
44 "extensions/storage_custom_bindings.js", 44 "extensions/storage_custom_bindings.js",
45 "extensions/tts_engine_custom_bindings.js", 45 "extensions/tts_engine_custom_bindings.js",
46 "extensions/types_custom_bindings.js", 46 "extensions/types_custom_bindings.js",
47 "extensions/windows_custom_bindings.js",
48 }; 47 };
49 static const size_t kJavascriptFilesSize = arraysize(kJavascriptFiles); 48 static const size_t kJavascriptFilesSize = arraysize(kJavascriptFiles);
50 49
51 // Must match kJavascriptFiles. 50 // Must match kJavascriptFiles.
52 static const int kResourceIDs[] = { 51 static const int kResourceIDs[] = {
53 IDR_BROWSER_ACTION_CUSTOM_BINDINGS_JS, 52 IDR_BROWSER_ACTION_CUSTOM_BINDINGS_JS,
54 IDR_CONTENT_SETTINGS_CUSTOM_BINDINGS_JS, 53 IDR_CONTENT_SETTINGS_CUSTOM_BINDINGS_JS,
55 IDR_EXPERIMENTAL_DECLARATIVE_CUSTOM_BINDINGS_JS, 54 IDR_EXPERIMENTAL_DECLARATIVE_CUSTOM_BINDINGS_JS,
56 IDR_DEVTOOLS_CUSTOM_BINDINGS_JS, 55 IDR_DEVTOOLS_CUSTOM_BINDINGS_JS,
57 IDR_INPUT_IME_CUSTOM_BINDINGS_JS, 56 IDR_INPUT_IME_CUSTOM_BINDINGS_JS,
58 IDR_OMNIBOX_CUSTOM_BINDINGS_JS, 57 IDR_OMNIBOX_CUSTOM_BINDINGS_JS,
59 IDR_PAGE_ACTION_CUSTOM_BINDINGS_JS, 58 IDR_PAGE_ACTION_CUSTOM_BINDINGS_JS,
60 IDR_STORAGE_CUSTOM_BINDINGS_JS, 59 IDR_STORAGE_CUSTOM_BINDINGS_JS,
61 IDR_TTS_ENGINE_CUSTOM_BINDINGS_JS, 60 IDR_TTS_ENGINE_CUSTOM_BINDINGS_JS,
62 IDR_TYPES_CUSTOM_BINDINGS_JS, 61 IDR_TYPES_CUSTOM_BINDINGS_JS,
63 IDR_WINDOWS_CUSTOM_BINDINGS_JS,
64 }; 62 };
65 static const size_t kResourceIDsSize = arraysize(kResourceIDs); 63 static const size_t kResourceIDsSize = arraysize(kResourceIDs);
66 64
67 static const char* kDependencies[] = { 65 static const char* kDependencies[] = {
68 "extensions/schema_generated_bindings.js", 66 "extensions/schema_generated_bindings.js",
69 }; 67 };
70 static const size_t kDependencyCount = arraysize(kDependencies); 68 static const size_t kDependencyCount = arraysize(kDependencies);
71 69
72 std::vector<v8::Extension*> result; 70 std::vector<v8::Extension*> result;
73 71
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 return allowed; 160 return allowed;
163 } else { 161 } else {
164 return allowed && 162 return allowed &&
165 !ExtensionAPI::GetInstance()->IsWholeAPIPrivileged(api_name); 163 !ExtensionAPI::GetInstance()->IsWholeAPIPrivileged(api_name);
166 } 164 }
167 } 165 }
168 166
169 } // namespace custom_bindings_util 167 } // namespace custom_bindings_util
170 168
171 } // namespace extensions 169 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/renderer/extensions/chrome_v8_context.cc ('k') | chrome/renderer/extensions/json_schema_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698