Even more explicit LayoutUnit conversion

third_party/WebKit/Source/core/layout/LayoutMedia.cpp

Index: third_party/WebKit/Source/core/layout/LayoutMedia.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutMedia.cpp b/third_party/WebKit/Source/core/layout/LayoutMedia.cpp
index dddb3e24d99ebd556730b865256942e3a5ce6444..2d653aa3dc00eafbc9b1658b2b312b79a265d18d 100644
--- a/third_party/WebKit/Source/core/layout/LayoutMedia.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutMedia.cpp
@@ -77,7 +77,7 @@ void LayoutMedia::layout()
             continue;
 
         LayoutBox* layoutBox = toLayoutBox(child);
-        layoutBox->setLocation(LayoutPoint(borderLeft(), borderTop()) + LayoutSize(paddingLeft(), paddingTop()));
+        layoutBox->setLocation(LayoutPoint(LayoutUnit(borderLeft()), borderTop()) + LayoutSize(paddingLeft(), paddingTop()));

[eae, 2016/01/30 05:18:36]

Isn't this exactly what contentBoxOffset() does (return border+padding)?
 
layoutBox->setLocation(contentBoxOffset());

[leviw_travelin_and_unemployed, 2016/01/30 05:36:06]

Awesome point :D

         // TODO(philipj): Remove the mutableStyleRef() and depend on CSS
         // width/height: inherit to match the media element size.
         layoutBox->mutableStyleRef().setHeight(Length(newSize.height(), Fixed));