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

Unified Diff: ppapi/proxy/resource_reply_thread_registrar.cc

Issue 869883003: Never lock the Pepper proxy lock on the IO thread (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix size_t vs int32_t Created 5 years, 9 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/resource_reply_thread_registrar.h ('k') | ppapi/proxy/tracked_callback_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/proxy/resource_reply_thread_registrar.cc
diff --git a/ppapi/proxy/resource_reply_thread_registrar.cc b/ppapi/proxy/resource_reply_thread_registrar.cc
index d0c697deab7338603c7db9f04f65bfa1279b55b1..274d90264923f6f642ffa65bd7b331f8aeb61826 100644
--- a/ppapi/proxy/resource_reply_thread_registrar.cc
+++ b/ppapi/proxy/resource_reply_thread_registrar.cc
@@ -50,11 +50,6 @@ void ResourceReplyThreadRegistrar::Unregister(PP_Resource resource) {
map_.erase(resource);
}
-void ResourceReplyThreadRegistrar::HandleOnIOThread(uint32 nested_msg_type) {
- base::AutoLock auto_lock(lock_);
- io_thread_message_types_.insert(nested_msg_type);
-}
-
scoped_refptr<base::MessageLoopProxy>
ResourceReplyThreadRegistrar::GetTargetThread(
const ResourceMessageReplyParams& reply_params,
@@ -72,9 +67,6 @@ ResourceReplyThreadRegistrar::GetTargetThread(
}
}
- if (io_thread_message_types_.count(nested_msg.type()) != 0)
- return scoped_refptr<base::MessageLoopProxy>();
-
return main_thread_;
}
« no previous file with comments | « ppapi/proxy/resource_reply_thread_registrar.h ('k') | ppapi/proxy/tracked_callback_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698