New media playback UI.

Source/core/html/shadow/MediaControlElementTypes.h

Index: Source/core/html/shadow/MediaControlElementTypes.h
diff --git a/Source/core/html/shadow/MediaControlElementTypes.h b/Source/core/html/shadow/MediaControlElementTypes.h
index 09ad25fe82421f4fb91c91483cb4f623a241aef4..f124bf254fc96293d0bc12ad47830d3795bf9aec 100644
--- a/Source/core/html/shadow/MediaControlElementTypes.h
+++ b/Source/core/html/shadow/MediaControlElementTypes.h
@@ -75,10 +75,31 @@ CORE_EXPORT MediaControlElementType mediaControlElementType(Node*);
 // ----------------------------
 
 class MediaControlElement : public WillBeGarbageCollectedMixin {
-public:
+private:

[philipj_slow, 2015/07/09 08:56:09]

Move these things to the private section at the bottom instead. If you can
remove show() and hide() too, better to just have a single way (public or
private) to update the inline style.

[liberato (no reviews please), 2015/07/09 22:35:33]

Done.

     void hide();
     void show();
 
+    // hide() or show() based on our fits / wanted state.  We want to show()
+    // if an only if we're wanted and we fit.
+    void updateShownState();
+
+public:
+    bool isShown();
+
+    // These hold the state about whether this control should be shown if
+    // space permits.  These will also show / hide as needed.
+    void setIsWanted(bool);
+    bool isWanted();
+
+    // Tell us whether we fit or not.  This will hide / show the control as
+    // needed, also.
+    void setDoesFit(bool);
+
+    // This returns the minimum width in pixels that we need to show
+    // this control.  Ideally, this would just be shorthand for the computed
+    // style, but this must also work when the control is hidden.
+    int minimumWidth();
+
     MediaControlElementType displayType() { return m_displayType; }
 
     DECLARE_VIRTUAL_TRACE();
@@ -95,6 +116,9 @@ private:
     MediaControls& m_mediaControls;
     MediaControlElementType m_displayType;
     RawPtrWillBeMember<HTMLElement> m_element;
+    bool m_isShown;

[philipj_slow, 2015/07/09 08:56:09]

Does this always equal m_isWanted && m_doesFit? If so remove it.

Also append ": 1" to consecutive bools to make them a bitfield.

[liberato (no reviews please), 2015/07/09 22:35:33]

Done.

+    bool m_isWanted;
+    bool m_doesFit;
 };
 
 // ----------------------------