Hide SpellCheckRequest from other classes in editing/spellcheck

third_party/WebKit/Source/core/editing/spellcheck/SpellCheckRequester.cpp

Index: third_party/WebKit/Source/core/editing/spellcheck/SpellCheckRequester.cpp
diff --git a/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckRequester.cpp b/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckRequester.cpp
index 2f92d767d3981f5eda9c7a6451c97e8c632d0c57..fe5cf138485f8478fd05c836ff49b9639c678e18 100644
--- a/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckRequester.cpp
+++ b/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckRequester.cpp
@@ -143,7 +143,13 @@ void SpellCheckRequester::timerFiredToProcessQueuedRequest(TimerBase*) {
   invokeRequest(m_requestQueue.takeFirst());
 }
 
-void SpellCheckRequester::requestCheckingFor(SpellCheckRequest* request) {
+void SpellCheckRequester::requestCheckingFor(const EphemeralRange& range) {
+  requestCheckingFor(range, 0);
+}
+
+void SpellCheckRequester::requestCheckingFor(const EphemeralRange& range,
+                                             int requestNum) {
+  SpellCheckRequest* request = SpellCheckRequest::create(range, requestNum);

[yosin_UTC9, 2017/03/09 01:28:15]

nit: s/SpellCheckRequest*/SpellCheckRequest* const/

Please add |DCHECK(!range.isNull())|.

[Xiaocheng, 2017/03/10 23:13:22]

SpellCheckRequest::ctor checks if the range is null.

Not sure if the current callers pass null ranges, and don't want to change the
code now...

   if (!request)
     return;