[refactor] - Remove unused WebTextInputType from WebViewImpl.

third_party/WebKit/Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 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 2292 matching lines @@
                 focusedFrame->inputMethodController().confirmComposition();
 
                 if (autofillClient)
                     autofillClient->setIgnoreTextChanges(false);
             }
             m_imeAcceptEvents = false;
         }
     }
 }
 
+
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 bool WebViewImpl::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 WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 bool WebViewImpl::confirmComposition()
 {
     return confirmComposition(DoNotKeepSelection);
 }
 
 bool WebViewImpl::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 WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 WebRange WebViewImpl::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 WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 WebTextInputInfo WebViewImpl::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 WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 WebTextInputType WebViewImpl::textInputType()
 {
     LocalFrame* focusedFrame = focusedLocalFrameInWidget();
     if (!focusedFrame)
         return WebTextInputTypeNone;
 
     if (!focusedFrame->selection().isAvailable()) {
         // "mouse-capture-inside-shadow.html" reaches here.
         return WebTextInputTypeNone;
     }
@@ skipping 23 matching lines @@
         if (type == InputTypeNames::search)
             return WebTextInputTypeSearch;
         if (type == InputTypeNames::email)
             return WebTextInputTypeEmail;
         if (type == InputTypeNames::number)
             return WebTextInputTypeNumber;
         if (type == InputTypeNames::tel)
             return WebTextInputTypeTelephone;
         if (type == InputTypeNames::url)
             return WebTextInputTypeURL;
-        if (type == InputTypeNames::date)
-            return WebTextInputTypeDate;
-        if (type == InputTypeNames::datetime_local)
-            return WebTextInputTypeDateTimeLocal;
-        if (type == InputTypeNames::month)
-            return WebTextInputTypeMonth;
-        if (type == InputTypeNames::time)
-            return WebTextInputTypeTime;
-        if (type == InputTypeNames::week)
-            return WebTextInputTypeWeek;
         if (type == InputTypeNames::text)
             return WebTextInputTypeText;
 
         return WebTextInputTypeNone;
     }
 
     if (isHTMLTextAreaElement(*element)) {
         if (toHTMLTextAreaElement(*element).isDisabledOrReadOnly())
             return WebTextInputTypeNone;
         return WebTextInputTypeTextArea;
     }
 
     if (element->isHTMLElement()) {
         if (toHTMLElement(element)->isDateTimeFieldElement())
             return WebTextInputTypeDateTimeField;
     }
 
     document->updateStyleAndLayoutTree();
     if (hasEditableStyle(*element))
         return WebTextInputTypeContentEditable;
 
     return WebTextInputTypeNone;
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 int WebViewImpl::textInputFlags()
 {
     Element* element = focusedElement();
     if (!element)
         return WebTextInputFlagNone;
 
     DEFINE_STATIC_LOCAL(AtomicString, autocompleteString, ("autocomplete"));
     DEFINE_STATIC_LOCAL(AtomicString, autocorrectString, ("autocorrect"));
     int flags = 0;
 
@@ skipping 56 matching lines @@
         return WebString();
     }
     if (isHTMLTextAreaElement(*element)) {
         const HTMLTextAreaElement& textarea = toHTMLTextAreaElement(*element);
         return textarea.fastGetAttribute(HTMLNames::inputmodeAttr).lower();
     }
 
     return WebString();
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 bool WebViewImpl::selectionBounds(WebRect& anchor, WebRect& focus) const
 {
     const Frame* frame = focusedCoreFrame();
     if (!frame || !frame->isLocalFrame())
         return false;
 
     const LocalFrame* localFrame = toLocalFrame(frame);
     if (!localFrame)
         return false;
     FrameSelection& selection = localFrame->selection();
@@ skipping 17 matching lines @@
     }
 
     anchor = localFrame->view()->contentsToViewport(anchor);
     focus = localFrame->view()->contentsToViewport(focus);
 
     if (!selection.selection().isBaseFirst())
         std::swap(anchor, focus);
     return true;
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 WebPlugin* WebViewImpl::focusedPluginIfInputMethodSupported(LocalFrame* frame)
 {
     WebPluginContainerImpl* container = WebLocalFrameImpl::currentPluginContainer(frame);
     if (container && container->supportsInputMethod())
         return container->plugin();
     return nullptr;
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 bool WebViewImpl::selectionTextDirection(WebTextDirection& start, WebTextDirection& end) const
 {
     const LocalFrame* frame = focusedLocalFrameInWidget();
     if (!frame)
         return false;
 
     const FrameSelection& selection = frame->selection();
     if (!selection.isAvailable()) {
         // plugins/mouse-capture-inside-shadow.html reaches here.
         return false;
     }
     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 WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 bool WebViewImpl::isSelectionAnchorFirst() const
 {
     const LocalFrame* frame = focusedLocalFrameInWidget();
     if (!frame)
         return false;
 
     FrameSelection& selection = frame->selection();
     if (!selection.isAvailable()) {
         // plugins/mouse-capture-inside-shadow.html reaches here.
         return false;
@@ skipping 13 matching lines @@
         return m_baseBackgroundColor;
     FrameView* view = m_page->deprecatedLocalMainFrame()->view();
     return view->documentBackgroundColor().rgb();
 }
 
 WebPagePopup* WebViewImpl::pagePopup() const
 {
     return m_pagePopup.get();
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 WebRange WebViewImpl::caretOrSelectionRange()
 {
     const 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();
 
     return focused->inputMethodController().getSelectionOffsets();
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 void WebViewImpl::setTextDirection(WebTextDirection direction)
 {
     // The Editor::setBaseWritingDirection() function checks if we can change
     // the text direction of the selected node and updates its DOM "dir"
     // attribute and its CSS "direction" property.
     // So, we just call the function as Safari does.
     const LocalFrame* focused = focusedLocalFrameInWidget();
     if (!focused)
         return;
 
@@ skipping 68 matching lines @@
     if (page())
         page()->pointerLockController().didLosePointerLock();
 }
 
 void WebViewImpl::didChangeWindowResizerRect()
 {
     if (mainFrameImpl()->frameView())
         mainFrameImpl()->frameView()->windowResizerRectChanged();
 }
 
+// TODO(ekaramad):This method is almost duplicated in WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 bool WebViewImpl::getCompositionCharacterBounds(WebVector<WebRect>& bounds)
 {
     WebRange range = compositionRange();
     if (range.isEmpty())
         return false;
 
     WebLocalFrame* frame = focusedFrame();
 
     // Only consider frames whose local root is the main frame. For other
     // local frames which have different local roots, the corresponding
     // WebFrameWidget will handle this task.
     if (frame->localRoot() != mainFrameImpl())
         return false;
 
     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 (!frame->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 WebFrameWidgetImpl as
+// well. This code needs to be refactored  (http://crbug.com/629721).
 void WebViewImpl::applyReplacementRange(const WebRange& range)
 {
     if (WebLocalFrame* frame = focusedFrame())
         frame->selectRange(range);
 }
 
 // WebView --------------------------------------------------------------------
 
 WebSettingsImpl* WebViewImpl::settingsImpl()
 {
@@ skipping 1720 matching lines @@
         return nullptr;
     return focusedFrame;
 }
 
 LocalFrame* WebViewImpl::focusedLocalFrameAvailableForIme() const
 {
     return m_imeAcceptEvents ? focusedLocalFrameInWidget() : nullptr;
 }
 
 } // namespace blink