New media playback UI.

Source/core/paint/MediaControlsPainter.h

Index: Source/core/paint/MediaControlsPainter.h
diff --git a/Source/core/paint/MediaControlsPainter.h b/Source/core/paint/MediaControlsPainter.h
index cf2746568425fa174115fd7b6938ec2d58e58950..ff93437dffb8792957a0ea24ae42c4c84730405a 100644
--- a/Source/core/paint/MediaControlsPainter.h
+++ b/Source/core/paint/MediaControlsPainter.h
@@ -49,6 +49,9 @@ public:
     static bool paintMediaOverlayPlayButton(LayoutObject*, const PaintInfo&, const IntRect&);
     static bool paintMediaCastButton(LayoutObject*, const PaintInfo&, const IntRect&);
     static void adjustMediaSliderThumbSize(ComputedStyle&);
+
+private:
+    static void adjustMediaSliderThumbPaintSize(const IntRect&, const ComputedStyle&, IntRect& rectOut);

[philipj_slow, 2015/06/17 12:52:25]

Can't this simply return an IntRect by value? Even with out C++11 move semantics
there's a good chance it would fall under
https://en.wikipedia.org/wiki/Return_value_optimization

[liberato (no reviews please), 2015/06/17 17:07:26]

possibly, but it will end up being functionally the same thing in the best case.

to be sure, i fiddled around with clang.  when it doesn't have the method body
to inline, it seems to do better with the explicit out param.  when it does have
the method body, as it will in this case, it is more aggressive and they end up
being almost identical.

 };
 
 } // namespace blink