[refactor] - Remove unused WebTextInputType from WebViewImpl.

third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp

 /*
  * Copyright (C) 2014 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 474 matching lines @@
                 focusedFrame->inputMethodController().confirmComposition();
 
                 if (autofillClient)
                     autofillClient->setIgnoreTextChanges(false);
             }
             m_imeAcceptEvents = false;
         }
     }
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 bool WebFrameWidgetImpl::setComposition(
     const WebString& text,
     const WebVector<WebCompositionUnderline>& underlines,
     int selectionStart,
     int selectionEnd)
 {
     LocalFrame* focused = focusedLocalFrameAvailableForIme();
     if (!focused)
         return false;
 
@@ skipping 33 matching lines @@
     // selectionStart and selectionEnd, WebKit somehow won't paint the selection
     // at all (see InlineTextBox::paint() function in InlineTextBox.cpp).
     // But the selection range actually takes effect.
     inputMethodController.setComposition(String(text),
         CompositionUnderlineVectorBuilder(underlines),
         selectionStart, selectionEnd);
 
     return text.isEmpty() || inputMethodController.hasComposition();
 }
 
+// TODO(ekaramad):These methods are almost duplicated in WebViewImpl as well.
+// This code needs to be refactored  (http://crbug.com/629721).
 bool WebFrameWidgetImpl::confirmComposition()
 {
     return confirmComposition(DoNotKeepSelection);
 }
 
 bool WebFrameWidgetImpl::confirmComposition(ConfirmCompositionBehavior selectionBehavior)
 {
     return confirmComposition(WebString(), selectionBehavior);
 }
 
@@ skipping 12 matching lines @@
     if (WebPlugin* plugin = focusedPluginIfInputMethodSupported(focused))
         return plugin->confirmComposition(text, selectionBehavior);
 
     // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
     // needs to be audited.  See http://crbug.com/590369 for more details.
     focused->document()->updateStyleAndLayoutIgnorePendingStylesheets();
 
     return focused->inputMethodController().confirmCompositionOrInsertText(text, selectionBehavior == KeepSelection ? InputMethodController::KeepSelection : InputMethodController::DoNotKeepSelection);
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 WebRange WebFrameWidgetImpl::compositionRange()
 {
     LocalFrame* focused = focusedLocalFrameAvailableForIme();
     if (!focused)
         return WebRange();
 
     const EphemeralRange range = focused->inputMethodController().compositionEphemeralRange();
     if (range.isNull())
         return WebRange();
 
     Element* editable = focused->selection().rootEditableElementOrDocumentElement();
     DCHECK(editable);
 
     // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
     // needs to be audited.  See http://crbug.com/590369 for more details.
     editable->document().updateStyleAndLayoutIgnorePendingStylesheets();
 
     return PlainTextRange::create(*editable, range);
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 WebTextInputInfo WebFrameWidgetImpl::textInputInfo()
 {
     WebTextInputInfo info;
 
     LocalFrame* focused = focusedLocalFrameInWidget();
     if (!focused)
         return info;
 
     FrameSelection& selection = focused->selection();
     if (!selection.isAvailable()) {
@@ skipping 41 matching lines @@
         PlainTextRange plainTextRange(PlainTextRange::create(*element, range));
         if (plainTextRange.isNotNull()) {
             info.compositionStart = plainTextRange.start();
             info.compositionEnd = plainTextRange.end();
         }
     }
 
     return info;
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 WebTextInputType WebFrameWidgetImpl::textInputType()
 {
     LocalFrame* focusedFrame = focusedLocalFrameInWidget();
     if (!focusedFrame)
         return WebTextInputTypeNone;
 
     if (!focusedFrame->selection().isAvailable()) {
         // "mouse-capture-inside-shadow.html" reaches here.
         return WebTextInputTypeNone;
     }
@@ skipping 57 matching lines @@
 WebColor WebFrameWidgetImpl::backgroundColor() const
 {
     if (isTransparent())
         return Color::transparent;
     if (!m_localRoot->frameView())
         return m_baseBackgroundColor;
     FrameView* view = m_localRoot->frameView();
     return view->documentBackgroundColor().rgb();
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 bool WebFrameWidgetImpl::selectionBounds(WebRect& anchor, WebRect& focus) const
 {
     const LocalFrame* localFrame = focusedLocalFrameInWidget();
     if (!localFrame)
         return false;
 
     FrameSelection& selection = localFrame->selection();
 
     if (selection.isCaret()) {
         // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
@@ skipping 14 matching lines @@
     IntRect scaledFocus(localFrame->view()->contentsToRootFrame(focus));
 
     anchor = scaledAnchor;
     focus = scaledFocus;
 
     if (!selection.selection().isBaseFirst())
         std::swap(anchor, focus);
     return true;
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 bool WebFrameWidgetImpl::selectionTextDirection(WebTextDirection& start, WebTextDirection& end) const
 {
     const LocalFrame* frame = focusedLocalFrameInWidget();
     if (!frame)
         return false;
 
     FrameSelection& selection = frame->selection();
     if (selection.selection().toNormalizedEphemeralRange().isNull())
         return false;
     start = toWebTextDirection(primaryDirectionOf(*selection.start().anchorNode()));
     end = toWebTextDirection(primaryDirectionOf(*selection.end().anchorNode()));
     return true;
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 bool WebFrameWidgetImpl::isSelectionAnchorFirst() const
 {
     if (const LocalFrame* frame = focusedLocalFrameInWidget())
         return frame->selection().selection().isBaseFirst();
     return false;
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 WebRange WebFrameWidgetImpl::caretOrSelectionRange()
 {
     LocalFrame* focused = focusedLocalFrameInWidget();
     if (!focused)
         return WebRange();
 
     // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
     // needs to be audited.  See http://crbug.com/590369 for more details.
     focused->document()->updateStyleAndLayoutIgnorePendingStylesheets();
 
@@ skipping 63 matching lines @@
 void WebFrameWidgetImpl::didNotAcquirePointerLock()
 {
     page()->pointerLockController().didNotAcquirePointerLock();
 }
 
 void WebFrameWidgetImpl::didLosePointerLock()
 {
     page()->pointerLockController().didLosePointerLock();
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 bool WebFrameWidgetImpl::getCompositionCharacterBounds(WebVector<WebRect>& bounds)
 {
     WebRange range = compositionRange();
     if (range.isEmpty())
         return false;
 
     LocalFrame* frame = focusedLocalFrameInWidget();
     if (!frame)
         return false;
 
     WebLocalFrameImpl* webLocalFrame = WebLocalFrameImpl::fromFrame(frame);
     size_t characterCount = range.length();
     size_t offset = range.startOffset();
     WebVector<WebRect> result(characterCount);
     WebRect webrect;
     for (size_t i = 0; i < characterCount; ++i) {
         if (!webLocalFrame->firstRectForCharacterRange(offset + i, 1, webrect)) {
             DLOG(ERROR) << "Could not retrieve character rectangle at " << i;
             return false;
         }
         result[i] = webrect;
     }
 
     bounds.swap(result);
     return true;
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
+// code needs to be refactored  (http://crbug.com/629721).
 void WebFrameWidgetImpl::applyReplacementRange(const WebRange& range)
 {
     if (LocalFrame* frame = focusedLocalFrameInWidget()) {
         // TODO(dglazkov): Going from LocalFrame to WebLocalFrameImpl seems
         // silly. What is going on here?
         WebLocalFrameImpl::fromFrame(frame)->selectRange(range);
     }
 }
 
 void WebFrameWidgetImpl::handleMouseLeave(LocalFrame& mainFrame, const WebMouseEvent& event)
@@ skipping 546 matching lines @@
 }
 
 LocalFrame* WebFrameWidgetImpl::focusedLocalFrameAvailableForIme() const
 {
     if (!m_imeAcceptEvents)
         return nullptr;
     return focusedLocalFrameInWidget();
 }
 
 } // namespace blink