[New Multicolumn] Make positionForPoint() work.

Source/core/layout/MultiColumnFragmentainerGroup.h

Index: Source/core/layout/MultiColumnFragmentainerGroup.h
diff --git a/Source/core/layout/MultiColumnFragmentainerGroup.h b/Source/core/layout/MultiColumnFragmentainerGroup.h
index 5c87287e77e580bbca8bbd5062090b3b33d737f8..71f8714e7f8fd73683bab116aec6f4b1a918feb6 100644
--- a/Source/core/layout/MultiColumnFragmentainerGroup.h
+++ b/Source/core/layout/MultiColumnFragmentainerGroup.h
@@ -60,6 +60,7 @@ public:
 
     LayoutSize flowThreadTranslationAtOffset(LayoutUnit offsetInFlowThread) const;
     LayoutUnit columnLogicalTopForOffset(LayoutUnit offsetInFlowThread) const;
+    LayoutPoint visualPointToFlowThreadPoint(const LayoutPoint& visualPoint) const;
     void collectLayerFragments(LayerFragments&, const LayoutRect& layerBoundingBox, const LayoutRect& dirtyRect) const;
     LayoutRect calculateOverflow() const;
 
@@ -83,6 +84,7 @@ private:
     LayoutUnit calculateColumnHeight(BalancedColumnHeightCalculation) const;
 
     LayoutRect columnRectAt(unsigned columnIndex) const;
+    LayoutUnit logicalTopInFlowThreadAt(unsigned columnIndex) const { return m_logicalTopInFlowThread + columnIndex * m_columnHeight; }
     LayoutRect flowThreadPortionRectAt(unsigned columnIndex) const;
     LayoutRect flowThreadPortionOverflowRect(const LayoutRect& flowThreadPortion, unsigned columnIndex, unsigned columnCount, LayoutUnit columnGap) const;
 
@@ -92,6 +94,11 @@ private:
     };
     unsigned columnIndexAtOffset(LayoutUnit offsetInFlowThread, ColumnIndexCalculationMode = ClampToExistingColumns) const;
 
+    // Return the column that the specified visual point belongs to. Only the coordinate on the
+    // column progression axis is relevant. Every point belongs to a column, even if said point is
+    // not inside any of the columns.
+    unsigned columnIndexAtVisualPoint(const LayoutPoint& visualPoint) const;

[Julien - ping for review, 2015/03/09 15:49:03]

It's clunky that we need to convert the point into flow-thread coordinate and
then ask for the column index.

Do you intent to reuse this code in the future? If not, it seems like just
converting a point to a column index would do the job inside one function.

[mstensho (USE GERRIT), 2015/03/12 11:43:27]

This method doesn't convert to flow-thread coordinates. I have no plans for
other uses of this method, but doesn't it really serve a purpose on its own? I
feel that visualPointToFlowThreadPoint() is heavy enough as it is.

+
     LayoutMultiColumnSet& m_columnSet;
 
     LayoutUnit m_logicalTop;