Move TextTrackContainer::updateSizes to the layout-side

Source/core/html/track/TextTrackContainer.cpp

 /*
  * Copyright (C) 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2012 Google 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.
@@ skipping 15 matching lines @@
  * (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/html/track/TextTrackContainer.h"
 
 #include "core/html/HTMLVideoElement.h"
 #include "core/html/track/CueTimeline.h"
 #include "core/layout/LayoutTextTrackContainer.h"
-#include "core/layout/LayoutVideo.h"
 
 namespace blink {
 
 TextTrackContainer::TextTrackContainer(Document& document)
     : HTMLDivElement(document)
-    , m_fontSize(0)
 {
 }
 
 PassRefPtrWillBeRawPtr<TextTrackContainer> TextTrackContainer::create(Document& document)
 {
     RefPtrWillBeRawPtr<TextTrackContainer> element = adoptRefWillBeNoop(new TextTrackContainer(document));
     element->setShadowPseudoId(AtomicString("-webkit-media-text-track-container", AtomicString::ConstructFromLiteral));
     element->setInlineStyleProperty(CSSPropertyDisplay, CSSValueNone);
     return element.release();
 }
@@ skipping 63 matching lines @@
         cue->updateDisplay(*this);
     }
 
     // 11. Return output.
     if (hasChildren())
         removeInlineStyleProperty(CSSPropertyDisplay);
     else
         setInlineStyleProperty(CSSPropertyDisplay, CSSValueNone);
 }
 
-void TextTrackContainer::updateSizes(LayoutObject* mediaElementLayoutObject)
-{
-    if (!document().isActive())
-        return;
-
-    if (!mediaElementLayoutObject || !mediaElementLayoutObject->isVideo())
-        return;
-
-    // FIXME: The video size is used to calculate the font size (a workaround
-    // for lack of per-spec vh/vw support) but the whole media element is used
-    // for cue rendering. This is inconsistent. See also the somewhat related
-    // spec bug: https://www.w3.org/Bugs/Public/show_bug.cgi?id=28105
-    IntSize videoSize = toLayoutVideo(mediaElementLayoutObject)->videoBox().size();
-
-    if (m_videoSize == videoSize)
-        return;
-    m_videoSize = videoSize;
-
-    float smallestDimension = std::min(m_videoSize.height(), m_videoSize.width());
-
-    float fontSize = smallestDimension * 0.05f;
-    if (fontSize != m_fontSize) {
-        m_fontSize = fontSize;
-        setInlineStyleProperty(CSSPropertyFontSize, fontSize, CSSPrimitiveValue::CSS_PX);
-    }
-}
-
 } // namespace blink