Chromoting's ContinueWindow implementation for Windows.

Chromoting's ContinueWindow implementation for Windows.

BUG=none
TEST=none
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=90124

[Jamie, 2011-06-21 23:09:45]

http://codereview.chromium.org/7219002/diff/5003/remoting/host/continue_windo...
File remoting/host/continue_window_win.cc (right):

http://codereview.chromium.org/7219002/diff/5003/remoting/host/continue_windo...
remoting/host/continue_window_win.cc:49: DWORD key_;
What is this designed to protect against? EndDialog is called in response to
WM_COMMAND anyway, so the dialog can be dismissed that way. Of course, that also
reactivates the session; is |key_| intended to prevent someone hiding the dialog
and preventing the user from using it to continue the session? If so, then it
seems like there are easier ways of doing that than sending WM_USER.

http://codereview.chromium.org/7219002/diff/5003/remoting/host/continue_windo...
remoting/host/continue_window_win.cc:92: case WM_APP:
I think WM_USER would be more appropriate here.

[garykac, 2011-06-22 00:36:59]

http://codereview.chromium.org/7219002/diff/5003/remoting/host/continue_windo...
File remoting/host/continue_window_win.cc (right):

http://codereview.chromium.org/7219002/diff/5003/remoting/host/continue_windo...
remoting/host/continue_window_win.cc:49: DWORD key_;
On 2011/06/21 23:09:45, Jamie wrote:
> What is this designed to protect against? EndDialog is called in response to
> WM_COMMAND anyway, so the dialog can be dismissed that way. Of course, that
also
> reactivates the session; is |key_| intended to prevent someone hiding the
dialog
> and preventing the user from using it to continue the session? If so, then it
> seems like there are easier ways of doing that than sending WM_USER.

Removed. Now that we're on the proper thread, I'm not seeing the problems that
led me to introduce this.

http://codereview.chromium.org/7219002/diff/5003/remoting/host/continue_windo...
remoting/host/continue_window_win.cc:92: case WM_APP:
On 2011/06/21 23:09:45, Jamie wrote:
> I think WM_USER would be more appropriate here.

Removed.

[Jamie, 2011-06-22 00:39:08]

LGTM.