Don't clear selection on focus change except for text form control element

Source/core/page/FocusController.cpp

Index: Source/core/page/FocusController.cpp
diff --git a/Source/core/page/FocusController.cpp b/Source/core/page/FocusController.cpp
index bdf43b6cf4144a2b28f93de3553567f2e3ace5f1..761539a93665ef2825b98dabeaca881eef8313b6 100644
--- a/Source/core/page/FocusController.cpp
+++ b/Source/core/page/FocusController.cpp
@@ -677,7 +677,11 @@ static void clearSelectionIfNeeded(LocalFrame* oldFocusedFrame, LocalFrame* newF
         }
     }
 
-    selection.clear();
+    if (newFocusedNode && (newFocusedNode->hasEditableStyle()
+        || (newFocusedNode->isElementNode() && isHTMLTextFormControlElement(toElement(newFocusedNode))))) {

[leviw_travelin_and_unemployed, 2014/07/30 19:19:19]

How about a named function to document why you're checking hasEditableStyle and
isHTMLTextFormControlElement?

[yosin_UTC9, 2014/08/01 04:39:19]

Fumm... There are no good reasons to exclude them.
I should think again.

+        selection.clear();
+        return;
+    }
 }
 
 bool FocusController::setFocusedElement(Element* element, PassRefPtr<Frame> newFocusedFrame, FocusType type)