Replace PluginThreadTaskRunner with a simpler MainPluginThreadTaskRunner.

remoting/client/plugin/main_plugin_thread_task_runner.h

Index: remoting/client/plugin/main_plugin_thread_task_runner.h
diff --git a/remoting/client/plugin/main_plugin_thread_task_runner.h b/remoting/client/plugin/main_plugin_thread_task_runner.h
new file mode 100644
index 0000000000000000000000000000000000000000..546a7a00e8e5689ec208b8f7a7a7078f115452a4
--- /dev/null
+++ b/remoting/client/plugin/main_plugin_thread_task_runner.h
@@ -0,0 +1,49 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef REMOTING_CLIENT_PLUGIN_MAIN_PLUGIN_THREAD_TASK_RUNNER_H_
+#define REMOTING_CLIENT_PLUGIN_MAIN_PLUGIN_THREAD_TASK_RUNNER_H_
+
+#include "base/callback_forward.h"
+#include "base/compiler_specific.h"
+#include "base/macros.h"
+#include "base/single_thread_task_runner.h"
+#include "base/time/time.h"
+#include "ppapi/utility/completion_callback_factory.h"
+
+namespace pp {
+class Core;
+}  // namespace pp
+
+namespace remoting {
+
+// SingleThreadTaskRunner implementation for the main plugin thread.
+class MainPluginThreadTaskRunner : public base::SingleThreadTaskRunner {

[Wez, 2015/10/20 23:05:48]

nit: I'd suggest calling this PepperMainThreadTaskRunner, since it's
PPAPI-specific, and that would fit the naming of the other PPAPI-specific code.

[Sergey Ulanov, 2015/10/20 23:28:05]

Done.

+ public:
+  MainPluginThreadTaskRunner();
+
+  // base::SingleThreadTaskRunner interface.
+  bool PostDelayedTask(const tracked_objects::Location& from_here,
+                       const base::Closure& task,
+                       base::TimeDelta delay) override;
+  bool PostNonNestableDelayedTask(const tracked_objects::Location& from_here,
+                                  const base::Closure& task,
+                                  base::TimeDelta delay) override;
+  bool RunsTasksOnCurrentThread() const override;
+
+ protected:
+  ~MainPluginThreadTaskRunner() override;
+
+ private:
+  void RunTask(int32_t result, const base::Closure& closure);

[Wez, 2015/10/20 23:05:48]

nit: Not at all obvious from the name & parameters what this actually does!

[Sergey Ulanov, 2015/10/20 23:28:05]

Added comment and renamed second parameter.

+
+  pp::Core* core_;
+  pp::CompletionCallbackFactory<MainPluginThreadTaskRunner> callback_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(MainPluginThreadTaskRunner);
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_CLIENT_PLUGIN_MAIN_PLUGIN_THREAD_TASK_RUNNER_H_