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

Unified Diff: ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_file_ref.cc

Issue 8826011: Remove PP_Module from parameters for PPB_Var.VarFromUtf8. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix windows build and some nacl tests. Created 9 years 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
Index: ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_file_ref.cc
diff --git a/ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_file_ref.cc b/ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_file_ref.cc
index 01951226651cd921641b053ca97e99728a96a646..0667f87cef732f2397d04a62a5795733e090612e 100644
--- a/ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_file_ref.cc
+++ b/ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_file_ref.cc
@@ -99,9 +99,7 @@ PP_Var GetName(PP_Resource file_ref) {
DebugPrintf("PPB_FileRef::GetName: %s\n", NaClSrpcErrorString(srpc_result));
if (srpc_result == NACL_SRPC_RESULT_OK) {
- if (DeserializeTo(GetMainSrpcChannel(), name_bytes.get(), length,
- 1, // argc
- &name))
+ if (DeserializeTo(name_bytes.get(), length, 1 /* argc */, &name))
return name;
}
return PP_MakeUndefined();
@@ -122,9 +120,7 @@ PP_Var GetPath(PP_Resource file_ref) {
NaClSrpcErrorString(srpc_result));
if (srpc_result == NACL_SRPC_RESULT_OK &&
- DeserializeTo(GetMainSrpcChannel(), path_bytes.get(), length,
- 1, // argc
- &path)) {
+ DeserializeTo(path_bytes.get(), length, 1 /* argc */, &path)) {
bbudge 2011/12/07 16:00:34 indentation + 2?
dmichael (off chromium) 2011/12/07 18:07:38 Done.
return path;
bbudge 2011/12/07 16:00:34 indentation - 2?
dmichael (off chromium) 2011/12/07 18:07:38 Done.
}
return PP_MakeUndefined();

Powered by Google App Engine
This is Rietveld 408576698