Make flexibleBox visible position cadidate only where it is editable.

Source/core/dom/Position.cpp

Index: Source/core/dom/Position.cpp
diff --git a/Source/core/dom/Position.cpp b/Source/core/dom/Position.cpp
index 699caff565faebf728692d8584433879feb4f063..54c9d4fb9b6b3b77433e4047e0bcf4f5b2c66705 100644
--- a/Source/core/dom/Position.cpp
+++ b/Source/core/dom/Position.cpp
@@ -891,7 +891,7 @@ bool Position::isCandidate() const
     if (isHTMLHtmlElement(*m_anchorNode))
         return false;
 
-    if (renderer->isLayoutBlockFlow() || renderer->isFlexibleBox() || renderer->isLayoutGrid()) {
+    if (renderer->isLayoutBlockFlow() || (renderer->isFlexibleBox() && m_anchorNode->hasEditableStyle()) || renderer->isLayoutGrid()) {

[pdr., 2015/04/06 18:25:46]

I'm not an expert in this area but this fix looks too local. Why don't we need
the hasEditableStyle check for grid or layoutblockflow?

         if (toLayoutBlock(renderer)->logicalHeight() || isHTMLBodyElement(*m_anchorNode)) {
             if (!Position::hasRenderedNonAnonymousDescendantsWithHeight(renderer))
                 return atFirstEditingPositionForNode() && !Position::nodeIsUserSelectNone(deprecatedNode());