Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1151)

Unified Diff: Source/core/rendering/RenderVTTCue.h

Issue 873483003: Move WebVTT snap-to-lines layout code to a helper class (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/core/rendering/RenderVTTCue.cpp » ('j') | Source/core/rendering/RenderVTTCue.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/RenderVTTCue.h
diff --git a/Source/core/rendering/RenderVTTCue.h b/Source/core/rendering/RenderVTTCue.h
index 5683da6d22f9722273ee4e90b7507b176fe35ce8..112a10259ac4ed79dda695bea611c5467a1f0c85 100644
--- a/Source/core/rendering/RenderVTTCue.h
+++ b/Source/core/rendering/RenderVTTCue.h
@@ -42,21 +42,10 @@ public:
private:
virtual void layout() override;
- bool isOutside() const;
- bool isOverlapping() const;
- bool shouldSwitchDirection(InlineFlowBox*, LayoutUnit) const;
-
- void moveBoxesByStep(LayoutUnit);
- bool switchDirection(bool&, LayoutUnit&);
-
- bool findFirstLineBox(InlineFlowBox*&);
- bool initializeLayoutParameters(InlineFlowBox*, LayoutUnit&, LayoutUnit&);
- void placeBoxInDefaultPosition(LayoutUnit, bool&);
- void repositionCueSnapToLinesSet();
+ void adjustForTopAndBottomMarginBorderAndPadding();
void repositionCueSnapToLinesNotSet();
RawPtrWillBeMember<VTTCue> m_cue;
- LayoutPoint m_specifiedPosition;
};
} // namespace blink
« no previous file with comments | « no previous file | Source/core/rendering/RenderVTTCue.cpp » ('j') | Source/core/rendering/RenderVTTCue.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698