Move VTT functionality from TextTrackCue.* to VTTCue.*

Source/core/rendering/RenderTextTrackCue.h

Index: Source/core/rendering/RenderTextTrackCue.h
diff --git a/Source/core/rendering/RenderTextTrackCue.h b/Source/core/rendering/RenderTextTrackCue.h
index ee0209792a3c9c353ccfec45be9f0a8a55b9aaf2..89ec3683118644bee752fa91892727b8f24b5dfa 100644
--- a/Source/core/rendering/RenderTextTrackCue.h
+++ b/Source/core/rendering/RenderTextTrackCue.h
@@ -27,36 +27,17 @@
 #define RenderTextTrackCue_h
 
 #include "core/rendering/RenderBlockFlow.h"
-#include "core/rendering/RenderInline.h"
-#include "platform/geometry/FloatPoint.h"
 
 namespace WebCore {
 
 class TextTrackCueBox;
 
-class RenderTextTrackCue FINAL : public RenderBlockFlow {
+class RenderTextTrackCue : public RenderBlockFlow {

[acolwell GONE FROM CHROMIUM, 2013/11/20 18:38:45]

I think we should just remove this object and have the RenderVTTCue just derive
from RenderBlockFlow. This object isn't really providing any value right now. We
can always add it back if a need presents itself.

[gasubic, 2013/11/21 03:07:38]

Done.

 public:
     explicit RenderTextTrackCue(TextTrackCueBox*);
 
 private:
-    virtual void layout() OVERRIDE;
     virtual bool supportsPartialLayout() const OVERRIDE { return false; }
-
-    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 repositionCueSnapToLinesNotSet();
-
-    TextTrackCue* m_cue;
-    FloatPoint m_fallbackPosition;
 };
 
 } // namespace WebCore