Marked IPC::ChannelProxy as non thread-safe. Added DCHECKs to verify that all public IPC::ChannelPr…

Marked IPC::ChannelProxy as non thread-safe. Added DCHECKs to verify that public methods (with Send() being the only exception) are called on the thread that created the object. 

Tests calling Send() from a wrong thread should be fixed first before similar check can be added to Send(). See http://crbug.com/163523 for details.

BUG=163091, 163523
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=171106

[alexeypa (please no reviews), 2012-11-29 18:42:36]

PTAL.

ipc/* - cpu@
remoting/* - wez@

[cpu_(ooo_6.6-7.5), 2012-11-29 20:16:11]

does the desktop_session file need to be in this CL?

[Wez, 2012-11-29 21:21:47]

On 2012/11/29 20:16:11, cpu wrote:
> does the desktop_session file need to be in this CL?

+1; Alex, the remoting/ change LGTM but please split the remoting/ change to a
separate CL & TBR=wez it.

[alexeypa (please no reviews), 2012-11-29 21:36:30]

On 2012/11/29 21:21:47, Wez wrote:
> On 2012/11/29 20:16:11, cpu wrote:
> > does the desktop_session file need to be in this CL?
> 
> +1; Alex, the remoting/ change LGTM but please split the remoting/ change to a
> separate CL & TBR=wez it.

Yes, it makes sense. I'll submit remoting/* changes as a separate CL. In the
meanwhile, IPC::ChannelProxy change broke a few tests. I'll post an update once
I get them fixed.

[alexeypa (please no reviews), 2012-11-30 03:35:39]

Please take another look. remoting/* changes have been submitted as
https://chromiumcodereview.appspot.com/11412254/.

The tests are passing now. remoting_unittests failure is expected. It will go
away once try bots will catch up. I filed http://crbug.com/163523 to track
fixing the tests that call IPC::ChannelProxy::Send() from a wrong thread.

[alexeypa (please no reviews), 2012-12-03 20:35:07]

Ping.

[cpu_(ooo_6.6-7.5), 2012-12-04 19:17:35]

sorry this dropped from the radar. lgtm.

[commit-bot: I haz the power, 2012-12-04 19:20:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/alexeypa@chromium.org/11308278/4002

[commit-bot: I haz the power, 2012-12-04 22:04:32]

Retried try job too often on linux_chromeos for step(s) browser_tests

[commit-bot: I haz the power, 2012-12-04 22:27:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/alexeypa@chromium.org/11308278/4002

[commit-bot: I haz the power, 2012-12-05 01:00:37]

Change committed as 171106