Replace the misspelled range within a selection to correct spelling.

Replace the misspelled range within a selection to correct spelling.

This CL changes the spell check correction to replace the misspelled range
within a selection instead of replacing the whole selection. This change enables
spell check of words selected with surrounding whitespace or punctuation. Such
selection happens on a double-click of a word with a trailing underscore in
Windows, for example. This CL requires the change in https://webkit.org/b/108513.

BUG=167485, 169659
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181968

[please use gerrit instead, 2013-02-11 21:18:27]

Groby: PTAL. This CL switches misspelling replacement from frame->replaceRange()
to frame->replaceMisspelledRange().

[please use gerrit instead, 2013-02-11 21:25:19]

John: PTAL. I am asking for your review because I am adding a new interface to
content/public/render_widget_host.h: replaceMisspelledRange(). Chrome is going
to use this method instead of replaceRange() for correcting spelling mistakes.

[please use gerrit instead, 2013-02-11 21:29:00]

Thomas: PTAL. I am adding a new IPC message ViewMsg_ReplaceMisspelling, which
Chrome is going to use instead of IPC message ViewMsg_Replace to fix spelling
mistakes.

[groby-ooo-7-16, 2013-02-11 21:30:50]

Spelling LGTM. 

Wondering if we should instead use a single message with an enum indicating what
we replace. I'd assume the security peeps might want to weigh in on that
question.

[Tom Sepez, 2013-02-11 21:36:37]

Messages LGTM.

> Wondering if we should instead use a single message with an enum indicating
what
> we replace. I'd assume the security peeps might want to weigh in on that
> question.
Either way is ok, where you get into trouble is when you don't check the enum
value coming across and then use it to index some array ...

[please use gerrit instead, 2013-02-11 21:47:45]

John: Would you mind reviewing all of the changes in this CL?

[jam, 2013-02-11 22:11:00]

On 2013/02/11 21:47:45, Rouslan Solomakhin wrote:
> John: Would you mind reviewing all of the changes in this CL?

lgtm

[please use gerrit instead, 2013-02-11 22:30:03]

Will submit to CQ tomorrow after webkit dependencies have been rolled past 
http://svn.webkit.org/repository/webkit/trunk@142494.

[commit-bot: I haz the power, 2013-02-12 16:37:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rouslan@chromium.org/12038037/2001

[commit-bot: I haz the power, 2013-02-12 17:47:13]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-02-12 18:04:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rouslan@chromium.org/12038037/2001

[commit-bot: I haz the power, 2013-02-12 19:16:53]

Change committed as 181968