Replace PluginThreadTaskRunner with a simpler MainPluginThreadTaskRunner.

remoting/client/plugin/chromoting_instance.h

 // Copyright (c) 2012 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_CHROMOTING_INSTANCE_H_
 #define REMOTING_CLIENT_PLUGIN_CHROMOTING_INSTANCE_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/thread_task_runner_handle.h"
 #include "ppapi/c/pp_instance.h"
 #include "ppapi/c/pp_rect.h"
 #include "ppapi/c/pp_resource.h"
 #include "ppapi/cpp/instance.h"
 #include "ppapi/cpp/text_input_controller.h"
 #include "ppapi/cpp/var.h"
 #include "remoting/client/client_context.h"
 #include "remoting/client/client_user_interface.h"
 #include "remoting/client/empty_cursor_filter.h"
 #include "remoting/client/key_event_mapper.h"
+#include "remoting/client/plugin/main_plugin_thread_task_runner.h"
 #include "remoting/client/plugin/pepper_cursor_setter.h"
 #include "remoting/client/plugin/pepper_input_handler.h"
-#include "remoting/client/plugin/pepper_plugin_thread_delegate.h"
 #include "remoting/client/plugin/pepper_video_renderer.h"
 #include "remoting/client/touch_input_scaler.h"
 #include "remoting/proto/event.pb.h"
 #include "remoting/protocol/client_stub.h"
 #include "remoting/protocol/clipboard_stub.h"
 #include "remoting/protocol/connection_to_host.h"
 #include "remoting/protocol/cursor_shape_stub.h"
 #include "remoting/protocol/input_event_tracker.h"
 #include "remoting/protocol/mouse_input_filter.h"
 #include "remoting/protocol/negotiating_client_authenticator.h"
@@ skipping 200 matching lines @@
   bool IsConnected();
 
   // Used as the |FetchSecretCallback| for Me2Me connections.
   // Uses the PIN request dialog in the webapp to obtain the shared secret.
   void FetchSecretFromDialog(
       bool pairing_supported,
       const protocol::SecretFetchedCallback& secret_fetched_callback);
 
   bool initialized_;
 
-  PepperPluginThreadDelegate plugin_thread_delegate_;
-  scoped_refptr<PluginThreadTaskRunner> plugin_task_runner_;
+  scoped_refptr<MainPluginThreadTaskRunner> plugin_task_runner_;

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

Does the ChromotingInstance need to know that this is a
MainPluginThreadTaskRunner, or could it hold it as a SingleThreadTaskRunner or
even TaskRunner, to avoid spreading that implementation detail?

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

Done.

   scoped_ptr<base::ThreadTaskRunnerHandle> thread_task_runner_handle_;
   scoped_ptr<jingle_glue::JingleThreadWrapper> thread_wrapper_;
   ClientContext context_;
   protocol::PerformanceTracker perf_tracker_;
   scoped_ptr<PepperVideoRenderer> video_renderer_;
   pp::View plugin_view_;
 
   // Contains the most-recently-reported desktop shape, if any.
   scoped_ptr<webrtc::DesktopRegion> desktop_shape_;
 
@@ skipping 33 matching lines @@
 
   // Weak reference to this instance, used for global logging and task posting.
   base::WeakPtrFactory<ChromotingInstance> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingInstance);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_PLUGIN_CHROMOTING_INSTANCE_H_