Fix JingleThreadWrapper initialization in PNaCl plugin.

Previously JingleThreadWrapper was initialized in the remoting PNaCl
plugin such that JingleThreadWrapper::current() was always returning
NULL. JingleThreadWrapper::current() is used only to implement
rtc::Thread::Send() and until recently Send() wasn't used in the parts
of webrtc used in remoting. Now rtc::StunRequest uses Send(), which
crashes the plugin.

Added JingleThreadWrapper::WrapTaskRunner() that properly initializes
current() and is used in the remoting plugin now.

Also JingleThreadWrapper is no longer passes SocketServer to the
parent as it is no longer necessary.

BUG=451643

Review URL: https://codereview.chromium.org/876433003

Cr-Commit-Position: refs/heads/master@{#313213}

1 files changed, 10 insertions, 5 deletions

diff --git a/jingle/glue/thread_wrapper.h b/jingle/glue/thread_wrapper.h
index 6a045bf..7969d2b 100644
--- a/jingle/glue/thread_wrapper.h
+++ b/jingle/glue/thread_wrapper.h
@@ -35,13 +35,15 @@ class JingleThreadWrapper : public base::MessageLoop::DestructionObserver,
   // MessageLoop is destroyed.
   static void EnsureForCurrentMessageLoop();
 
-  // Returns thread wrapper for the current thread. NULL is returned
-  // if EnsureForCurrentMessageLoop() has never been called for this
-  // thread.
+  // Creates JingleThreadWrapper for |task_runner| that runs tasks on the
+  // current thread.
+  static scoped_ptr<JingleThreadWrapper> WrapTaskRunner(
+      scoped_refptr<base::SingleThreadTaskRunner> task_runner);
+
+  // Returns thread wrapper for the current thread or nullptr if it doesn't
+  // exist.
   static JingleThreadWrapper* current();
 
-  explicit JingleThreadWrapper(
-     scoped_refptr<base::SingleThreadTaskRunner> task_runner);
   ~JingleThreadWrapper() override;
 
   // Sets whether the thread can be used to send messages
@@ -95,6 +97,9 @@ class JingleThreadWrapper : public base::MessageLoop::DestructionObserver,
   typedef std::map<int, rtc::Message> MessagesQueue;
   struct PendingSend;
 
+  explicit JingleThreadWrapper(
+     scoped_refptr<base::SingleThreadTaskRunner> task_runner);
+
   void PostTaskInternal(
       int delay_ms, rtc::MessageHandler* handler,
       uint32 message_id, rtc::MessageData* data);