Drop LayoutVTTCue::adjustForTopAndBottomMarginBorderAndPadding

Source/core/layout/LayoutVTTCue.cpp

 /*
  * Copyright (C) 2012 Victor Carbune (victor@rosedu.org)
  *
  * 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 255 matching lines @@
     // position to their current position, and then jump to the step labeled
     // done positioning below. If there are multiple such positions that are
     // equidistant from their current position, use the highest one amongst
     // them; if there are several at that height, then use the leftmost one
     // amongst them.
 
     // 5. Otherwise, jump to the step labeled done positioning below. (The
     // boxes will unfortunately overlap.)
 }
 
-void LayoutVTTCue::adjustForTopAndBottomMarginBorderAndPadding()
-{
-    // Accommodate extra top and bottom padding, border or margin.
-    // Note: this is supported only for internal UA styling, not through the cue selector.
-    if (!hasInlineDirectionBordersPaddingOrMargin())
-        return;
-    IntRect containerRect = containingBlock()->absoluteBoundingBoxRect();
-    IntRect cueRect = absoluteBoundingBoxRect();
-
-    int topOverflow = cueRect.y() - containerRect.y();
-    int bottomOverflow = containerRect.y() + containerRect.height() - cueRect.y() - cueRect.height();
-
-    int adjustment = 0;
-    if (topOverflow < 0)
-        adjustment = -topOverflow;
-    else if (bottomOverflow < 0)
-        adjustment = bottomOverflow;
-
-    if (!adjustment)
-        return;
-
-    setY(location().y() + adjustment);
-}
-
 void LayoutVTTCue::layout()
 {
     LayoutBlockFlow::layout();
 
     ASSERT(firstChild());
 
     LayoutState state(*this, locationOffset());
 
     // Determine the area covered by the media controls, if any. If the controls
     // are present, they are the next sibling of the text track container, which
     // is our parent. (LayoutMedia ensures that the media controls are laid out
     // before text tracks, so that the layout is up-to-date here.)
     ASSERT(parent()->node()->isTextTrackContainer());
     IntRect controlsRect;
     if (LayoutObject* parentSibling = parent()->nextSibling()) {
         // Only a part of the media controls is used for overlap avoidance.
         MediaControls* controls = toMediaControls(parentSibling->node());
         if (LayoutObject* controlsLayout = controls->layoutObjectForTextTrackLayout())
             controlsRect = controlsLayout->absoluteBoundingBoxRect();
     }
 
     // http://dev.w3.org/html5/webvtt/#dfn-apply-webvtt-cue-settings - step 13.
-    if (!std::isnan(m_snapToLinesPosition)) {
+    if (!std::isnan(m_snapToLinesPosition))
         SnapToLinesLayouter(*this, controlsRect).layout();
-
-        adjustForTopAndBottomMarginBorderAndPadding();
-    } else {
+    else
         repositionCueSnapToLinesNotSet();
-    }
 }
 
 } // namespace blink