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

Unified Diff: ppapi/proxy/ppb_url_loader_proxy.cc

Issue 7874002: This patch tries to remove most of the manual registration for Pepper interfaces, and replaces it... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 3 months 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/ppb_url_loader_proxy.h ('k') | ppapi/proxy/ppb_url_request_info_proxy.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/ppb_url_loader_proxy.cc
===================================================================
--- ppapi/proxy/ppb_url_loader_proxy.cc (revision 100758)
+++ ppapi/proxy/ppb_url_loader_proxy.cc (working copy)
@@ -70,9 +70,8 @@
INTERFACE_ID_PPB_URL_LOADER, params));
}
-InterfaceProxy* CreateURLLoaderProxy(Dispatcher* dispatcher,
- const void* target_interface) {
- return new PPB_URLLoader_Proxy(dispatcher, target_interface);
+InterfaceProxy* CreateURLLoaderProxy(Dispatcher* dispatcher) {
+ return new PPB_URLLoader_Proxy(dispatcher);
}
} // namespace
@@ -346,9 +345,8 @@
std::string read_buffer;
};
-PPB_URLLoader_Proxy::PPB_URLLoader_Proxy(Dispatcher* dispatcher,
- const void* target_interface)
- : InterfaceProxy(dispatcher, target_interface),
+PPB_URLLoader_Proxy::PPB_URLLoader_Proxy(Dispatcher* dispatcher)
+ : InterfaceProxy(dispatcher),
callback_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)),
host_urlloader_trusted_interface_(NULL) {
}
@@ -363,18 +361,6 @@
}
// static
-const InterfaceProxy::Info* PPB_URLLoader_Proxy::GetInfo() {
- static const Info info = {
- thunk::GetPPB_URLLoader_Thunk(),
- PPB_URLLOADER_INTERFACE,
- INTERFACE_ID_PPB_URL_LOADER,
- false,
- &CreateURLLoaderProxy,
- };
- return &info;
-}
-
-// static
const InterfaceProxy::Info* PPB_URLLoader_Proxy::GetTrustedInfo() {
static const Info info = {
thunk::GetPPB_URLLoaderTrusted_Thunk(),
@@ -443,7 +429,7 @@
void PPB_URLLoader_Proxy::OnMsgCreate(PP_Instance instance,
HostResource* result) {
- EnterFunctionNoLock<ResourceCreationAPI> enter(instance, true);
+ thunk::EnterResourceCreation enter(instance);
if (enter.succeeded()) {
result->SetHostResource(instance,
enter.functions()->CreateURLLoader(instance));
@@ -456,8 +442,7 @@
uint32_t serialized_callback) {
// Have to be careful to always issue the callback, so don't return early.
EnterHostFromHostResource<PPB_URLLoader_API> enter(loader);
- EnterFunctionNoLock<ResourceCreationAPI> enter_creation(
- loader.instance(), true);
+ thunk::EnterResourceCreation enter_creation(loader.instance());
PP_CompletionCallback callback = ReceiveCallback(serialized_callback);
« no previous file with comments | « ppapi/proxy/ppb_url_loader_proxy.h ('k') | ppapi/proxy/ppb_url_request_info_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698