Fixing threading issues in remoting::DesktopProcess. The UI thread is now owned by the caller of re…

Fixing threading issues in remoting::DesktopProcess. The UI thread is now owned by the caller of remoting::DesktopThread so it can be properly shared between the caller and the class.

BUG=157808, 157886
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=164243

[alexeypa (please no reviews), 2012-10-25 18:17:31]

PTAL.

[Sergey Ulanov, 2012-10-25 20:29:31]

LGTM once my comment is addressed.

http://codereview.chromium.org/11272036/diff/1/remoting/host/desktop_process_...
File remoting/host/desktop_process_main.cc (right):

http://codereview.chromium.org/11272036/diff/1/remoting/host/desktop_process_...
remoting/host/desktop_process_main.cc:89: FROM_HERE,
MessageLoop::QuitClosure());
this should be RunLoop::QuitClosure() created by the RunLoop instance you use
below to run the loop.

[alexeypa (please no reviews), 2012-10-25 20:36:21]

Thanks.

http://codereview.chromium.org/11272036/diff/1/remoting/host/desktop_process_...
File remoting/host/desktop_process_main.cc (right):

http://codereview.chromium.org/11272036/diff/1/remoting/host/desktop_process_...
remoting/host/desktop_process_main.cc:89: FROM_HERE,
MessageLoop::QuitClosure());
On 2012/10/25 20:29:32, sergeyu wrote:
> this should be RunLoop::QuitClosure() created by the RunLoop instance you use
> below to run the loop.

Done.

[commit-bot: I haz the power, 2012-10-25 20:37:56]

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