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

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

Issue 8840007: GetDocumentURL is added to PPB_Testing_Dev. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fixed codereview issues. 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/browser_ppb_testing_rpc_server.cc
diff --git a/ppapi/native_client/src/shared/ppapi_proxy/browser_ppb_testing_rpc_server.cc b/ppapi/native_client/src/shared/ppapi_proxy/browser_ppb_testing_rpc_server.cc
index ceb8aaedb61b5a6945811747a639c2973d9a228c..7b929c4f6098497dccabc82f5aeed327aa4758db 100644
--- a/ppapi/native_client/src/shared/ppapi_proxy/browser_ppb_testing_rpc_server.cc
+++ b/ppapi/native_client/src/shared/ppapi_proxy/browser_ppb_testing_rpc_server.cc
@@ -90,4 +90,26 @@ void PpbTestingRpcServer::PPB_Testing_GetLiveObjectsForInstance(
rpc->result = NACL_SRPC_RESULT_OK;
}
+void PpbTestingRpcServer::PPB_Testing_GetDocumentURL(
+ NaClSrpcRpc* rpc,
+ NaClSrpcClosure* done,
+ PP_Instance instance,
+ nacl_abi_size_t* components_bytes, char* components,
+ nacl_abi_size_t* url_bytes, char* url) {
+ NaClSrpcClosureRunner runner(done);
+ rpc->result = NACL_SRPC_RESULT_APP_ERROR;
+ if (*components_bytes != sizeof(struct PP_URLComponents_Dev))
+ return;
+ if (*url_bytes != ppapi_proxy::kMaxVarSize)
+ return;
+
+ struct PP_Var pp_url = PPBTestingInterface()->GetDocumentURL(
+ instance, reinterpret_cast<struct PP_URLComponents_Dev*>(components));
+
+ if (!ppapi_proxy::SerializeTo(&pp_url, url, url_bytes))
+ return;
+
+ DebugPrintf("PPB_Testing_Dev::GetDocumentURL\n");
yzshen1 2011/12/14 19:16:54 nit: I think it is better to also output relevant
ygorshenin 2011/12/15 11:48:22 Done.
+ rpc->result = NACL_SRPC_RESULT_OK;
+}

Powered by Google App Engine
This is Rietveld 408576698