Code cleanup related to unified text checker

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

 /*
  * Copyright (C) 2006, 2007 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * 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.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 214 matching lines @@
     : m_client(&client)
     , m_start(start)
     , m_end(end)
 {
 }
 
 TextCheckingHelper::~TextCheckingHelper()
 {
 }
 
-String TextCheckingHelper::findFirstMisspelling(int& firstMisspellingOffset, bool markAll)
-{
-    // TODO(dglazkov): The use of updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
-    // see http://crbug.com/590369 for more details.
-    m_start.document()->updateStyleAndLayoutIgnorePendingStylesheets();
-
-    WordAwareIterator it(m_start, m_end);
-    firstMisspellingOffset = 0;
-
-    String firstMisspelling;
-    int currentChunkOffset = 0;
-
-    while (!it.atEnd()) {
-        int length = it.length();
-
-        // Skip some work for one-space-char hunks
-        if (!(length == 1 && it.characterAt(0) == ' ')) {
-
-            int misspellingLocation = -1;
-            int misspellingLength = 0;
-            m_client->textChecker().checkSpellingOfString(it.substring(0, length), &misspellingLocation, &misspellingLength);
-
-            // 5490627 shows that there was some code path here where the String constructor below crashes.
-            // We don't know exactly what combination of bad input caused this, so we're making this much
-            // more robust against bad input on release builds.
-            DCHECK_GE(misspellingLength, 0);
-            DCHECK_GE(misspellingLocation, -1);
-            DCHECK(!misspellingLength || misspellingLocation >= 0);
-            DCHECK_LT(misspellingLocation, length);
-            DCHECK_LE(misspellingLength, length);
-            DCHECK_LE(misspellingLocation + misspellingLength, length);
-
-            if (misspellingLocation >= 0 && misspellingLength > 0 && misspellingLocation < length && misspellingLength <= length && misspellingLocation + misspellingLength <= length) {
-
-                // Compute range of misspelled word
-                const EphemeralRange misspellingRange = calculateCharacterSubrange(EphemeralRange(m_start, m_end), currentChunkOffset + misspellingLocation, misspellingLength);
-
-                // Remember first-encountered misspelling and its offset.
-                if (!firstMisspelling) {
-                    firstMisspellingOffset = currentChunkOffset + misspellingLocation;
-                    firstMisspelling = it.substring(misspellingLocation, misspellingLength);
-                }
-
-                // Store marker for misspelled word.
-                misspellingRange.document().markers().addMarker(misspellingRange.startPosition(), misspellingRange.endPosition(), DocumentMarker::Spelling);
-
-                // Bail out if we're marking only the first misspelling, and not all instances.
-                if (!markAll)
-                    break;
-            }
-        }
-
-        currentChunkOffset += length;
-        it.advance();
-    }
-
-    return firstMisspelling;
-}
-
 String TextCheckingHelper::findFirstMisspellingOrBadGrammar(int& outFirstFoundOffset)
 {
-    if (!unifiedTextCheckerEnabled())
-        return "";
-
     String firstFoundItem;
     String misspelledWord;
 
     // Initialize out parameter; it will be updated if we find something to return.
     outFirstFoundOffset = 0;
 
     // Expand the search range to encompass entire paragraphs, since text checking needs that much context.
     // Determine the character offset from the start of the paragraph to the start of the original search range,
     // since we will want to ignore results in this area.
     Position paragraphStart = startOfParagraph(createVisiblePosition(m_start)).toParentAnchoredPosition();
@@ skipping 51 matching lines @@
             break;
         VisiblePosition newParagraphStart = startOfNextParagraph(createVisiblePosition(paragraphEnd));
         paragraphStart = newParagraphStart.toParentAnchoredPosition();
         paragraphEnd = endOfParagraph(newParagraphStart).toParentAnchoredPosition();
         firstIteration = false;
         totalLengthProcessed += currentLength;
     }
     return firstFoundItem;
 }
 
-bool TextCheckingHelper::markAllMisspellings()
-{
-    // Use the "markAll" feature of findFirstMisspelling. Ignore the return value and the "out parameter";
-    // all we need to do is mark every instance.
-    int ignoredOffset;
-    return findFirstMisspelling(ignoredOffset, true).isEmpty();
-}
-
-bool TextCheckingHelper::unifiedTextCheckerEnabled() const
-{
-    DCHECK(m_start.isNotNull());
-    Document& doc = m_start.computeContainerNode()->document();
-    return blink::unifiedTextCheckerEnabled(doc.frame());
-}
-
-bool unifiedTextCheckerEnabled(const LocalFrame* frame)
-{
-    if (!frame)
-        return false;
-
-    const Settings* settings = frame->settings();
-    if (!settings)
-        return false;
-
-    return settings->unifiedTextCheckerEnabled();
-}
-
 } // namespace blink