Remove remoting::Clipboard::Stop().

Remove remoting::Clipboard::Stop().

Clipboard is single-threaded, so Stop() is not useful there.

Committed: https://crrev.com/45f92d83826113bb931988eb53c23d94cb4f152c
Cr-Commit-Position: refs/heads/master@{#316952}

[Sergey Ulanov, 2015-02-17 20:06:14]

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

[Lambros, 2015-02-17 21:37:33]

lgtm

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_win.cc
File remoting/host/clipboard_win.cc (right):

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_win....
remoting/host/clipboard_win.cc:110: ~Clipboard() override;
~ClipboardWin() ?
I didn't know C++ lets you use the base class name for the dtor :)

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_win....
remoting/host/clipboard_win.cc:147: window_.reset();
Don't need this line?

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_x11.cc
File remoting/host/clipboard_x11.cc (right):

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_x11....
remoting/host/clipboard_x11.cc:58: client_clipboard_.reset();
Don't need?

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_x11....
remoting/host/clipboard_x11.cc:63: display_ = nullptr;
Don't need?

[Sergey Ulanov, 2015-02-17 23:51:37]

New patchsets have been uploaded after l-g-t-m from lambroslambrou@chromium.org

[Sergey Ulanov, 2015-02-17 23:51:42]

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_win.cc
File remoting/host/clipboard_win.cc (right):

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_win....
remoting/host/clipboard_win.cc:110: ~Clipboard() override;
On 2015/02/17 21:37:33, Lambros wrote:
> ~ClipboardWin() ?
> I didn't know C++ lets you use the base class name for the dtor :)

Done.

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_win....
remoting/host/clipboard_win.cc:147: window_.reset();
On 2015/02/17 21:37:33, Lambros wrote:
> Don't need this line?

Done.

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_x11.cc
File remoting/host/clipboard_x11.cc (right):

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_x11....
remoting/host/clipboard_x11.cc:58: client_clipboard_.reset();
On 2015/02/17 21:37:33, Lambros wrote:
> Don't need?

Done.

https://codereview.chromium.org/930403002/diff/1/remoting/host/clipboard_x11....
remoting/host/clipboard_x11.cc:63: display_ = nullptr;
On 2015/02/17 21:37:33, Lambros wrote:
> Don't need?

Done.

[Sergey Ulanov, 2015-02-18 19:06:36]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2015-02-18 19:07:33]

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

[commit-bot: I haz the power, 2015-02-18 19:36:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-18 19:36:26]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Sergey Ulanov, 2015-02-19 00:52:28]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2015-02-19 00:54:06]

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

[commit-bot: I haz the power, 2015-02-19 01:24:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-19 01:25:13]

Patchset 3 (id:??) landed as
https://crrev.com/45f92d83826113bb931988eb53c23d94cb4f152c
Cr-Commit-Position: refs/heads/master@{#316952}