HTMLTextAreaElement.setSelectionRange should not change focus.

Source/core/html/HTMLTextAreaElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2010 Apple Inc. All rights reserved.
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2007 Samuel Weinig (sam@webkit.org)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 202 matching lines @@
 
     const AtomicString& dirnameAttrValue = fastGetAttribute(dirnameAttr);
     if (!dirnameAttrValue.isNull())
         encoding.appendData(dirnameAttrValue, directionForFormData());
     return true;
 }
 
 void HTMLTextAreaElement::resetImpl()
 {
     setNonDirtyValue(defaultValue());
+    unsigned endOfString = m_value.length();
+    setSelectionRange(endOfString, endOfString, SelectionHasNoDirection, NotChangeSelection);

[tkent, 2014/07/24 01:04:22]

Why only resetImpl() has this though childrenChanged and setDefaultValue don't?

[yoichio, 2014/07/24 03:06:06]

I found also childrenChanged and setDefaultValue needs set selection to end even
if resetting a same value.
Thus I add extra setSelectionRange calling into the early return point in
setValueCommon. 

 }
 
 bool HTMLTextAreaElement::hasCustomFocusLogic() const
 {
     return true;
 }
 
 bool HTMLTextAreaElement::isKeyboardFocusable() const
 {
     // If a given text area can be focused at all, then it will always be keyboard focusable.
@@ skipping 131 matching lines @@
         return;
 
     m_value = normalizedValue;
     setInnerEditorValue(m_value);
     if (eventBehavior == DispatchNoEvent)
         setLastChangeWasNotUserEdit();
     updatePlaceholderVisibility(false);
     setNeedsStyleRecalc(SubtreeStyleChange);
     m_suggestedValue = String();
 
-    // Set the caret to the end of the text value.
-    if (document().focusedElement() == this) {
+    if (renderer()) {

[tkent, 2014/07/24 01:04:22]

Checking renderer() is not correct.  We should call setSelectionRange even if
the element has display:none.  We should check isFinishedParsingChildren().

[yoichio, 2014/07/24 03:06:07]

Done.

+        // Set the caret to the end of the text value except for initialize.
         unsigned endOfString = m_value.length();
-        setSelectionRange(endOfString, endOfString);
+        setSelectionRange(endOfString, endOfString, SelectionHasNoDirection, NotChangeSelection);
     }
 
     notifyFormStateChanged();
     if (eventBehavior == DispatchNoEvent) {
         setTextAsOfLastFormControlChangeEvent(normalizedValue);
     } else {
         if (eventBehavior == DispatchInputAndChangeEvent)
             dispatchFormControlInputEvent();
         dispatchFormControlChangeEvent();
     }
@@ skipping 168 matching lines @@
 {
     return true;
 }
 
 bool HTMLTextAreaElement::supportsAutofocus() const
 {
     return true;
 }
 
 }