Better spellcheck_test

third_party/WebKit/Source/core/testing/Internals.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 52 matching lines @@
 #include "core/dom/shadow/FlatTreeTraversal.h"
 #include "core/dom/shadow/SelectRuleFeatureSet.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/editing/Editor.h"
 #include "core/editing/PlainTextRange.h"
 #include "core/editing/SurroundingText.h"
 #include "core/editing/iterators/TextIterator.h"
 #include "core/editing/markers/DocumentMarker.h"
 #include "core/editing/markers/DocumentMarkerController.h"
 #include "core/editing/serializers/Serialization.h"
+#include "core/editing/spellcheck/IdleSpellCheckCallback.h"
 #include "core/editing/spellcheck/SpellCheckRequester.h"
 #include "core/editing/spellcheck/SpellChecker.h"
 #include "core/frame/EventHandlerRegistry.h"
 #include "core/frame/FrameConsole.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalDOMWindow.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
 #include "core/frame/VisualViewport.h"
 #include "core/html/HTMLContentElement.h"
@@ skipping 1404 matching lines @@
   if (!requester) {
     exceptionState.throwDOMException(
         InvalidAccessError,
         "No spell check requestor can be obtained for the provided document.");
     return -1;
   }
 
   return requester->lastProcessedSequence();
 }
 
+String Internals::idleTimeSpellCheckerState(Document* document,
+                                            ExceptionState& exceptionState) {
+  if (!document || !document->frame()) {
+    exceptionState.throwDOMException(
+        InvalidAccessError,
+        "No frame can be obtained from the provided document.");
+    return String();
+  }
+
+  IdleSpellCheckCallback::State state =
+      document->frame()->spellChecker().idleSpellCheckCallback().state();
+  switch (state) {
+    case IdleSpellCheckCallback::State::kInactive:

[yosin_UTC9, 2017/03/03 04:44:59]

Optional: yet another solution is having
FOR_EACH_IDLE_SPELL_CHECK_CALLBACK_STATE(V) and
utilize here and printer.

#define FOR_EACH_IDLE_SPELL_CHECK_CALLBACK_STATE(V) \
  V(Inactive) V(HotModeReqquested) ...

enum State {
#define V(name) k##name
  FOR_EACH_IDLE_SPELL_CHECK_CALLBACK_STATE(V) 
#undef V
};

std::ostream& operator<<(...) {
  const char* const kTexts[] = {
#define V(name) #name
  FOR_EACH_IDLE_SPELL_CHECK_CALLBACK_STATE(V) 
#undef V
  };
}

etc.

See [1] for usage pattern.

[1] https://cs.chromium.org/search/?q=file:webkit+FOR_EACH_&type=cs

[Xiaocheng, 2017/03/03 18:56:41]

Updated with FOR_EACH(V)

Thanks for the advice.

+      return "inactive";
+    case IdleSpellCheckCallback::State::kHotModeRequested:
+      return "hot-mode-requested";
+    case IdleSpellCheckCallback::State::kInHotModeInvocation:
+      return "in-hot-mode-invocation";
+    case IdleSpellCheckCallback::State::kColdModeTimerStarted:
+      return "cold-mode-timer-started";
+    case IdleSpellCheckCallback::State::kColdModeRequested:
+      return "cold-mode-requested";
+    case IdleSpellCheckCallback::State::kInColdModeInvocation:
+      return "in-cold-mode-invocation";
+  }
+}
+
+void Internals::runIdleTimeSpellChecker(Document* document,
+                                        ExceptionState& exceptionState) {
+  if (!document || !document->frame()) {
+    exceptionState.throwDOMException(
+        InvalidAccessError,
+        "No frame can be obtained from the provided document.");
+    return;
+  }
+
+  document->frame()
+      ->spellChecker()
+      .idleSpellCheckCallback()
+      .forceInvocationForTesting();
+}
+
 Vector<AtomicString> Internals::userPreferredLanguages() const {
   return blink::userPreferredLanguages();
 }
 
 // Optimally, the bindings generator would pass a Vector<AtomicString> here but
 // this is not supported yet.
 void Internals::setUserPreferredLanguages(const Vector<String>& languages) {
   Vector<AtomicString> atomicLanguages;
   for (size_t i = 0; i < languages.size(); ++i)
     atomicLanguages.push_back(AtomicString(languages[i]));
@@ skipping 1669 matching lines @@
 
 void Internals::crash() {
   CHECK(false) << "Intentional crash";
 }
 
 void Internals::setIsLowEndDevice(bool isLowEndDevice) {
   MemoryCoordinator::setIsLowEndDeviceForTesting(isLowEndDevice);
 }
 
 }  // namespace blink