Implement ChromiumBridge::clipboardWritePlainText()

This is an implementation of the bridge method for copying arbitrary plaintext onto a clipboard in Chromium. The method is declared upstream, in a patch associated with
https://bugs.webkit.org/show_bug.cgi?id=29634

TEST=copying works locally
BUG=https://bugs.webkit.org/show_bug.cgi?id=29634 upstream

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27176

[apavlov, 2009-09-22 16:51:27]

Please review.

[pfeldman, 2009-09-22 17:07:16]

LGTM given you rip ChromiumBridge.cpp change and run try bots on it.

http://codereview.chromium.org/211052/diff/1/7
File webkit/api/src/ChromiumBridge.cpp (right):

http://codereview.chromium.org/211052/diff/1/7#newcode111
Line 111: void ChromiumBridge::clipboardWritePlainText(const String& plainText)
could you rip this so that we could land it right now?

[darin (slow to review), 2009-09-22 18:39:27]

http://codereview.chromium.org/211052/diff/5002/5003
File webkit/api/public/WebClipboard.h (right):

http://codereview.chromium.org/211052/diff/5002/5003#newcode64
Line 64: virtual void writePlainText(const WebString& plainText) = 0;
nit: please list this above writeHTML to match the order of the read* functions.

nit: please leave the parameter unnamed since it doesn't add any information. 
that is webkit style.

http://codereview.chromium.org/211052/diff/5002/5006
File webkit/tools/test_shell/mock_webclipboard_impl.cc (right):

http://codereview.chromium.org/211052/diff/5002/5006#newcode66
Line 66: void MockWebClipboardImpl::writePlainText(const WebKit::WebString&
plainText) {
nit: plain_text

http://codereview.chromium.org/211052/diff/5002/5007
File webkit/tools/test_shell/mock_webclipboard_impl.h (right):

http://codereview.chromium.org/211052/diff/5002/5007#newcode28
Line 28: virtual void writePlainText(const WebKit::WebString& plainText);
nit: please use google style "plain_text"

http://codereview.chromium.org/211052/diff/5002/5007#newcode36
Line 36: WebKit::WebString m_plainText;
sigh..  this file is failing to use google style :(

[apavlov, 2009-09-23 11:28:17]

http://codereview.chromium.org/211052/diff/5002/5003
File webkit/api/public/WebClipboard.h (right):

http://codereview.chromium.org/211052/diff/5002/5003#newcode64
Line 64: virtual void writePlainText(const WebString& plainText) = 0;
On 2009/09/22 18:39:27, darin wrote:
> nit: please list this above writeHTML to match the order of the read*
functions.
> 
> nit: please leave the parameter unnamed since it doesn't add any information. 
> that is webkit style.

Done.

http://codereview.chromium.org/211052/diff/5002/5006
File webkit/tools/test_shell/mock_webclipboard_impl.cc (right):

http://codereview.chromium.org/211052/diff/5002/5006#newcode66
Line 66: void MockWebClipboardImpl::writePlainText(const WebKit::WebString&
plainText) {
On 2009/09/22 18:39:27, darin wrote:
> nit: plain_text

Done.

http://codereview.chromium.org/211052/diff/5002/5007
File webkit/tools/test_shell/mock_webclipboard_impl.h (right):

http://codereview.chromium.org/211052/diff/5002/5007#newcode28
Line 28: virtual void writePlainText(const WebKit::WebString& plainText);
On 2009/09/22 18:39:27, darin wrote:
> nit: please use google style "plain_text"

Done.

[darin (slow to review), 2009-09-23 16:58:36]

thanks.  as for cleaning up the rest of mock_webclipboard_impl.{h,cc}, i think
that's best left for another CL.