New media playback UI.

Source/core/layout/LayoutMedia.cpp

 /*
  * Copyright (C) 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 10 matching lines @@
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 
 #include "core/layout/LayoutMedia.h"
 
 #include "core/html/HTMLMediaElement.h"
+#include "core/html/shadow/MediaControls.h"
 #include "core/layout/LayoutView.h"
 
 namespace blink {
 
 LayoutMedia::LayoutMedia(HTMLMediaElement* video)
     : LayoutImage(video)
 {
     setImageResource(LayoutImageResource::create());
 }
 
@@ skipping 39 matching lines @@
         LayoutBox* layoutBox = toLayoutBox(child);
         layoutBox->setLocation(LayoutPoint(borderLeft(), borderTop()) + LayoutSize(paddingLeft(), paddingTop()));
         // 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));
         layoutBox->mutableStyleRef().setWidth(Length(newSize.width(), Fixed));
         layoutBox->forceLayout();
     }
 
     clearNeedsLayout();
+
+    // Notify our MediaControls that a layout has happened.
+    if (mediaElement() && mediaElement()->mediaControls() && newSize.width() != oldSize.width())
+        mediaElement()->mediaControls()->notifyPanelWidthChanged(newSize.width().toInt());

[philipj_slow, 2015/07/21 12:02:21]

Is there any particular reason to convert to int here instead of passing a
LayoutUnit? It wouldn't make much of a difference, but would make it more
explicit in the media controls code that this comes from layout, and make it
harder to update it from offsetWidth() or similar.

[liberato (no reviews please), 2015/07/27 20:26:09]

Done.

 }
 
 bool LayoutMedia::isChildAllowed(LayoutObject* child, const ComputedStyle&) const
 {
     // Two types of child layout objects are allowed: media controls
     // and the text track container. Filter children by node type.
     ASSERT(child->node());
 
     // The user agent stylesheet (mediaControls.css) has
     // ::-webkit-media-controls { display: flex; }. If author style
     // sets display: inline we would get an inline layoutObject as a child
     // of replaced content, which is not supposed to be possible. This
     // check can be removed if ::-webkit-media-controls is made
     // internal.
     if (child->node()->isMediaControls())
         return child->isFlexibleBox();
 
     if (child->node()->isTextTrackContainer())
         return true;
 
     return false;
 }
 
 void LayoutMedia::paintReplaced(const PaintInfo&, const LayoutPoint&)
 {
 }
 
 } // namespace blink