RenderFlexibleBox should be treated as a candidate for visible position.

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 e4d56f2eef5cf9056e96b40c1c478e6fed7ff663..eb069bf693e1814eb09806a0a67b6ac149b1bd91 100644
--- a/Source/core/dom/Position.cpp
+++ b/Source/core/dom/Position.cpp
@@ -855,7 +855,7 @@ bool Position::isCandidate() const
     if (isHTMLHtmlElement(*m_anchorNode))
         return false;
 
-    if (renderer->isRenderBlockFlow()) {
+    if (renderer->isRenderBlockFlow() || renderer->isFlexibleBox()) {

[Manuel Rego, 2015/02/10 22:31:37]

I think this is failing for grid too. Could you fix it?

I think you can use here !renderer->isRenderBlockFlow() to cover both cases.

Thanks!

[Manuel Rego, 2015/02/10 22:43:24]

Mmmm, I've just realized that this won't make any sense. I'm wondering if we
should just add isRenderGrid or if this "if" is actually needed for something
else.

Anyway, this is failing in grid too.

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