ALL-IN-ONE Stop SpellChecker from checking or marking grammar

Stop SpellChecker from checking or marking grammar

In this patch, all member functions of SpellChecker no longer search
for or mark grammar mistakes. The following refactoring are also done
for code simplification:

1. A private function of SpellChecker no longer takes grammar-related
parameters or has the word "grammar" in their names. It is completely
removed if it only performs grammar checking.

2. If a wrapper function is made trivial after step 1, it absorbs the
body of the wrapped function. This step applies to markMisspellings()
and markMisspellingsAndBadGrammar().

3. resolveTextCheckingTypeMask() is removed because it is not used any
more.

WebFrameTest is also modified by removing everything related to grammar
checking or marking.

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

[Xiaocheng, 2016-08-01 05:57:23]

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

[commit-bot: I haz the power, 2016-08-01 05:57:31]

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-01 06:07:30]

Description was changed from

==========
Stop SpellChecking from checking or marking grammar mistakes

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

to

==========
Stop SpellChecking from checking or marking grammar

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

[Xiaocheng, 2016-08-01 06:28:30]

Description was changed from

==========
Stop SpellChecking from checking or marking grammar

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

to

==========
Stop SpellChecker from checking or marking grammar

In this patch, all member functions of SpellChecker no longer search
for or mark grammar mistakes. The following refactoring are also done
for code simplification:

1. A private function of SpellChecker no longer takes grammar-related
parameters or has the word "grammar" in their names. It is completely
removed if it only performs grammar checking.

2. If a wrapper function is made trivial after step 1, it absorbs the
body of the wrapped function. This step applies to markMisspellings()
and markMisspellingsAndBadGrammar().

3. resolveTextCheckingTypeMask() is removed because it is not used any
more.

WebFrameTest is also modified by removing everything related to grammar
checking or marking.

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

[Xiaocheng, 2016-08-01 06:28:30]

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

[commit-bot: I haz the power, 2016-08-01 07:00:23]

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

[commit-bot: I haz the power, 2016-08-01 07:00:23]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Xiaocheng, 2016-08-01 08:44:31]

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

[Xiaocheng, 2016-08-01 10:36:50]

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

[commit-bot: I haz the power, 2016-08-01 10:36:57]

Dry run: 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-01 11:53:55]

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

[commit-bot: I haz the power, 2016-08-01 11:53:56]

Dry run: This issue passed the CQ dry run.