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

Unified Diff: ppapi/proxy/host_var_serialization_rules.cc

Issue 4096008: Var serialization-related proxy stuff. This allows vars to be sent over IPC... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 1 month 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
« no previous file with comments | « ppapi/proxy/host_var_serialization_rules.h ('k') | ppapi/proxy/plugin_var_serialization_rules.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/host_var_serialization_rules.cc
===================================================================
--- ppapi/proxy/host_var_serialization_rules.cc (revision 0)
+++ ppapi/proxy/host_var_serialization_rules.cc (revision 0)
@@ -0,0 +1,91 @@
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "ppapi/proxy/host_var_serialization_rules.h"
+
+#include "base/logging.h"
+#include "ppapi/c/dev/ppb_var_deprecated.h"
+
+namespace pp {
+namespace proxy {
+
+HostVarSerializationRules::HostVarSerializationRules(
+ const PPB_Var_Deprecated* var_interface,
+ PP_Module pp_module)
+ : var_interface_(var_interface),
+ pp_module_(pp_module) {
+}
+
+HostVarSerializationRules::~HostVarSerializationRules() {
+}
+
+void HostVarSerializationRules::SendCallerOwned(const PP_Var& var,
+ std::string* str_val) {
+ if (var.type == PP_VARTYPE_STRING)
+ VarToString(var, str_val);
+}
+
+PP_Var HostVarSerializationRules::BeginReceiveCallerOwned(
+ const PP_Var& var,
+ const std::string* str_val) {
+ if (var.type == PP_VARTYPE_STRING) {
+ // Convert the string to the context of the current process.
+ return var_interface_->VarFromUtf8(pp_module_, str_val->c_str(),
+ static_cast<uint32_t>(str_val->size()));
+ }
+ return var;
+}
+
+void HostVarSerializationRules::EndReceiveCallerOwned(const PP_Var& var) {
+ if (var.type == PP_VARTYPE_STRING) {
+ // Destroy the string BeginReceiveCallerOwned created above.
+ var_interface_->Release(var);
+ }
+}
+
+PP_Var HostVarSerializationRules::ReceivePassRef(const PP_Var& var,
+ const std::string& str_val) {
+ if (var.type == PP_VARTYPE_STRING) {
+ // Convert the string to the context of the current process.
+ return var_interface_->VarFromUtf8(pp_module_, str_val.c_str(),
+ static_cast<uint32_t>(str_val.size()));
+ }
+
+ // See PluginVarSerialization::BeginSendPassRef for an example.
+ if (var.type == PP_VARTYPE_OBJECT)
+ var_interface_->AddRef(var);
+ return var;
+}
+
+void HostVarSerializationRules::BeginSendPassRef(const PP_Var& var,
+ std::string* str_val) {
+ // See PluginVarSerialization::ReceivePassRef for an example. We don't need
+ // to do anything here other than convert the string.
+ if (var.type == PP_VARTYPE_STRING)
+ VarToString(var, str_val);
+}
+
+void HostVarSerializationRules::EndSendPassRef(const PP_Var& var) {
+ // See PluginVarSerialization::ReceivePassRef for an example. We don't need
+ // to do anything here.
+}
+
+void HostVarSerializationRules::VarToString(const PP_Var& var,
+ std::string* str) {
+ DCHECK(var.type == PP_VARTYPE_STRING);
+
+ // This could be optimized to avoid an extra string copy by going to a lower
+ // level of the browser's implementation of strings where we already have
+ // a std::string.
+ uint32_t len = 0;
+ const char* data = var_interface_->VarToUtf8(var, &len);
+ str->assign(data, len);
+}
+
+void HostVarSerializationRules::ReleaseObjectRef(const PP_Var& var) {
+ var_interface_->Release(var);
+}
+
+} // namespace proxy
+} // namespace pp
Property changes on: ppapi/proxy/host_var_serialization_rules.cc
___________________________________________________________________
Added: svn:eol-style
+ LF
« no previous file with comments | « ppapi/proxy/host_var_serialization_rules.h ('k') | ppapi/proxy/plugin_var_serialization_rules.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698