focus() behaviour differs depending on how value is set

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 222 matching lines @@
     return isFocusable();
 }
 
 bool HTMLTextAreaElement::shouldShowFocusRingOnMouseFocus() const
 {
     return true;
 }
 
 void HTMLTextAreaElement::updateFocusAppearance(bool restorePreviousSelection)
 {
-    if (!restorePreviousSelection || !hasCachedSelection()) {
+    if (!restorePreviousSelection) {
         // If this is the first focus, set a caret at the beginning of the text.

[tkent, 2014/04/10 01:37:39]

This comment is obsolete.  Please remove it.

[harpreet.sk, 2014/04/10 06:13:27]

Done.

         // This matches some browsers' behavior; see bug 11746 Comment #15.
         // http://bugs.webkit.org/show_bug.cgi?id=11746#c15
         setSelectionRange(0, 0);
     } else
         restoreCachedSelection();
 
     if (document().frame())
         document().frame()->selection().revealSelection();
 }
 
@@ skipping 309 matching lines @@
 {
     return true;
 }
 
 bool HTMLTextAreaElement::supportsAutofocus() const
 {
     return true;
 }
 
 }