Add scrollbar logical width while computing intrinsic logical width

Source/core/rendering/RenderGrid.cpp

Index: Source/core/rendering/RenderGrid.cpp
diff --git a/Source/core/rendering/RenderGrid.cpp b/Source/core/rendering/RenderGrid.cpp
index e7daaecd24e539d34fa4095dad4fb805159dccc8..e81c58ec5ea6bd9bb8212ba5236df45f6984c965 100644
--- a/Source/core/rendering/RenderGrid.cpp
+++ b/Source/core/rendering/RenderGrid.cpp
@@ -376,7 +376,11 @@ void RenderGrid::computeIntrinsicLogicalWidths(LayoutUnit& minLogicalWidth, Layo
         minLogicalWidth += minTrackBreadth;
         maxLogicalWidth += maxTrackBreadth;
 
-        // FIXME: This should add in the scrollbarWidth (e.g. see RenderFlexibleBox).
+        maxLogicalWidth = std::max(minLogicalWidth, maxLogicalWidth);

[Julien - ping for review, 2014/09/09 14:39:40]

Is this max required as we ensure that minTrackBreadth is less than
maxTrackBreadth above?

[Sunil Ratnu, 2014/09/09 15:22:55]

In this case, its not needed as before calling computeIntrinsicLogicalWidths(),
we have
minLogicalWidth = maxLogicalWidth = 0; But I was unsure that this will always
remain equal or 0 because of the FIXME present in
computePreferredLogicalWidths(), hence thought of keeping std::max here (Can be
handy in cases when we have different values for these).

I will upload the next patch including this as well other changes once we can be
sure of what to with std::max.

Please do let me know whether we should still keep it or not?

+
+        LayoutUnit scrollbarWidth = instrinsicScrollbarLogicalWidth();

[Julien - ping for review, 2014/09/09 14:39:40]

Let's correct the typo in the function name while touching this code (it should
be intrinsicScrollbarLogicalWidth).

[Sunil Ratnu, 2014/09/09 17:28:30]

Done.

+        maxLogicalWidth += scrollbarWidth;
+        minLogicalWidth += scrollbarWidth;
     }
 }