Fix JingleThreadWrapper initialization in PNaCl plugin.

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
Committed: https://crrev.com/8eb15cc4766a09c82d0e4f60739b7675f23d4e60
Cr-Commit-Position: refs/heads/master@{#313213}

[Sergey Ulanov, 2015-01-26 23:14:16]

Patchset #1 (id:1) has been deleted

[Sergey Ulanov, 2015-01-26 23:40:49]

Patchset #1 (id:20001) has been deleted

[Sergey Ulanov, 2015-01-26 23:41:06]

Patchset #1 (id:40001) has been deleted

[Sergey Ulanov, 2015-01-26 23:42:10]

Patchset #2 (id:80001) has been deleted

[Sergey Ulanov, 2015-01-26 23:42:36]

Patchset #1 (id:60001) has been deleted

[Sergey Ulanov, 2015-01-26 23:45:31]

sergeyu@chromium.org changed reviewers:
+ rmsousa@chromium.org

[rmsousa, 2015-01-27 00:48:49]

lgtm

https://codereview.chromium.org/876433003/diff/100001/remoting/client/plugin/...
File remoting/client/plugin/chromoting_instance.cc (right):

https://codereview.chromium.org/876433003/diff/100001/remoting/client/plugin/...
remoting/client/plugin/chromoting_instance.cc:201: ProcessLogToUI("Initializing
thread wrapper");
Is this a debugging leftover?

[Sergey Ulanov, 2015-01-27 01:27:29]

https://codereview.chromium.org/876433003/diff/100001/remoting/client/plugin/...
File remoting/client/plugin/chromoting_instance.cc (right):

https://codereview.chromium.org/876433003/diff/100001/remoting/client/plugin/...
remoting/client/plugin/chromoting_instance.cc:201: ProcessLogToUI("Initializing
thread wrapper");
On 2015/01/27 00:48:48, rmsousa wrote:
> Is this a debugging leftover?

Yes. Thanks!

[Sergey Ulanov, 2015-01-27 01:28:58]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2015-01-27 01:30:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/876433003/120001

[commit-bot: I haz the power, 2015-01-27 02:54:24]

Committed patchset #2 (id:120001)

[commit-bot: I haz the power, 2015-01-27 02:55:39]

Patchset 2 (id:??) landed as
https://crrev.com/8eb15cc4766a09c82d0e4f60739b7675f23d4e60
Cr-Commit-Position: refs/heads/master@{#313213}