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 318 matching lines @@
 
 String HTMLTextAreaElement::value() const
 {
     updateValue();
     return m_value;
 }
 
 void HTMLTextAreaElement::setValue(const String& value, TextFieldEventBehavior eventBehavior)
 {
     RefPtrWillBeRawPtr<HTMLTextAreaElement> protector(this);
-    setValueCommon(value, eventBehavior);
+    bool isUpdated = setValueCommon(value, eventBehavior);
     m_isDirty = true;
     setNeedsValidityCheck();
+
+    if (isUpdated) {
+        // Set the caret to the end of the text value.
+        unsigned endOfString = m_value.length();
+        setSelectionRange(endOfString, endOfString);
+    }
+
     if (document().focusedElement() == this)
         document().frameHost()->chrome().client().didUpdateTextOfFocusedElementByNonUserInput();
 }
 
 void HTMLTextAreaElement::setNonDirtyValue(const String& value)
 {
     setValueCommon(value, DispatchNoEvent);
     m_isDirty = false;
     setNeedsValidityCheck();
 }
 
-void HTMLTextAreaElement::setValueCommon(const String& newValue, TextFieldEventBehavior eventBehavior)
+bool HTMLTextAreaElement::setValueCommon(const String& newValue, TextFieldEventBehavior eventBehavior)
 {
     // Code elsewhere normalizes line endings added by the user via the keyboard or pasting.
     // We normalize line endings coming from JavaScript here.
     String normalizedValue = newValue.isNull() ? "" : newValue;
     normalizedValue.replace("\r\n", "\n");
     normalizedValue.replace('\r', '\n');
 
     // Return early because we don't want to move the caret or trigger other side effects
     // when the value isn't changing. This matches Firefox behavior, at least.
     if (normalizedValue == value())
-        return;
+        return false;
 
     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) {
-        unsigned endOfString = m_value.length();
-        setSelectionRange(endOfString, endOfString);
-    }
-
     notifyFormStateChanged();
     if (eventBehavior == DispatchNoEvent) {
         setTextAsOfLastFormControlChangeEvent(normalizedValue);
     } else {
         if (eventBehavior == DispatchInputAndChangeEvent)
             dispatchFormControlInputEvent();
         dispatchFormControlChangeEvent();
     }
+    return true;
 }
 
 void HTMLTextAreaElement::setInnerEditorValue(const String& value)
 {
     HTMLTextFormControlElement::setInnerEditorValue(value);
     m_valueIsUpToDate = true;
 }
 
 String HTMLTextAreaElement::defaultValue() const
 {
@@ skipping 163 matching lines @@
 {
     return true;
 }
 
 bool HTMLTextAreaElement::supportsAutofocus() const
 {
     return true;
 }
 
 }