Remove media controls when not in use.

Source/core/html/HTMLMediaElement.h

Index: Source/core/html/HTMLMediaElement.h
diff --git a/Source/core/html/HTMLMediaElement.h b/Source/core/html/HTMLMediaElement.h
index c4df625a816d7280341172d594160b7dc9638965..5afd294b6853e4806858a861e142249a05d9a088 100644
--- a/Source/core/html/HTMLMediaElement.h
+++ b/Source/core/html/HTMLMediaElement.h
@@ -166,9 +166,13 @@ public:
     void setMediaKeys(MediaKeys*, ExceptionState&);
     DEFINE_ATTRIBUTE_EVENT_LISTENER(needkey);
 
-    // controls
+    // This reflects the DOM attribute.

[philipj_slow, 2014/03/12 16:43:34]

If you add [Reflect] to HTMLMediaElement.idl you can remove both of these
instead, since neither are actually needed internally.

     bool controls() const;
     void setControls(bool);
+
+    // This version takes scripting and full screen mode into account.
+    bool shouldDisplayControls() const;

[philipj_slow, 2014/03/12 16:43:34]

Bikeshed: shouldShowControls() to go with MediaControls::show() and
HTMLMediaElement::showingTrackWithSameKind()?

+
     virtual double volume() const OVERRIDE FINAL;
     virtual void setVolume(double, ExceptionState&) OVERRIDE FINAL;
     virtual bool muted() const OVERRIDE FINAL;