ppapi: Remove use of MessageLoopProxy and deprecated MessageLoop APIs

ppapi/proxy/message_handler.cc

Index: ppapi/proxy/message_handler.cc
diff --git a/ppapi/proxy/message_handler.cc b/ppapi/proxy/message_handler.cc
index af9a45f8380eea2def1013c930d6453c628b8534..9ac57e017bd704f54bcf01bf2aa72003250b3256 100644
--- a/ppapi/proxy/message_handler.cc
+++ b/ppapi/proxy/message_handler.cc
@@ -4,6 +4,8 @@
 
 #include "ppapi/proxy/message_handler.h"
 
+#include "base/location.h"
+#include "base/single_thread_task_runner.h"
 #include "ipc/ipc_message.h"
 #include "ppapi/proxy/plugin_dispatcher.h"
 #include "ppapi/proxy/ppapi_messages.h"
@@ -93,10 +95,8 @@ MessageHandler::~MessageHandler() {
   if (message_loop_->message_loop_proxy().get()) {
     // The posted task won't have the proxy lock, but that's OK, it doesn't
     // touch any internal state; it's a direct call on the plugin's function.
-    message_loop_->message_loop_proxy()->PostTask(FROM_HERE,
-        base::Bind(handler_if_->Destroy,
-                   instance_,
-                   user_data_));
+    message_loop_->message_loop_proxy()->PostTask(
+        FROM_HERE, base::Bind(handler_if_->Destroy, instance_, user_data_));
   }
 }
 
@@ -105,23 +105,19 @@ bool MessageHandler::LoopIsValid() const {
 }
 
 void MessageHandler::HandleMessage(ScopedPPVar var) {
-  message_loop_->message_loop_proxy()->PostTask(FROM_HERE,
-      RunWhileLocked(base::Bind(&HandleMessageWrapper,
-                                handler_if_->HandleMessage,
-                                instance_,
-                                user_data_,
-                                var)));
+  message_loop_->message_loop_proxy()->PostTask(
+      FROM_HERE, RunWhileLocked(base::Bind(&HandleMessageWrapper,
+                                           handler_if_->HandleMessage,
+                                           instance_, user_data_, var)));
 }
 
 void MessageHandler::HandleBlockingMessage(ScopedPPVar var,
                                            scoped_ptr<IPC::Message> reply_msg) {
-  message_loop_->message_loop_proxy()->PostTask(FROM_HERE,
-      RunWhileLocked(base::Bind(&HandleBlockingMessageWrapper,
-                                handler_if_->HandleBlockingMessage,
-                                instance_,
-                                user_data_,
-                                var,
-                                base::Passed(reply_msg.Pass()))));
+  message_loop_->message_loop_proxy()->PostTask(

[bbudge, 2015/06/10 20:57:46]

Looks like this was missed.

[Sami, 2015/06/11 18:03:46]

message_loop_ is a MessageLoopResource, not a base::MessageLoop, so I wasn't
sure if I should be reworking it or not. I was under the impression that PPAPI
defines some interfaces that are called from the runtime, so I was careful about
not changing too many things. Do you think this function should be renamed too?

Edit: scratch that, I've now renamed it too.

+      FROM_HERE,
+      RunWhileLocked(base::Bind(
+          &HandleBlockingMessageWrapper, handler_if_->HandleBlockingMessage,
+          instance_, user_data_, var, base::Passed(reply_msg.Pass()))));
 }
 
 MessageHandler::MessageHandler(