Enable WebVTT regions for runtime testing, updated tests and minor fixes

Source/core/rendering/RenderTextTrackCue.cpp

Index: Source/core/rendering/RenderTextTrackCue.cpp
diff --git a/Source/core/rendering/RenderTextTrackCue.cpp b/Source/core/rendering/RenderTextTrackCue.cpp
index b1327afd8c06cd4f3d42eb885a6d908b7bfa8597..d4b3a269b8b214d9e87f95444f4fb006520dc625 100644
--- a/Source/core/rendering/RenderTextTrackCue.cpp
+++ b/Source/core/rendering/RenderTextTrackCue.cpp
@@ -24,9 +24,9 @@
  */
 
 #include "config.h"
-
 #include "core/rendering/RenderTextTrackCue.h"
 
+#include "RuntimeEnabledFeatures.h"

[acolwell GONE FROM CHROMIUM, 2013/10/08 22:09:06]

nit: not needed anymore

[vcarbune.chromium, 2013/10/12 22:33:55]

Done.

 #include "core/html/track/TextTrackCue.h"
 #include "core/rendering/RenderView.h"
 
@@ -42,14 +42,12 @@ void RenderTextTrackCue::layout()
 {
     RenderBlockFlow::layout();
 
-#if ENABLE(WEBVTT_REGIONS)
     // If WebVTT Regions are used, the regular WebVTT layout algorithm is no
     // longer necessary, since cues having the region parameter set do not have
     // any positioning parameters. Also, in this case, the regions themselves
     // have positioning information.
     if (!m_cue->regionId().isEmpty())
         return;
-#endif
 
     LayoutStateMaintainer statePusher(view(), this, locationOffset(), hasTransform() || hasReflection() || style()->isFlippedBlocksWritingMode());