Cleanup function names in SpellChecker and TextCheckingHelper

Cleanup function names in SpellChecker and TextCheckingHelper

The above classes no longer performs any grammar checking. This patch
cleans up their member function names so that one can be clear that the
two classes only performs spell checking.

BUG=619452
TEST=n/a; no behavior change

Committed: https://crrev.com/a10daeeca303db977aa23218f2ac71abbb2eaa2e
Cr-Commit-Position: refs/heads/master@{#409168}

[Xiaocheng, 2016-08-02 09:09:52]

The CQ bit was checked by xiaochengh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-02 09:10:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xiaocheng, 2016-08-02 09:22:54]

xiaochengh@chromium.org changed reviewers:
+ yosin@chromium.org

[Xiaocheng, 2016-08-02 09:22:54]

PTAL.

https://codereview.chromium.org/2205823002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h (right):

https://codereview.chromium.org/2205823002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h:104: void
markMisspellingsWithTextCheckingHelper(const VisibleSelection&);
I wish I could come up with a better name but didn't.

What's the different between this function and the public |markMisspellings|
(originally |markMisspellingsAndBadGrammar|)?

[Xiaocheng, 2016-08-02 09:23:33]

Description was changed from

==========
Cleanup function names in SpellChecker and TextCheckingHelper

The above classes no longer performs any grammar checking. This patch
cleans up their member function names so that one can be clear that the
two classes only performs spell checking.

BUG=619452
TEST=n/a; no visible change
==========

to

==========
Cleanup function names in SpellChecker and TextCheckingHelper

The above classes no longer performs any grammar checking. This patch
cleans up their member function names so that one can be clear that the
two classes only performs spell checking.

BUG=619452
TEST=n/a; no behavior change
==========

[yosin_UTC9, 2016-08-02 09:34:01]

lgtm

https://codereview.chromium.org/2205823002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h (right):

https://codereview.chromium.org/2205823002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h:104: void
markMisspellingsWithTextCheckingHelper(const VisibleSelection&);
On 2016/08/02 at 09:22:54, Xiaocheng wrote:
> I wish I could come up with a better name but didn't.
> 
> What's the different between this function and the public |markMisspellings|
(originally |markMisspellingsAndBadGrammar|)?
Once we remove all grammar checking code, we might have better idea about
naming.

[commit-bot: I haz the power, 2016-08-02 10:33:58]

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

[commit-bot: I haz the power, 2016-08-02 10:33:59]

Dry run: This issue passed the CQ dry run.

[Xiaocheng, 2016-08-02 10:50:25]

The CQ bit was checked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2016-08-02 10:50:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-02 10:53:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-02 10:55:51]

Description was changed from

==========
Cleanup function names in SpellChecker and TextCheckingHelper

The above classes no longer performs any grammar checking. This patch
cleans up their member function names so that one can be clear that the
two classes only performs spell checking.

BUG=619452
TEST=n/a; no behavior change
==========

to

==========
Cleanup function names in SpellChecker and TextCheckingHelper

The above classes no longer performs any grammar checking. This patch
cleans up their member function names so that one can be clear that the
two classes only performs spell checking.

BUG=619452
TEST=n/a; no behavior change

Committed: https://crrev.com/a10daeeca303db977aa23218f2ac71abbb2eaa2e
Cr-Commit-Position: refs/heads/master@{#409168}
==========

[commit-bot: I haz the power, 2016-08-02 10:55:52]

Patchset 1 (id:??) landed as
https://crrev.com/a10daeeca303db977aa23218f2ac71abbb2eaa2e
Cr-Commit-Position: refs/heads/master@{#409168}