diff options
author | zelidrag@chromium.org <zelidrag@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-16 08:49:30 +0000 |
---|---|---|
committer | zelidrag@chromium.org <zelidrag@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-16 08:49:30 +0000 |
commit | 99b370004f7d287d9904275255967d8e6edf1e7d (patch) | |
tree | d0a69d77ff9df70e6e587fb704bb7b40339090c4 /chrome/renderer/resources/event_bindings.js | |
parent | 76066e5180a794b39d8f9550937b37b05f7793e8 (diff) | |
download | chromium_src-99b370004f7d287d9904275255967d8e6edf1e7d.zip chromium_src-99b370004f7d287d9904275255967d8e6edf1e7d.tar.gz chromium_src-99b370004f7d287d9904275255967d8e6edf1e7d.tar.bz2 |
Revert 81865 - New fileBrowserPrivate and fileHandler APIs added + plus magic needed to safely hand over access to local file system elements from content extension to 3rd party extension.As agreed with aa@ and asargent@, this new API defines following event:chrome.fileHandler.onExecute.addListener(function(id, file_entries) {}This event is invoked when user selects files in ChromeOS file browser. The extension needs to register itself as file content hanlder with following manifest changes: ... "file_browser_actions": [ { "id" : "ActionIdentifier", "default_title" : "Action title", "default_icon" : "icon.png", "file_filters" : [ "filesystem:*.jpeg", ... ] } ... ], ...BUG=chromium-os:11996TEST=ExtensionApiTest.FileBrowserTest, ExtensionManifestTest.FileBrowserActionsReview URL: http://codereview.chromium.org/6749021
TBR=zelidrag@chromium.org
Review URL: http://codereview.chromium.org/6873031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81866 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/resources/event_bindings.js')
-rw-r--r-- | chrome/renderer/resources/event_bindings.js | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/chrome/renderer/resources/event_bindings.js b/chrome/renderer/resources/event_bindings.js index 935d256..3410686 100644 --- a/chrome/renderer/resources/event_bindings.js +++ b/chrome/renderer/resources/event_bindings.js @@ -7,7 +7,6 @@ var chrome = chrome || {}; native function GetChromeHidden(); native function AttachEvent(eventName); native function DetachEvent(eventName); - native function GetExternalFileEntry(fileDefinition); var chromeHidden = GetChromeHidden(); @@ -89,18 +88,6 @@ var chrome = chrome || {}; if (attachedNamedEvents[name]) { if (args) { args = chromeHidden.JSON.parse(args); - // TODO(zelidrag|aa): Remove this hack from here once we enable event - // JSON payload unpacking on C++ side. - if (name == "fileBrowserHandler.onExecute") { - if (args.length != 2) - return; - var fileList = args[1]; - // The second parameter for this event's payload is file definition - // dictionary that we used to reconstruct File API's Entry instance - // here. - for (var i = 0; i < fileList.length; i++) - args[1][i] = GetExternalFileEntry(fileList[i]); - } } return attachedNamedEvents[name].dispatch.apply( attachedNamedEvents[name], args); |