Do not beep on failed copy/cut.

Do not beep on failed copy/cut.

WebCore::systemBeep() is only used by WebCore::Editor and
WebCore::EditorCommand, so we silence it.

BUG=36963
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=158534

[pwnall-personal, 2013-09-25 11:58:35]

I looked into http://crbug.com/36963 and solved it according to the comments in
the bug. Can you please take a look?

I can update this CL to remove systemBeep altogether, if that makes more sense.

[jochen (gone - plz use gerrit), 2013-09-26 20:30:01]

I think it's preferable to also remove all callsites

[pwnall-personal, 2013-09-28 00:02:25]

Thank you for the quick feedback, and sorry it took a while to turn this around!

On 2013/09/26 20:30:01, jochen wrote:
> I think it's preferable to also remove all callsites

Done.

Should I also remove Sound from platform, or should I leave it there so it can
point to this bug?

[jochen (gone - plz use gerrit), 2013-09-30 10:00:42]

On 2013/09/28 00:02:25, pwnall wrote:
> Thank you for the quick feedback, and sorry it took a while to turn this
around!
> 
> On 2013/09/26 20:30:01, jochen wrote:
> > I think it's preferable to also remove all callsites
> 
> Done.
> 
> Should I also remove Sound from platform, or should I leave it there so it can
> point to this bug?

Please remove it entirely. THe bug number is already referenced from the CL
description which will be in the history.

[pwnall-personal, 2013-09-30 10:55:45]

On 2013/09/30 10:00:42, jochen wrote:
> On 2013/09/28 00:02:25, pwnall wrote:
> > Thank you for the quick feedback, and sorry it took a while to turn this
> around!
> > 
> > On 2013/09/26 20:30:01, jochen wrote:
> > > I think it's preferable to also remove all callsites
> > 
> > Done.
> > 
> > Should I also remove Sound from platform, or should I leave it there so it
can
> > point to this bug?
> 
> Please remove it entirely. THe bug number is already referenced from the CL
> description which will be in the history.

Thank you for your feedback!
Done.

[jochen (gone - plz use gerrit), 2013-09-30 11:07:27]

lgtm, thanks!

[commit-bot: I haz the power, 2013-09-30 11:54:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/costan@gmail.com/24569002/10001

[commit-bot: I haz the power, 2013-09-30 12:48:18]

Change committed as 158534