Add a skeleton of idle time spell checker

Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet. Follow-up patches will do that.

Design doc: https://docs.google.com/document/d/1vo71cnlGOqOuXyy1zjzXSlUIjG7GsWgRjqmkrq1g5hk/edit?usp=sharing

BUG=517298
Committed: https://crrev.com/71334324b1fce43df19e2941f43ea44f266600f0
Cr-Commit-Position: refs/heads/master@{#438492}

[Xiaocheng, 2016-12-14 08:07:29]

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

[commit-bot: I haz the power, 2016-12-14 08:07:49]

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

[Xiaocheng, 2016-12-14 08:20:45]

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

[commit-bot: I haz the power, 2016-12-14 08:20:57]

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

[Xiaocheng, 2016-12-14 08:25:11]

Description was changed from

==========
Add a skeleton of idle time spell checker

BUG=
==========

to

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet.

BUG=517298
==========

[Xiaocheng, 2016-12-14 08:38:53]

Description was changed from

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet.

BUG=517298
==========

to

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet.

Design doc:
https://docs.google.com/document/d/1vo71cnlGOqOuXyy1zjzXSlUIjG7GsWgRjqmkrq1g5...

BUG=517298
==========

[Xiaocheng, 2016-12-14 08:40:06]

Description was changed from

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet.

Design doc:
https://docs.google.com/document/d/1vo71cnlGOqOuXyy1zjzXSlUIjG7GsWgRjqmkrq1g5...

BUG=517298
==========

to

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet. Follow-up patches will do that.

Design doc:
https://docs.google.com/document/d/1vo71cnlGOqOuXyy1zjzXSlUIjG7GsWgRjqmkrq1g5...

BUG=517298
==========

[Xiaocheng, 2016-12-14 08:40:16]

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

[Xiaocheng, 2016-12-14 08:40:16]

PTAL.

[tkent, 2016-12-14 08:45:50]

lgtm

https://codereview.chromium.org/2578493002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/editing/spellcheck/IdleSpellCheckCallback.h
(right):

https://codereview.chromium.org/2578493002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/editing/spellcheck/IdleSpellCheckCallback.h:45:
Inactive,
Please prepend 'k' to each items.

Reference: [names-enum-members] in
https://www.chromium.org/blink/coding-style#TOC-Names

[Xiaocheng, 2016-12-14 09:08:12]

The CQ bit was checked by xiaochengh@chromium.org

[Xiaocheng, 2016-12-14 09:08:13]

Thanks for the review.

Committing with the enum class members prefixed with 'k'.

https://codereview.chromium.org/2578493002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/editing/spellcheck/IdleSpellCheckCallback.h
(right):

https://codereview.chromium.org/2578493002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/editing/spellcheck/IdleSpellCheckCallback.h:45:
Inactive,
On 2016/12/14 at 08:45:50, tkent wrote:
> Please prepend 'k' to each items.
> 
> Reference: [names-enum-members] in
https://www.chromium.org/blink/coding-style#TOC-Names

Done.

[Xiaocheng, 2016-12-14 09:08:13]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2578493002/#ps60001
(title: "Use kCamelCase")

[commit-bot: I haz the power, 2016-12-14 09:08:21]

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-12-14 12:55:05]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1481706492273820,
"parent_rev": "5fff160b76fb8a6825e0ad7afa9a8692cfe8cc4b", "commit_rev":
"17e32c61dd61651ba101c6e13fbcd1cc135e3bf9"}

[commit-bot: I haz the power, 2016-12-14 12:55:30]

Description was changed from

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet. Follow-up patches will do that.

Design doc:
https://docs.google.com/document/d/1vo71cnlGOqOuXyy1zjzXSlUIjG7GsWgRjqmkrq1g5...

BUG=517298
==========

to

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet. Follow-up patches will do that.

Design doc:
https://docs.google.com/document/d/1vo71cnlGOqOuXyy1zjzXSlUIjG7GsWgRjqmkrq1g5...

BUG=517298

Review-Url: https://codereview.chromium.org/2578493002
==========

[commit-bot: I haz the power, 2016-12-14 12:55:31]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-12-14 12:57:59]

Description was changed from

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet. Follow-up patches will do that.

Design doc:
https://docs.google.com/document/d/1vo71cnlGOqOuXyy1zjzXSlUIjG7GsWgRjqmkrq1g5...

BUG=517298

Review-Url: https://codereview.chromium.org/2578493002
==========

to

==========
Add a skeleton of idle time spell checker

This patch introduces a new class IdleSpellCheckCallback for implementing
most of the idle time spell checker. The new class currently has only a
skeleton implementing the state machine as specified in the design doc.

It does not contain any real spell checking logic, and is not exposed to
any other class, yet. Follow-up patches will do that.

Design doc:
https://docs.google.com/document/d/1vo71cnlGOqOuXyy1zjzXSlUIjG7GsWgRjqmkrq1g5...

BUG=517298

Committed: https://crrev.com/71334324b1fce43df19e2941f43ea44f266600f0
Cr-Commit-Position: refs/heads/master@{#438492}
==========

[commit-bot: I haz the power, 2016-12-14 12:58:00]

Patchset 4 (id:??) landed as
https://crrev.com/71334324b1fce43df19e2941f43ea44f266600f0
Cr-Commit-Position: refs/heads/master@{#438492}