Revert of Implement complete lifecycle transition for IdleSpellCheckCallback

Revert of Implement complete lifecycle transition for IdleSpellCheckCallback (patchset #7 id:120001 of https://codereview.chromium.org/2701983002/ )

Reason for revert:
It looks like this causes compile errors (e.g. https://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac%20Builder&number=62075)

Original issue's description:
> Implement complete lifecycle transition for IdleSpellCheckCallback
> 
> This patch implements complete lifecycle state transition for
> IdleSpellCheckCallback. With this patch, idle spell check callback
> will be invoked at correct timings. The only thing remaining is to
> fill in the member functions which actually perform checking when
> the callback is invoked.
> 
> BUG=517298
> TEST=webkit_unit_tests --gtest_filter=IdleSpellCheckCallbackTest.*
> 
> Review-Url: https://codereview.chromium.org/2701983002
> Cr-Commit-Position: refs/heads/master@{#453159}
> Committed: https://chromium.googlesource.com/chromium/src/+/234937696e399887e0acef2ad037c35febca001c

TBR=tkent@chromium.org,yosin@chromium.org,xiaochengh@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=517298

[hajimehoshi, 2017-02-27 05:56:33]

The CQ bit was checked by hajimehoshi@chromium.org

[hajimehoshi, 2017-02-27 05:56:33]

Created Revert of Implement complete lifecycle transition for
IdleSpellCheckCallback

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

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:58:11]

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

[commit-bot: I haz the power, 2017-02-27 05:58:12]

Failed to apply patch for third_party/WebKit/Source/core/editing/BUILD.gn:
While running git apply --index -p1;
  error: patch failed: third_party/WebKit/Source/core/editing/BUILD.gn:262
  error: third_party/WebKit/Source/core/editing/BUILD.gn: patch does not apply

Patch:       third_party/WebKit/Source/core/editing/BUILD.gn
Index: third_party/WebKit/Source/core/editing/BUILD.gn
diff --git a/third_party/WebKit/Source/core/editing/BUILD.gn
b/third_party/WebKit/Source/core/editing/BUILD.gn
index
54b72303775ad5b960e771481ebceaa3c860c5aa..cbb366f7ac72e6e1aa33d717f65260b83046e887
100644
--- a/third_party/WebKit/Source/core/editing/BUILD.gn
+++ b/third_party/WebKit/Source/core/editing/BUILD.gn
@@ -262,7 +262,6 @@
     "iterators/TextSearcherICUTest.cpp",
     "markers/DocumentMarkerControllerTest.cpp",
     "serializers/StyledMarkupSerializerTest.cpp",
-    "spellcheck/IdleSpellCheckCallbackTest.cpp",
     "spellcheck/SpellCheckTestBase.cpp",
     "spellcheck/SpellCheckTestBase.h",
     "spellcheck/SpellCheckerTest.cpp",

[Xiaocheng, 2017-02-27 05:59:21]

Already reverted as 453161

[hajimehoshi, 2017-02-27 06:00:43]

Thanks

[hajimehoshi, 2017-02-27 06:03:45]

Description was changed from

==========
Revert of Implement complete lifecycle transition for IdleSpellCheckCallback
(patchset #7 id:120001 of https://codereview.chromium.org/2701983002/ )

Reason for revert:
It looks like this causes compile errors (e.g.
https://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac%20Builder&n...)

Original issue's description:
> Implement complete lifecycle transition for IdleSpellCheckCallback
> 
> This patch implements complete lifecycle state transition for
> IdleSpellCheckCallback. With this patch, idle spell check callback
> will be invoked at correct timings. The only thing remaining is to
> fill in the member functions which actually perform checking when
> the callback is invoked.
> 
> BUG=517298
> TEST=webkit_unit_tests --gtest_filter=IdleSpellCheckCallbackTest.*
> 
> Review-Url: https://codereview.chromium.org/2701983002
> Cr-Commit-Position: refs/heads/master@{#453159}
> Committed:
https://chromium.googlesource.com/chromium/src/+/234937696e399887e0acef2ad037...

TBR=tkent@chromium.org,yosin@chromium.org,xiaochengh@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=517298
==========

to

==========
Revert of Implement complete lifecycle transition for IdleSpellCheckCallback
(patchset #7 id:120001 of https://codereview.chromium.org/2701983002/ )

Reason for revert:
It looks like this causes compile errors (e.g.
https://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac%20Builder&n...)

Original issue's description:
> Implement complete lifecycle transition for IdleSpellCheckCallback
> 
> This patch implements complete lifecycle state transition for
> IdleSpellCheckCallback. With this patch, idle spell check callback
> will be invoked at correct timings. The only thing remaining is to
> fill in the member functions which actually perform checking when
> the callback is invoked.
> 
> BUG=517298
> TEST=webkit_unit_tests --gtest_filter=IdleSpellCheckCallbackTest.*
> 
> Review-Url: https://codereview.chromium.org/2701983002
> Cr-Commit-Position: refs/heads/master@{#453159}
> Committed:
https://chromium.googlesource.com/chromium/src/+/234937696e399887e0acef2ad037...

TBR=tkent@chromium.org,yosin@chromium.org,xiaochengh@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=517298
==========