diff options
author | rafaelw@chromium.org <rafaelw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-01 18:34:23 +0000 |
---|---|---|
committer | rafaelw@chromium.org <rafaelw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-01 18:34:23 +0000 |
commit | 70f2c448125ebe327d97596af088d8e81e82b901 (patch) | |
tree | 38c994b955371122258ad4e2bd009730d68d4274 | |
parent | 5c096b1116bb81f9dde31fee1663c13598f2e855 (diff) | |
download | chromium_src-70f2c448125ebe327d97596af088d8e81e82b901.zip chromium_src-70f2c448125ebe327d97596af088d8e81e82b901.tar.gz chromium_src-70f2c448125ebe327d97596af088d8e81e82b901.tar.bz2 |
make chrome.tabs.executeScript/insertCss details arg required.
they were marked as optional and if either functions were called no arguments, the extension process was terminated because it thought was expecting an argument list.
BUG=33628
Review URL: http://codereview.chromium.org/558049
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37713 0039d316-1c4b-4281-b951-d872f2087c98
-rwxr-xr-x | chrome/common/extensions/api/extension_api.json | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/chrome/common/extensions/api/extension_api.json b/chrome/common/extensions/api/extension_api.json index f3a6182..7f82edf4 100755 --- a/chrome/common/extensions/api/extension_api.json +++ b/chrome/common/extensions/api/extension_api.json @@ -901,8 +901,7 @@ "code": {"type": "string", "optional": true, "description": "JavaScript code to execute."}, "file": {"type": "string", "optional": true, "description": "JavaScript file to execute."}, "allFrames": {"type": "boolean", "optional": true, "description": "If allFrames is true, this function injects script into all frames of current page. By default, it's false and only inject script into top main frame."} - }, - "optional": true + } }, { "type": "function", @@ -927,8 +926,7 @@ "code": {"type": "string", "optional": true, "description": "CSS code to be injected."}, "file": {"type": "string", "optional": true, "description": "CSS file to be injected."}, "allFrames": {"type": "boolean", "optional": true, "description": "If allFrames is true, this function injects css text into all frames of current page. By default, it's false and only inject css text into top main frame."} - }, - "optional": true + } }, { "type": "function", |