diff options
| author | asargent@chromium.org <asargent@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-20 02:48:01 +0000 |
|---|---|---|
| committer | asargent@chromium.org <asargent@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-20 02:48:01 +0000 |
| commit | 602542d6a7ea6313321d9badfe232df3daf12cef (patch) | |
| tree | cda79b0ce15174a989df6925f891b3d23036f9d5 /chrome/renderer/extensions/extension_dispatcher.h | |
| parent | 415fe1d7c0dc7c06aaae7840c4d42d8120420fcf (diff) | |
| download | chromium_src-602542d6a7ea6313321d9badfe232df3daf12cef.zip chromium_src-602542d6a7ea6313321d9badfe232df3daf12cef.tar.gz chromium_src-602542d6a7ea6313321d9badfe232df3daf12cef.tar.bz2 | |
Use Value objects instead of JSON strings in extension API response IPCs.
When we're sending the result of an extension API method call, we currently
convert a Value object into a JSON string in the browser process, and then call
JSON.parse in javascript on the renderer side. This CL changes that to pass the
Value object in the IPC, and uses V8ValueConverter to turn the Value into the
appropriate v8 native object, avoiding the need to parse JSON.
This is a first step towards supporting binary data for API request
parameters and responses.
BUG=122675
TEST=No new functionality ; existing unit and browser tests should still pass.
Review URL: https://chromiumcodereview.appspot.com/10041047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133133 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/extensions/extension_dispatcher.h')
| -rw-r--r-- | chrome/renderer/extensions/extension_dispatcher.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/renderer/extensions/extension_dispatcher.h b/chrome/renderer/extensions/extension_dispatcher.h index 4e87366..cb7f60a 100644 --- a/chrome/renderer/extensions/extension_dispatcher.h +++ b/chrome/renderer/extensions/extension_dispatcher.h @@ -102,7 +102,7 @@ class ExtensionDispatcher : public content::RenderProcessObserver { void OnExtensionResponse(int request_id, bool success, - const std::string& response, + const base::ListValue& response, const std::string& error); // Checks that the current context contains an extension that has permission |