Move ownership of IdleSpellCheckCallback from LocalFrame to SpellChecker

Move ownership of IdleSpellCheckCallback from LocalFrame to SpellChecker

IdleSpellCheckCallback is an implementation detail of Blink's SpellChecker,
and should better be owned by and hidden in SpellChecker, instead of being
exposed from LocalFrame.

This patch hence moves ownership of IdleSpellCheckCallback to SpellChecker.

BUG=517298, 695359
Review-Url: https://codereview.chromium.org/2714113003
Cr-Commit-Position: refs/heads/master@{#453157}
Committed: https://chromium.googlesource.com/chromium/src/+/d9e6a0b2197094e8fd4b4f4cb66f19bdfd213403

[Xiaocheng, 2017-02-24 18:45:51]

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

[commit-bot: I haz the power, 2017-02-24 18:46:25]

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

[Xiaocheng, 2017-02-24 18:48:30]

Description was changed from

==========
Move ownership of IdleSpellCheckCallback from LocalFrame to SpellChecker

BUG=517298, 695359
==========

to

==========
Move ownership of IdleSpellCheckCallback from LocalFrame to SpellChecker

IdleSpellCheckCallback is an implementation detail of Blink's SpellChecker,
and should better be owned by and hidden in SpellChecker, instead of being
exposed from LocalFrame.

This patch hence moves ownership of IdleSpellCheckCallback to SpellChecker.

BUG=517298, 695359
==========

[Xiaocheng, 2017-02-24 19:06:58]

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

[commit-bot: I haz the power, 2017-02-24 19:07:21]

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, 2017-02-24 23:00:00]

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

[commit-bot: I haz the power, 2017-02-24 23:00:01]

Dry run: This issue passed the CQ dry run.

[Xiaocheng, 2017-02-24 23:01:38]

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

[Xiaocheng, 2017-02-24 23:01:38]

PTAL.

This patch follows the discussion in https://codereview.chromium.org/2701983002.

[yosin_UTC9, 2017-02-25 00:52:53]

lgtm

Thanks for quick work!

[Xiaocheng, 2017-02-25 01:02:23]

The CQ bit was checked by xiaochengh@chromium.org

[Xiaocheng, 2017-02-25 01:02:31]

The CQ bit was unchecked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2017-02-25 01:02:32]

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

[Xiaocheng, 2017-02-25 01:02:36]

The CQ bit was unchecked by xiaochengh@chromium.org

[tkent, 2017-02-26 20:37:35]

lgtm

[Xiaocheng, 2017-02-27 03:24:07]

The CQ bit was checked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2017-02-27 03:24: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, 2017-02-27 05:20:52]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488165847976820,
"parent_rev": "a06e40ba77e156543163d150fb13a6064eb2e3da", "commit_rev":
"d9e6a0b2197094e8fd4b4f4cb66f19bdfd213403"}

[commit-bot: I haz the power, 2017-02-27 05:21:55]

Description was changed from

==========
Move ownership of IdleSpellCheckCallback from LocalFrame to SpellChecker

IdleSpellCheckCallback is an implementation detail of Blink's SpellChecker,
and should better be owned by and hidden in SpellChecker, instead of being
exposed from LocalFrame.

This patch hence moves ownership of IdleSpellCheckCallback to SpellChecker.

BUG=517298, 695359
==========

to

==========
Move ownership of IdleSpellCheckCallback from LocalFrame to SpellChecker

IdleSpellCheckCallback is an implementation detail of Blink's SpellChecker,
and should better be owned by and hidden in SpellChecker, instead of being
exposed from LocalFrame.

This patch hence moves ownership of IdleSpellCheckCallback to SpellChecker.

BUG=517298, 695359

Review-Url: https://codereview.chromium.org/2714113003
Cr-Commit-Position: refs/heads/master@{#453157}
Committed:
https://chromium.googlesource.com/chromium/src/+/d9e6a0b2197094e8fd4b4f4cb66f...
==========

[commit-bot: I haz the power, 2017-02-27 05:21:56]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/d9e6a0b2197094e8fd4b4f4cb66f...