Update TextTrackCue API
authoreric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 25 Feb 2012 00:58:55 +0000 (00:58 +0000)
committereric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 25 Feb 2012 00:58:55 +0000 (00:58 +0000)
https://bugs.webkit.org/show_bug.cgi?id=79368

Source/WebCore:

Change TextTrackCue.alignment to .align, TextTrackCue.linePosition and .textPosition to .line
and .position, and TextTrackCue.direction to .vertical. Change direction values "horizontal"
to "","vertical" to "rl", and "vertical-lr" to "lr".

Reviewed by Eric Seidel.

No new tests, updated existing tests for API changes.

* html/track/TextTrackCue.cpp:
(WebCore::horizontalKeyword):
(WebCore::verticalGrowingLeftKeyword):
(WebCore):
(WebCore::verticalGrowingRightKeyword):
(WebCore::verticalKeywordOLD):
(WebCore::verticallrKeywordOLD):
(WebCore::TextTrackCue::TextTrackCue):
(WebCore::TextTrackCue::vertical):
(WebCore::TextTrackCue::setVertical):
(WebCore::TextTrackCue::setLine):
(WebCore::TextTrackCue::setPosition):
(WebCore::TextTrackCue::align):
(WebCore::TextTrackCue::setAlign):
(WebCore::TextTrackCue::parseSettings):
* html/track/TextTrackCue.h:
(TextTrackCue):
(WebCore::TextTrackCue::line):
(WebCore::TextTrackCue::position):
* html/track/TextTrackCue.idl:

LayoutTests:

Reviewed by Eric Seidel.

* media/track/track-add-remove-cue-expected.txt:
* media/track/track-add-remove-cue.html:
* media/track/track-cue-mutable-expected.txt:
* media/track/track-cue-mutable.html:
* media/track/track-webvtt-tc013-settings-expected.txt:
* media/track/track-webvtt-tc013-settings.html:
* media/track/track-webvtt-tc014-alignment-expected.txt:
* media/track/track-webvtt-tc014-alignment.html:
* media/track/track-webvtt-tc015-positioning-expected.txt:
* media/track/track-webvtt-tc015-positioning.html:
* media/track/track-webvtt-tc016-align-positioning-expected.txt:
* media/track/track-webvtt-tc016-align-positioning.html:
* media/track/track-webvtt-tc017-line-position-expected.txt:
* media/track/track-webvtt-tc017-line-position.html:
* media/track/track-webvtt-tc018-align-text-line-position-expected.txt:
* media/track/track-webvtt-tc018-align-text-line-position.html:
* media/track/track-webvtt-tc020-cue-size-align-expected.txt:
* media/track/track-webvtt-tc020-cue-size-align.html:
* media/track/track-webvtt-tc021-valign-expected.txt:
* media/track/track-webvtt-tc021-valign.html:
* platform/mac/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@108872 268f45cc-cd09-0410-ab3c-d52691b4dbfc

25 files changed:
LayoutTests/ChangeLog
LayoutTests/media/track/track-add-remove-cue-expected.txt
LayoutTests/media/track/track-add-remove-cue.html
LayoutTests/media/track/track-cue-mutable-expected.txt
LayoutTests/media/track/track-cue-mutable.html
LayoutTests/media/track/track-webvtt-tc013-settings-expected.txt
LayoutTests/media/track/track-webvtt-tc013-settings.html
LayoutTests/media/track/track-webvtt-tc014-alignment-expected.txt
LayoutTests/media/track/track-webvtt-tc014-alignment.html
LayoutTests/media/track/track-webvtt-tc015-positioning-expected.txt
LayoutTests/media/track/track-webvtt-tc015-positioning.html
LayoutTests/media/track/track-webvtt-tc016-align-positioning-expected.txt
LayoutTests/media/track/track-webvtt-tc016-align-positioning.html
LayoutTests/media/track/track-webvtt-tc017-line-position-expected.txt
LayoutTests/media/track/track-webvtt-tc017-line-position.html
LayoutTests/media/track/track-webvtt-tc018-align-text-line-position-expected.txt
LayoutTests/media/track/track-webvtt-tc018-align-text-line-position.html
LayoutTests/media/track/track-webvtt-tc020-cue-size-align-expected.txt
LayoutTests/media/track/track-webvtt-tc020-cue-size-align.html
LayoutTests/media/track/track-webvtt-tc021-valign-expected.txt
LayoutTests/media/track/track-webvtt-tc021-valign.html
Source/WebCore/ChangeLog
Source/WebCore/html/track/TextTrackCue.cpp
Source/WebCore/html/track/TextTrackCue.h
Source/WebCore/html/track/TextTrackCue.idl

index 2a0f288..23ee6a8 100644 (file)
@@ -1,3 +1,32 @@
+2012-02-24  Eric Carlson  <eric.carlson@apple.com>
+
+        Update TextTrackCue API
+        https://bugs.webkit.org/show_bug.cgi?id=79368
+
+        Reviewed by Eric Seidel.
+
+        * media/track/track-add-remove-cue-expected.txt:
+        * media/track/track-add-remove-cue.html:
+        * media/track/track-cue-mutable-expected.txt:
+        * media/track/track-cue-mutable.html:
+        * media/track/track-webvtt-tc013-settings-expected.txt:
+        * media/track/track-webvtt-tc013-settings.html:
+        * media/track/track-webvtt-tc014-alignment-expected.txt:
+        * media/track/track-webvtt-tc014-alignment.html:
+        * media/track/track-webvtt-tc015-positioning-expected.txt:
+        * media/track/track-webvtt-tc015-positioning.html:
+        * media/track/track-webvtt-tc016-align-positioning-expected.txt:
+        * media/track/track-webvtt-tc016-align-positioning.html:
+        * media/track/track-webvtt-tc017-line-position-expected.txt:
+        * media/track/track-webvtt-tc017-line-position.html:
+        * media/track/track-webvtt-tc018-align-text-line-position-expected.txt:
+        * media/track/track-webvtt-tc018-align-text-line-position.html:
+        * media/track/track-webvtt-tc020-cue-size-align-expected.txt:
+        * media/track/track-webvtt-tc020-cue-size-align.html:
+        * media/track/track-webvtt-tc021-valign-expected.txt:
+        * media/track/track-webvtt-tc021-valign.html:
+        * platform/mac/Skipped:
+
 2012-02-24  Noel Gordon  <noel.gordon@gmail.com>
 
         [chromium] JPEG RGB image with Adode Marker fails to turbo swizzle decode
index d881b9b..659353f 100644 (file)
@@ -16,12 +16,12 @@ EXPECTED (textCue.id == 'sausage-cue') OK
 EXPECTED (textCue.startTime == '33') OK
 EXPECTED (textCue.endTime == '3.4') OK
 EXPECTED (textCue.pauseOnExit == 'false') OK
-EXPECTED (textCue.direction == 'horizontal') OK
+EXPECTED (textCue.vertical == '') OK
 EXPECTED (textCue.snapToLines == 'true') OK
-EXPECTED (textCue.linePosition == '-1') OK
-EXPECTED (textCue.textPosition == '50') OK
+EXPECTED (textCue.line == '-1') OK
+EXPECTED (textCue.position == '50') OK
 EXPECTED (textCue.size == '100') OK
-EXPECTED (textCue.alignment == 'middle') OK
+EXPECTED (textCue.align == 'middle') OK
 
 *** Add the new cue to a track, make sure it is inserted correctly.
 RUN(testTrack.track.addCue(textCue))
@@ -39,12 +39,12 @@ EXPECTED (newCue.id == 'test') OK
 EXPECTED (newCue.startTime == '0') OK
 EXPECTED (newCue.endTime == '1') OK
 EXPECTED (newCue.pauseOnExit == 'false') OK
-EXPECTED (newCue.direction == 'horizontal') OK
+EXPECTED (newCue.vertical == '') OK
 EXPECTED (newCue.snapToLines == 'true') OK
-EXPECTED (newCue.linePosition == '-1') OK
-EXPECTED (newCue.textPosition == '50') OK
+EXPECTED (newCue.line == '-1') OK
+EXPECTED (newCue.position == '50') OK
 EXPECTED (newCue.size == '100') OK
-EXPECTED (newCue.alignment == 'middle') OK
+EXPECTED (newCue.align == 'middle') OK
 
 *** Remove a cue created with addCue().
 RUN(testTrack.track.removeCue(textCue))
index d40c849..dbd3dbc 100644 (file)
                 testExpected("textCue.startTime", 33);
                 testExpected("textCue.endTime", 3.4);
                 testExpected("textCue.pauseOnExit", false);
-                testExpected("textCue.direction", "horizontal");
+                testExpected("textCue.vertical", "");
                 testExpected("textCue.snapToLines", true);
-                testExpected("textCue.linePosition", -1);
-                testExpected("textCue.textPosition", 50);
+                testExpected("textCue.line", -1);
+                testExpected("textCue.position", 50);
                 testExpected("textCue.size", 100);
-                testExpected("textCue.alignment", "middle");
+                testExpected("textCue.align", "middle");
 
                 consoleWrite("<br>*** Add the new cue to a track, make sure it is inserted correctly.");
                 run("testTrack.track.addCue(textCue)");
                 testExpected("newCue.startTime", 0.0);
                 testExpected("newCue.endTime", 1.0);
                 testExpected("newCue.pauseOnExit", false);
-                testExpected("newCue.direction", "horizontal");
+                testExpected("newCue.vertical", "");
                 testExpected("newCue.snapToLines", true);
-                testExpected("newCue.linePosition", -1);
-                testExpected("newCue.textPosition", 50);
+                testExpected("newCue.line", -1);
+                testExpected("newCue.position", 50);
                 testExpected("newCue.size", 100);
-                testExpected("newCue.alignment", "middle");
+                testExpected("newCue.align", "middle");
 
                 consoleWrite("<br>*** Remove a cue created with addCue().");
                 run("testTrack.track.removeCue(textCue)");
index a5f11d0..49a9960 100644 (file)
@@ -5,12 +5,12 @@ RUN(textCue = cues.getCueById('1'))
 EXPECTED (textCue.startTime == '0') OK
 EXPECTED (textCue.endTime == '1') OK
 EXPECTED (textCue.pauseOnExit == 'false') OK
-EXPECTED (textCue.direction == 'horizontal') OK
+EXPECTED (textCue.vertical == '') OK
 EXPECTED (textCue.snapToLines == 'true') OK
-EXPECTED (textCue.linePosition == '-1') OK
-EXPECTED (textCue.textPosition == '50') OK
+EXPECTED (textCue.line == '-1') OK
+EXPECTED (textCue.position == '50') OK
 EXPECTED (textCue.size == '100') OK
-EXPECTED (textCue.alignment == 'middle') OK
+EXPECTED (textCue.align == 'middle') OK
 
 ** Modify cue values.
 RUN(textCue.startTime = 1.1)
@@ -23,34 +23,34 @@ RUN(textCue.pauseOnExit = true)
 EXPECTED (textCue.pauseOnExit == 'true') OK
 
 On setting, the text track cue writing direction must be set to the value ... is a case-sensitive match for the new value, if any. If none of the values match, then the user agent must instead throw a SyntaxError exception.
-http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-direction
-TEST(textCue.direction = 'VERTICAL') THROWS(DOMException.SYNTAX_ERR) OK
-EXPECTED (textCue.direction == 'horizontal') OK
-RUN(textCue.direction = 'vertical')
-EXPECTED (textCue.direction == 'vertical') OK
+http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-vertical
+TEST(textCue.vertical = 'RL') THROWS(DOMException.SYNTAX_ERR) OK
+EXPECTED (textCue.vertical == '') OK
+RUN(textCue.vertical = 'rl')
+EXPECTED (textCue.vertical == 'rl') OK
 
 RUN(textCue.snapToLines = false)
 EXPECTED (textCue.snapToLines == 'false') OK
 
 On setting, if the text track cue snap-to-lines flag is not set, and the new value is negative or greater than 100, then throw an IndexSizeError exception.
-http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-lineposition
-TEST(textCue.linePosition = -2) THROWS(DOMException.INDEX_SIZE_ERR) OK
-TEST(textCue.linePosition = 102) THROWS(DOMException.INDEX_SIZE_ERR) OK
-EXPECTED (textCue.linePosition == '-1') OK
-RUN(textCue.linePosition = 42)
-EXPECTED (textCue.linePosition == '42') OK
+http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-line
+TEST(textCue.line = -2) THROWS(DOMException.INDEX_SIZE_ERR) OK
+TEST(textCue.line = 102) THROWS(DOMException.INDEX_SIZE_ERR) OK
+EXPECTED (textCue.line == '-1') OK
+RUN(textCue.line = 42)
+EXPECTED (textCue.line == '42') OK
 RUN(textCue.snapToLines = true)
-RUN(textCue.linePosition = -2)
-EXPECTED (textCue.linePosition == '-2') OK
-RUN(textCue.linePosition = 102)
-EXPECTED (textCue.linePosition == '102') OK
+RUN(textCue.line = -2)
+EXPECTED (textCue.line == '-2') OK
+RUN(textCue.line = 102)
+EXPECTED (textCue.line == '102') OK
 
 On setting, if the new value is negative or greater than 100, then throw an IndexSizeError exception. Otherwise, set the text track cue text position to the new value.
-http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-lineposition
-TEST(textCue.textPosition = -200) THROWS(DOMException.INDEX_SIZE_ERR) OK
-TEST(textCue.textPosition = 110) THROWS(DOMException.INDEX_SIZE_ERR) OK
-RUN(textCue.textPosition = 11)
-EXPECTED (textCue.textPosition == '11') OK
+http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-line
+TEST(textCue.position = -200) THROWS(DOMException.INDEX_SIZE_ERR) OK
+TEST(textCue.position = 110) THROWS(DOMException.INDEX_SIZE_ERR) OK
+RUN(textCue.position = 11)
+EXPECTED (textCue.position == '11') OK
 
 On setting, if the new value is negative or greater than 100, then throw an IndexSizeError exception. Otherwise, set the text track cue size to the new value.
 http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-size
@@ -60,11 +60,11 @@ RUN(textCue.size = 57)
 EXPECTED (textCue.size == '57') OK
 
 On setting, the text track cue alignment must be set to the value ... is a case-sensitive match for the new value, if any. If none of the values match, then the user agent must instead throw a SyntaxError exception.
-http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-alignment
-TEST(textCue.alignment = 'End') THROWS(DOMException.SYNTAX_ERR) OK
-EXPECTED (textCue.alignment == 'middle') OK
-RUN(textCue.alignment = 'end')
-EXPECTED (textCue.alignment == 'end') OK
+http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-align
+TEST(textCue.align = 'End') THROWS(DOMException.SYNTAX_ERR) OK
+EXPECTED (textCue.align == 'middle') OK
+RUN(textCue.align = 'end')
+EXPECTED (textCue.align == 'end') OK
 
 END OF TEST
 
index 2702277..fa1d888 100644 (file)
                 testExpected("textCue.startTime", 0);
                 testExpected("textCue.endTime", 1.0);
                 testExpected("textCue.pauseOnExit", false);
-                testExpected("textCue.direction", "horizontal");
+                testExpected("textCue.vertical", "");
                 testExpected("textCue.snapToLines", true);
-                testExpected("textCue.linePosition", -1);
-                testExpected("textCue.textPosition", 50);
+                testExpected("textCue.line", -1);
+                testExpected("textCue.position", 50);
                 testExpected("textCue.size", 100);
-                testExpected("textCue.alignment", "middle");
+                testExpected("textCue.align", "middle");
 
                 consoleWrite("<br>** Modify cue values.");
 
                 run("textCue.pauseOnExit = true");
                 testExpected("textCue.pauseOnExit", true);
 
-                logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-direction", 
+                logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-vertical", 
                            "On setting, the text track cue writing direction must be set to the value ... is a case-sensitive match for the new value, if any. If none of the values match, then the user agent must instead throw a SyntaxError exception.");
-                testException("textCue.direction = 'VERTICAL'", "DOMException.SYNTAX_ERR");
-                testExpected("textCue.direction", "horizontal");
-                run("textCue.direction = 'vertical'");
-                testExpected("textCue.direction", "vertical");
+                testException("textCue.vertical = 'RL'", "DOMException.SYNTAX_ERR");
+                testExpected("textCue.vertical", "");
+                run("textCue.vertical = 'rl'");
+                testExpected("textCue.vertical", "rl");
 
                 consoleWrite("");
                 run("textCue.snapToLines = false");
                 testExpected("textCue.snapToLines", false);
 
-                logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-lineposition",
+                logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-line",
                             "On setting, if the text track cue snap-to-lines flag is not set, and the new value is negative or greater than 100, then throw an IndexSizeError exception.");
-                testException("textCue.linePosition = -2", "DOMException.INDEX_SIZE_ERR");
-                testException("textCue.linePosition = 102", "DOMException.INDEX_SIZE_ERR");
-                testExpected("textCue.linePosition", -1);
-                run("textCue.linePosition = 42");
-                testExpected("textCue.linePosition", 42);
+                testException("textCue.line = -2", "DOMException.INDEX_SIZE_ERR");
+                testException("textCue.line = 102", "DOMException.INDEX_SIZE_ERR");
+                testExpected("textCue.line", -1);
+                run("textCue.line = 42");
+                testExpected("textCue.line", 42);
                 run("textCue.snapToLines = true");
-                run("textCue.linePosition = -2");
-                testExpected("textCue.linePosition", -2);
-                run("textCue.linePosition = 102");
-                testExpected("textCue.linePosition", 102);
+                run("textCue.line = -2");
+                testExpected("textCue.line", -2);
+                run("textCue.line = 102");
+                testExpected("textCue.line", 102);
 
-                logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-lineposition", 
+                logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-line", 
                            "On setting, if the new value is negative or greater than 100, then throw an IndexSizeError exception. Otherwise, set the text track cue text position to the new value.");
-                testException("textCue.textPosition = -200", "DOMException.INDEX_SIZE_ERR");
-                testException("textCue.textPosition = 110", "DOMException.INDEX_SIZE_ERR");
-                run("textCue.textPosition = 11");
-                testExpected("textCue.textPosition", 11);
+                testException("textCue.position = -200", "DOMException.INDEX_SIZE_ERR");
+                testException("textCue.position = 110", "DOMException.INDEX_SIZE_ERR");
+                run("textCue.position = 11");
+                testExpected("textCue.position", 11);
 
                 logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-size",
                            "On setting, if the new value is negative or greater than 100, then throw an IndexSizeError exception. Otherwise, set the text track cue size to the new value.");
                 run("textCue.size = 57");
                 testExpected("textCue.size", 57);
 
-                logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-alignment",
+                logSpecURL("http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-align",
                            "On setting, the text track cue alignment must be set to the value ... is a case-sensitive match for the new value, if any. If none of the values match, then the user agent must instead throw a SyntaxError exception.");
-                testException("textCue.alignment = 'End'", "DOMException.SYNTAX_ERR");
-                testExpected("textCue.alignment", "middle");
-                run("textCue.alignment = 'end'");
-                testExpected("textCue.alignment", "end");
+                testException("textCue.align = 'End'", "DOMException.SYNTAX_ERR");
+                testExpected("textCue.align", "middle");
+                run("textCue.align = 'end'");
+                testExpected("textCue.align", "end");
 
                 consoleWrite("");
                 endTest();
index f5710fd..42baf44 100644 (file)
@@ -3,40 +3,40 @@ Tests WebVTT settings.
 
 *** Testing text track 0
 EXPECTED (cues.length == '4') OK
-EXPECTED (cues[0].linePosition == '100') OK
-EXPECTED (cues[0].textPosition == '50') OK
-EXPECTED (cues[0].alignment == 'middle') OK
-EXPECTED (cues[0].direction == 'horizontal') OK
-EXPECTED (cues[1].linePosition == '-1') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[1].direction == 'horizontal') OK
-EXPECTED (cues[2].linePosition == '-1') OK
-EXPECTED (cues[2].textPosition == '50') OK
-EXPECTED (cues[2].alignment == 'middle') OK
-EXPECTED (cues[2].direction == 'horizontal') OK
-EXPECTED (cues[3].linePosition == '-1') OK
-EXPECTED (cues[3].textPosition == '50') OK
-EXPECTED (cues[3].alignment == 'middle') OK
-EXPECTED (cues[3].direction == 'horizontal') OK
+EXPECTED (cues[0].line == '100') OK
+EXPECTED (cues[0].position == '50') OK
+EXPECTED (cues[0].align == 'middle') OK
+EXPECTED (cues[0].vertical == '') OK
+EXPECTED (cues[1].line == '-1') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[1].vertical == '') OK
+EXPECTED (cues[2].line == '-1') OK
+EXPECTED (cues[2].position == '50') OK
+EXPECTED (cues[2].align == 'middle') OK
+EXPECTED (cues[2].vertical == '') OK
+EXPECTED (cues[3].line == '-1') OK
+EXPECTED (cues[3].position == '50') OK
+EXPECTED (cues[3].align == 'middle') OK
+EXPECTED (cues[3].vertical == '') OK
 
 *** Testing text track 1
 EXPECTED (cues.length == '4') OK
-EXPECTED (cues[0].linePosition == '100') OK
-EXPECTED (cues[0].textPosition == '50') OK
-EXPECTED (cues[0].alignment == 'middle') OK
-EXPECTED (cues[0].direction == 'horizontal') OK
-EXPECTED (cues[1].linePosition == '-1') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[1].direction == 'horizontal') OK
-EXPECTED (cues[2].linePosition == '-1') OK
-EXPECTED (cues[2].textPosition == '50') OK
-EXPECTED (cues[2].alignment == 'middle') OK
-EXPECTED (cues[2].direction == 'horizontal') OK
-EXPECTED (cues[3].linePosition == '-1') OK
-EXPECTED (cues[3].textPosition == '50') OK
-EXPECTED (cues[3].alignment == 'middle') OK
-EXPECTED (cues[3].direction == 'horizontal') OK
+EXPECTED (cues[0].line == '100') OK
+EXPECTED (cues[0].position == '50') OK
+EXPECTED (cues[0].align == 'middle') OK
+EXPECTED (cues[0].vertical == '') OK
+EXPECTED (cues[1].line == '-1') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[1].vertical == '') OK
+EXPECTED (cues[2].line == '-1') OK
+EXPECTED (cues[2].position == '50') OK
+EXPECTED (cues[2].align == 'middle') OK
+EXPECTED (cues[2].vertical == '') OK
+EXPECTED (cues[3].line == '-1') OK
+EXPECTED (cues[3].position == '50') OK
+EXPECTED (cues[3].align == 'middle') OK
+EXPECTED (cues[3].vertical == '') OK
 END OF TEST
 
index 7722a52..9385607 100644 (file)
                     tests:
                     [
                         {
-                            property : "linePosition",
+                            property : "line",
                             values : [100, -1, -1, -1],
                         },
                         {
-                            property : "textPosition",
+                            property : "position",
                             values : [50, 50, 50, 50],
                         },
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["middle", "middle", "middle", "middle"],
                         },
                         {
-                            property : "direction",
-                            values : ["horizontal", "horizontal", "horizontal", "horizontal"],
+                            property : "vertical",
+                            values : ["", "", "", ""],
                         },
                     ],
                 };
index efed51e..e2ce9dd 100644 (file)
@@ -3,23 +3,23 @@ Tests cue alignment from settings.
 
 *** Testing text track 0
 EXPECTED (cues.length == '4') OK
-EXPECTED (cues[0].alignment == 'start') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[2].alignment == 'end') OK
-EXPECTED (cues[3].alignment == 'middle') OK
+EXPECTED (cues[0].align == 'start') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[2].align == 'end') OK
+EXPECTED (cues[3].align == 'middle') OK
 
 *** Testing text track 1
 EXPECTED (cues.length == '4') OK
-EXPECTED (cues[0].alignment == 'start') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[2].alignment == 'end') OK
-EXPECTED (cues[3].alignment == 'middle') OK
+EXPECTED (cues[0].align == 'start') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[2].align == 'end') OK
+EXPECTED (cues[3].align == 'middle') OK
 
 *** Testing text track 2
 EXPECTED (cues.length == '4') OK
-EXPECTED (cues[0].alignment == 'middle') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[2].alignment == 'middle') OK
-EXPECTED (cues[3].alignment == 'middle') OK
+EXPECTED (cues[0].align == 'middle') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[2].align == 'middle') OK
+EXPECTED (cues[3].align == 'middle') OK
 END OF TEST
 
index cf734bd..2b4cb4b 100644 (file)
@@ -28,7 +28,7 @@
                     tests:
                     [
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["start", "middle", "end", "middle"],
                         },
                     ],
@@ -47,7 +47,7 @@
                     tests:
                     [
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["middle", "middle", "middle", "middle"],
                         },
                     ],
index 585be83..53ca120 100644 (file)
@@ -3,27 +3,27 @@ Tests cue text position from settings.
 
 *** Testing text track 0
 EXPECTED (cues.length == '4') OK
-EXPECTED (cues[0].textPosition == '0') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[2].textPosition == '50') OK
-EXPECTED (cues[3].textPosition == '100') OK
+EXPECTED (cues[0].position == '0') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[2].position == '50') OK
+EXPECTED (cues[3].position == '100') OK
 
 *** Testing text track 1
 EXPECTED (cues.length == '4') OK
-EXPECTED (cues[0].textPosition == '0') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[2].textPosition == '50') OK
-EXPECTED (cues[3].textPosition == '100') OK
+EXPECTED (cues[0].position == '0') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[2].position == '50') OK
+EXPECTED (cues[3].position == '100') OK
 
 *** Testing text track 2
 EXPECTED (cues.length == '8') OK
-EXPECTED (cues[0].textPosition == '50') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[2].textPosition == '50') OK
-EXPECTED (cues[3].textPosition == '50') OK
-EXPECTED (cues[4].textPosition == '50') OK
-EXPECTED (cues[5].textPosition == '50') OK
-EXPECTED (cues[6].textPosition == '50') OK
-EXPECTED (cues[7].textPosition == '50') OK
+EXPECTED (cues[0].position == '50') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[2].position == '50') OK
+EXPECTED (cues[3].position == '50') OK
+EXPECTED (cues[4].position == '50') OK
+EXPECTED (cues[5].position == '50') OK
+EXPECTED (cues[6].position == '50') OK
+EXPECTED (cues[7].position == '50') OK
 END OF TEST
 
index f2336f4..21621fd 100644 (file)
@@ -28,7 +28,7 @@
                     tests:
                     [
                         {
-                            property : "textPosition",
+                            property : "position",
                             values : [0, 50, 50, 100],
                         },
                     ],
@@ -47,7 +47,7 @@
                     tests:
                     [
                         {
-                            property : "textPosition",
+                            property : "position",
                             values : [50, 50, 50, 50, 50, 50, 50, 50],
                         },
                     ],
index 8d3e366..4fbbf26 100644 (file)
@@ -3,20 +3,20 @@ Tests cue text position and alignment from settings.
 
 *** Testing text track 0
 EXPECTED (cues.length == '3') OK
-EXPECTED (cues[0].textPosition == '10') OK
-EXPECTED (cues[0].alignment == 'start') OK
-EXPECTED (cues[1].textPosition == '20') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[2].textPosition == '80') OK
-EXPECTED (cues[2].alignment == 'end') OK
+EXPECTED (cues[0].position == '10') OK
+EXPECTED (cues[0].align == 'start') OK
+EXPECTED (cues[1].position == '20') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[2].position == '80') OK
+EXPECTED (cues[2].align == 'end') OK
 
 *** Testing text track 1
 EXPECTED (cues.length == '3') OK
-EXPECTED (cues[0].textPosition == '10') OK
-EXPECTED (cues[0].alignment == 'middle') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[2].textPosition == '50') OK
-EXPECTED (cues[2].alignment == 'middle') OK
+EXPECTED (cues[0].position == '10') OK
+EXPECTED (cues[0].align == 'middle') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[2].position == '50') OK
+EXPECTED (cues[2].align == 'middle') OK
 END OF TEST
 
index e3fe358..8a9aa3e 100644 (file)
                     tests:
                     [
                         {
-                            property : "textPosition",
+                            property : "position",
                             values : [10, 20, 80],
                         },
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["start", "middle", "end"],
                         },
                     ],
                     tests:
                     [
                         {
-                            property : "textPosition",
+                            property : "position",
                             values : [10, 50, 50],
                         },
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["middle", "middle", "middle"],
                         },
                     ],
index d19d284..496ff7e 100644 (file)
@@ -3,34 +3,34 @@ Tests cue line position from settings.
 
 *** Testing text track 0
 EXPECTED (cues.length == '7') OK
-EXPECTED (cues[0].linePosition == '0') OK
+EXPECTED (cues[0].line == '0') OK
 EXPECTED (cues[0].snapToLines == 'false') OK
-EXPECTED (cues[1].linePosition == '0') OK
+EXPECTED (cues[1].line == '0') OK
 EXPECTED (cues[1].snapToLines == 'true') OK
-EXPECTED (cues[2].linePosition == '50') OK
+EXPECTED (cues[2].line == '50') OK
 EXPECTED (cues[2].snapToLines == 'false') OK
-EXPECTED (cues[3].linePosition == '5') OK
+EXPECTED (cues[3].line == '5') OK
 EXPECTED (cues[3].snapToLines == 'true') OK
-EXPECTED (cues[4].linePosition == '100') OK
+EXPECTED (cues[4].line == '100') OK
 EXPECTED (cues[4].snapToLines == 'false') OK
-EXPECTED (cues[5].linePosition == '-1') OK
+EXPECTED (cues[5].line == '-1') OK
 EXPECTED (cues[5].snapToLines == 'true') OK
-EXPECTED (cues[6].linePosition == '500') OK
+EXPECTED (cues[6].line == '500') OK
 EXPECTED (cues[6].snapToLines == 'true') OK
 
 *** Testing text track 1
 EXPECTED (cues.length == '6') OK
-EXPECTED (cues[0].linePosition == '-1') OK
+EXPECTED (cues[0].line == '-1') OK
 EXPECTED (cues[0].snapToLines == 'true') OK
-EXPECTED (cues[1].linePosition == '-1') OK
+EXPECTED (cues[1].line == '-1') OK
 EXPECTED (cues[1].snapToLines == 'true') OK
-EXPECTED (cues[2].linePosition == '-1') OK
+EXPECTED (cues[2].line == '-1') OK
 EXPECTED (cues[2].snapToLines == 'true') OK
-EXPECTED (cues[3].linePosition == '-1') OK
+EXPECTED (cues[3].line == '-1') OK
 EXPECTED (cues[3].snapToLines == 'true') OK
-EXPECTED (cues[4].linePosition == '-1') OK
+EXPECTED (cues[4].line == '-1') OK
 EXPECTED (cues[4].snapToLines == 'true') OK
-EXPECTED (cues[5].linePosition == '-1') OK
+EXPECTED (cues[5].line == '-1') OK
 EXPECTED (cues[5].snapToLines == 'true') OK
 END OF TEST
 
index 7618fd3..46ef8f1 100644 (file)
@@ -27,7 +27,7 @@
                     tests:
                     [
                         {
-                            property : "linePosition",
+                            property : "line",
                             values : [0, 0, 50, 5, 100, -1, 500],
                         },
                         {
@@ -50,7 +50,7 @@
                     tests:
                     [
                         {
-                            property : "linePosition",
+                            property : "line",
                             values : [-1, -1, -1, -1, -1, -1, -1],
                         },
                         {
index a830e58..10ab916 100644 (file)
@@ -3,40 +3,40 @@ Tests cue alignment, line and text position from settings.
 
 *** Testing text track 0
 EXPECTED (cues.length == '5') OK
-EXPECTED (cues[0].alignment == 'start') OK
-EXPECTED (cues[0].textPosition == '10') OK
-EXPECTED (cues[0].linePosition == '0') OK
+EXPECTED (cues[0].align == 'start') OK
+EXPECTED (cues[0].position == '10') OK
+EXPECTED (cues[0].line == '0') OK
 EXPECTED (cues[0].snapToLines == 'false') OK
-EXPECTED (cues[1].alignment == 'start') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[1].linePosition == '0') OK
+EXPECTED (cues[1].align == 'start') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[1].line == '0') OK
 EXPECTED (cues[1].snapToLines == 'true') OK
-EXPECTED (cues[2].alignment == 'middle') OK
-EXPECTED (cues[2].textPosition == '80') OK
-EXPECTED (cues[2].linePosition == '80') OK
+EXPECTED (cues[2].align == 'middle') OK
+EXPECTED (cues[2].position == '80') OK
+EXPECTED (cues[2].line == '80') OK
 EXPECTED (cues[2].snapToLines == 'false') OK
-EXPECTED (cues[3].alignment == 'end') OK
-EXPECTED (cues[3].textPosition == '30') OK
-EXPECTED (cues[3].linePosition == '5') OK
+EXPECTED (cues[3].align == 'end') OK
+EXPECTED (cues[3].position == '30') OK
+EXPECTED (cues[3].line == '5') OK
 EXPECTED (cues[3].snapToLines == 'true') OK
-EXPECTED (cues[4].alignment == 'middle') OK
-EXPECTED (cues[4].textPosition == '60') OK
-EXPECTED (cues[4].linePosition == '-3') OK
+EXPECTED (cues[4].align == 'middle') OK
+EXPECTED (cues[4].position == '60') OK
+EXPECTED (cues[4].line == '-3') OK
 EXPECTED (cues[4].snapToLines == 'true') OK
 
 *** Testing text track 1
 EXPECTED (cues.length == '3') OK
-EXPECTED (cues[0].alignment == 'middle') OK
-EXPECTED (cues[0].textPosition == '50') OK
-EXPECTED (cues[0].linePosition == '-1') OK
+EXPECTED (cues[0].align == 'middle') OK
+EXPECTED (cues[0].position == '50') OK
+EXPECTED (cues[0].line == '-1') OK
 EXPECTED (cues[0].snapToLines == 'true') OK
-EXPECTED (cues[1].alignment == 'end') OK
-EXPECTED (cues[1].textPosition == '0') OK
-EXPECTED (cues[1].linePosition == '-1') OK
+EXPECTED (cues[1].align == 'end') OK
+EXPECTED (cues[1].position == '0') OK
+EXPECTED (cues[1].line == '-1') OK
 EXPECTED (cues[1].snapToLines == 'true') OK
-EXPECTED (cues[2].alignment == 'middle') OK
-EXPECTED (cues[2].textPosition == '60') OK
-EXPECTED (cues[2].linePosition == '-3') OK
+EXPECTED (cues[2].align == 'middle') OK
+EXPECTED (cues[2].position == '60') OK
+EXPECTED (cues[2].line == '-3') OK
 EXPECTED (cues[2].snapToLines == 'true') OK
 END OF TEST
 
index 9772f4b..594e6ee 100644 (file)
                     tests:
                     [
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["start", "start", "middle", "end", "middle"],
                         },
                         {
-                            property : "textPosition",
+                            property : "position",
                             values : [10, 50, 80, 30, 60],
                         },
                         {
-                            property : "linePosition",
+                            property : "line",
                             values : [0, 0, 80, 5, -3],
                         },
                         {
                     tests:
                     [
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["middle", "end", "middle"],
                         },
                         {
-                            property : "textPosition",
+                            property : "position",
                             values : [50, 0, 60],
                         },
                         {
-                            property : "linePosition",
+                            property : "line",
                             values : [-1, -1, -3],
                         },
                         {
index 8b53d4e..6bb69ce 100644 (file)
@@ -4,19 +4,19 @@ Tests cue size and alignment from settings.
 *** Testing text track 0
 EXPECTED (cues.length == '3') OK
 EXPECTED (cues[0].size == '100') OK
-EXPECTED (cues[0].alignment == 'start') OK
+EXPECTED (cues[0].align == 'start') OK
 EXPECTED (cues[1].size == '10') OK
-EXPECTED (cues[1].alignment == 'end') OK
+EXPECTED (cues[1].align == 'end') OK
 EXPECTED (cues[2].size == '0') OK
-EXPECTED (cues[2].alignment == 'middle') OK
+EXPECTED (cues[2].align == 'middle') OK
 
 *** Testing text track 1
 EXPECTED (cues.length == '3') OK
 EXPECTED (cues[0].size == '100') OK
-EXPECTED (cues[0].alignment == 'middle') OK
+EXPECTED (cues[0].align == 'middle') OK
 EXPECTED (cues[1].size == '100') OK
-EXPECTED (cues[1].alignment == 'end') OK
+EXPECTED (cues[1].align == 'end') OK
 EXPECTED (cues[2].size == '100') OK
-EXPECTED (cues[2].alignment == 'middle') OK
+EXPECTED (cues[2].align == 'middle') OK
 END OF TEST
 
index 8f66c0e..5eb3d7f 100644 (file)
@@ -31,7 +31,7 @@
                             values : [100, 10, 0],
                         },
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["start", "end", "middle"],
                         },
                     ],
@@ -54,7 +54,7 @@
                             values : [100, 100, 100],
                         },
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["middle", "end", "middle"],
                         },
                     ],
index 95a1ecc..54717b1 100644 (file)
@@ -3,32 +3,32 @@ Tests cue vertical alignment (direction) from settings.
 
 *** Testing text track 0
 EXPECTED (cues.length == '3') OK
-EXPECTED (cues[0].direction == 'vertical') OK
-EXPECTED (cues[0].alignment == 'middle') OK
-EXPECTED (cues[0].textPosition == '50') OK
-EXPECTED (cues[1].direction == 'vertical-lr') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[2].direction == 'vertical') OK
-EXPECTED (cues[2].alignment == 'start') OK
-EXPECTED (cues[2].textPosition == '0') OK
+EXPECTED (cues[0].vertical == 'rl') OK
+EXPECTED (cues[0].align == 'middle') OK
+EXPECTED (cues[0].position == '50') OK
+EXPECTED (cues[1].vertical == 'lr') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[2].vertical == 'rl') OK
+EXPECTED (cues[2].align == 'start') OK
+EXPECTED (cues[2].position == '0') OK
 
 *** Testing text track 1
 EXPECTED (cues.length == '3') OK
-EXPECTED (cues[0].direction == 'vertical') OK
-EXPECTED (cues[0].alignment == 'middle') OK
-EXPECTED (cues[0].textPosition == '50') OK
-EXPECTED (cues[1].direction == 'vertical-lr') OK
-EXPECTED (cues[1].alignment == 'middle') OK
-EXPECTED (cues[1].textPosition == '50') OK
-EXPECTED (cues[2].direction == 'vertical') OK
-EXPECTED (cues[2].alignment == 'start') OK
-EXPECTED (cues[2].textPosition == '0') OK
+EXPECTED (cues[0].vertical == 'rl') OK
+EXPECTED (cues[0].align == 'middle') OK
+EXPECTED (cues[0].position == '50') OK
+EXPECTED (cues[1].vertical == 'lr') OK
+EXPECTED (cues[1].align == 'middle') OK
+EXPECTED (cues[1].position == '50') OK
+EXPECTED (cues[2].vertical == 'rl') OK
+EXPECTED (cues[2].align == 'start') OK
+EXPECTED (cues[2].position == '0') OK
 
 *** Testing text track 2
 EXPECTED (cues.length == '3') OK
-EXPECTED (cues[0].direction == 'horizontal') OK
-EXPECTED (cues[1].direction == 'horizontal') OK
-EXPECTED (cues[2].direction == 'horizontal') OK
+EXPECTED (cues[0].vertical == '') OK
+EXPECTED (cues[1].vertical == '') OK
+EXPECTED (cues[2].vertical == '') OK
 END OF TEST
 
index 625f2af..523f813 100644 (file)
                     tests:
                     [
                         {
-                            property : "direction",
-                            values : ["vertical", "vertical-lr", "vertical"],
+                            property : "vertical",
+                            values : ["rl", "lr", "rl"],
                         },
                         {
-                            property : "alignment",
+                            property : "align",
                             values : ["middle", "middle", "start"],
                         },
                         {
-                            property : "textPosition",
+                            property : "position",
                             values : [50, 50, 0],
                         },
                     ],
@@ -55,8 +55,8 @@
                     tests:
                     [
                         {
-                            property : "direction",
-                            values : ["horizontal", "horizontal", "horizontal"],
+                            property : "vertical",
+                            values : ["", "", ""],
                         },
                     ],
                 };
index 47cc92f..28d6ad1 100644 (file)
@@ -1,3 +1,37 @@
+2012-02-24  Eric Carlson  <eric.carlson@apple.com>
+
+        Update TextTrackCue API
+        https://bugs.webkit.org/show_bug.cgi?id=79368
+
+        Change TextTrackCue.alignment to .align, TextTrackCue.linePosition and .textPosition to .line
+        and .position, and TextTrackCue.direction to .vertical. Change direction values "horizontal" 
+        to "","vertical" to "rl", and "vertical-lr" to "lr".
+
+        Reviewed by Eric Seidel.
+
+        No new tests, updated existing tests for API changes.
+
+        * html/track/TextTrackCue.cpp:
+        (WebCore::horizontalKeyword):
+        (WebCore::verticalGrowingLeftKeyword):
+        (WebCore):
+        (WebCore::verticalGrowingRightKeyword):
+        (WebCore::verticalKeywordOLD):
+        (WebCore::verticallrKeywordOLD):
+        (WebCore::TextTrackCue::TextTrackCue):
+        (WebCore::TextTrackCue::vertical):
+        (WebCore::TextTrackCue::setVertical):
+        (WebCore::TextTrackCue::setLine):
+        (WebCore::TextTrackCue::setPosition):
+        (WebCore::TextTrackCue::align):
+        (WebCore::TextTrackCue::setAlign):
+        (WebCore::TextTrackCue::parseSettings):
+        * html/track/TextTrackCue.h:
+        (TextTrackCue):
+        (WebCore::TextTrackCue::line):
+        (WebCore::TextTrackCue::position):
+        * html/track/TextTrackCue.idl:
+
 2012-02-24  Adrienne Walker  <enne@google.com>
 
         Fix uninitialized variables in HarfBuzzShaperBase
index 59c3f3e..6630ca1 100644 (file)
@@ -46,51 +46,66 @@ namespace WebCore {
 
 static const int invalidCueIndex = -1;
 
-static const AtomicString& startKeyword()
+static const String& startKeyword()
 {
-    DEFINE_STATIC_LOCAL(const AtomicString, start, ("start"));
+    DEFINE_STATIC_LOCAL(const String, start, ("start"));
     return start;
 }
 
-static const AtomicString& middleKeyword()
+static const String& middleKeyword()
 {
-    DEFINE_STATIC_LOCAL(const AtomicString, middle, ("middle"));
+    DEFINE_STATIC_LOCAL(const String, middle, ("middle"));
     return middle;
 }
 
-static const AtomicString& endKeyword()
+static const String& endKeyword()
 {
-    DEFINE_STATIC_LOCAL(const AtomicString, end, ("end"));
+    DEFINE_STATIC_LOCAL(const String, end, ("end"));
     return end;
 }
 
-static const AtomicString& horizontalKeyword()
+static const String& horizontalKeyword()
 {
-    DEFINE_STATIC_LOCAL(const AtomicString, horizontal, ("horizontal"));
-    return horizontal;
+    return emptyString();
 }
 
-static const AtomicString& verticalKeyword()
+static const String& verticalGrowingLeftKeyword()
 {
-    DEFINE_STATIC_LOCAL(const AtomicString, vertical, ("vertical"));
+    DEFINE_STATIC_LOCAL(const String, vertical, ("rl"));
     return vertical;
 }
-static const AtomicString& verticallrKeyword()
+
+static const String& verticalGrowingRightKeyword()
+{
+    DEFINE_STATIC_LOCAL(const String, verticallr, ("lr"));
+    return verticallr;
+}
+
+// FIXME: remove this once https://bugs.webkit.org/show_bug.cgi?id=78706 has been fixed.
+static const String& verticalKeywordOLD()
+{
+    DEFINE_STATIC_LOCAL(const String, vertical, ("vertical"));
+    return vertical;
+}
+
+// FIXME: remove this once https://bugs.webkit.org/show_bug.cgi?id=78706 has been fixed.
+static const String& verticallrKeywordOLD()
 {
-    DEFINE_STATIC_LOCAL(const AtomicString, verticallr, ("vertical-lr"));
+    DEFINE_STATIC_LOCAL(const String, verticallr, ("vertical-lr"));
     return verticallr;
 }
     
+
 TextTrackCue::TextTrackCue(ScriptExecutionContext* context, const String& id, double start, double end, const String& content, const String& settings, bool pauseOnExit)
     : m_id(id)
     , m_startTime(start)
     , m_endTime(end)
     , m_content(content)
-    , m_writingDirection(Horizontal)
     , m_linePosition(-1)
     , m_textPosition(50)
     , m_cueSize(100)
     , m_cueIndex(invalidCueIndex)
+    , m_writingDirection(Horizontal)
     , m_cueAlignment(Middle)
     , m_scriptExecutionContext(context)
     , m_isActive(false)
@@ -166,35 +181,35 @@ void TextTrackCue::setPauseOnExit(bool value)
     cueDidChange();
 }
 
-const String& TextTrackCue::direction() const
+const String& TextTrackCue::vertical() const
 {
     switch (m_writingDirection) {
     case Horizontal: 
         return horizontalKeyword();
     case VerticalGrowingLeft:
-        return verticalKeyword();
+        return verticalGrowingLeftKeyword();
     case VerticalGrowingRight:
-        return verticallrKeyword();
+        return verticalGrowingRightKeyword();
     default:
         ASSERT_NOT_REACHED();
         return emptyString();
     }
 }
 
-void TextTrackCue::setDirection(const String& value, ExceptionCode& ec)
+void TextTrackCue::setVertical(const String& value, ExceptionCode& ec)
 {
-    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-direction
+    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-vertical
     // On setting, the text track cue writing direction must be set to the value given 
     // in the first cell of the row in the table above whose second cell is a 
     // case-sensitive match for the new value, if any. If none of the values match, then
     // the user agent must instead throw a SyntaxError exception.
     
-    Direction direction = m_writingDirection;
+    WritingDirection direction = m_writingDirection;
     if (value == horizontalKeyword())
         direction = Horizontal;
-    else if (value == verticalKeyword())
+    else if (value == verticalGrowingLeftKeyword())
         direction = VerticalGrowingLeft;
-    else if (value == verticallrKeyword())
+    else if (value == verticalGrowingRightKeyword())
         direction = VerticalGrowingRight;
     else
         ec = SYNTAX_ERR;
@@ -217,9 +232,9 @@ void TextTrackCue::setSnapToLines(bool value)
     cueDidChange();
 }
 
-void TextTrackCue::setLinePosition(int position, ExceptionCode& ec)
+void TextTrackCue::setLine(int position, ExceptionCode& ec)
 {
-    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-lineposition
+    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-line
     // On setting, if the text track cue snap-to-lines flag is not set, and the new
     // value is negative or greater than 100, then throw an IndexSizeError exception.
     if (!m_snapToLines && (position < 0 || position > 100)) {
@@ -236,9 +251,9 @@ void TextTrackCue::setLinePosition(int position, ExceptionCode& ec)
     cueDidChange();
 }
 
-void TextTrackCue::setTextPosition(int position, ExceptionCode& ec)
+void TextTrackCue::setPosition(int position, ExceptionCode& ec)
 {
-    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-lineposition
+    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-position
     // On setting, if the new value is negative or greater than 100, then throw an IndexSizeError exception.
     // Otherwise, set the text track cue text position to the new value.
     if (position < 0 || position > 100) {
@@ -274,7 +289,7 @@ void TextTrackCue::setSize(int size, ExceptionCode& ec)
     cueDidChange();
 }
 
-const String& TextTrackCue::alignment() const
+const String& TextTrackCue::align() const
 {
     switch (m_cueAlignment) {
     case Start:
@@ -289,9 +304,9 @@ const String& TextTrackCue::alignment() const
     }
 }
 
-void TextTrackCue::setAlignment(const String& value, ExceptionCode& ec)
+void TextTrackCue::setAlign(const String& value, ExceptionCode& ec)
 {
-    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-alignment
+    // http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrackcue-align
     // On setting, the text track cue alignment must be set to the value given in the 
     // first cell of the row in the table above whose second cell is a case-sensitive
     // match for the new value, if any. If none of the values match, then the user
@@ -405,9 +420,9 @@ void TextTrackCue::parseSettings(const String& input)
             {
             // 1-3 - Collect the next word and set the writing direction accordingly.
             String writingDirection = WebVTTParser::collectWord(input, &position);
-            if (writingDirection == verticalKeyword())
+            if (writingDirection == verticalKeywordOLD())
                 m_writingDirection = VerticalGrowingLeft;
-            else if (writingDirection == verticallrKeyword())
+            else if (writingDirection == verticallrKeywordOLD())
                 m_writingDirection = VerticalGrowingRight;
             }
             break;
index b3e0f32..156c14b 100644 (file)
@@ -51,9 +51,6 @@ public:
         return adoptRef(new TextTrackCue(context, id, start, end, content, settings, pauseOnExit));
     }
 
-    enum Direction { Horizontal, VerticalGrowingLeft, VerticalGrowingRight };
-    enum Alignment { Start, Middle, End };
-
     virtual ~TextTrackCue();
 
     TextTrack* track() const;
@@ -71,23 +68,23 @@ public:
     bool pauseOnExit() const { return m_pauseOnExit; }
     void setPauseOnExit(bool);
 
-    const String& direction() const;
-    void setDirection(const String&, ExceptionCode&);
+    const String& vertical() const;
+    void setVertical(const String&, ExceptionCode&);
 
     bool snapToLines() const { return m_snapToLines; }
     void setSnapToLines(bool);
 
-    int linePosition() const { return m_linePosition; }
-    void setLinePosition(int, ExceptionCode&);
+    int line() const { return m_linePosition; }
+    void setLine(int, ExceptionCode&);
 
-    int textPosition() const { return m_textPosition; }
-    void setTextPosition(int, ExceptionCode&);
+    int position() const { return m_textPosition; }
+    void setPosition(int, ExceptionCode&);
 
     int size() const { return m_cueSize; }
     void setSize(int, ExceptionCode&);
 
-    const String& alignment() const;
-    void setAlignment(const String&, ExceptionCode&);
+    const String& align() const;
+    void setAlign(const String&, ExceptionCode&);
 
     const String& text() const { return m_content; }
     void setText(const String&);
@@ -132,13 +129,17 @@ private:
     double m_startTime;
     double m_endTime;
     String m_content;
-    Direction m_writingDirection;
     int m_linePosition;
     int m_textPosition;
     int m_cueSize;
     int m_cueIndex;
 
+    enum WritingDirection { Horizontal, VerticalGrowingLeft, VerticalGrowingRight };
+    WritingDirection m_writingDirection;
+
+    enum Alignment { Start, Middle, End };
     Alignment m_cueAlignment;
+
     RefPtr<DocumentFragment> m_documentFragment;
     RefPtr<TextTrack> m_track;
 
index 12a8279..d552525 100644 (file)
@@ -42,16 +42,16 @@ module html {
         attribute double endTime;
         attribute boolean pauseOnExit;
 
-        attribute DOMString direction
+        attribute DOMString vertical
             setter raises (DOMException);
         attribute boolean snapToLines;
-        attribute long linePosition
+        attribute long line
             setter raises (DOMException);
-        attribute long textPosition
+        attribute long position
             setter raises (DOMException);
         attribute long size
             setter raises (DOMException);
-        attribute DOMString alignment
+        attribute DOMString align
             setter raises (DOMException);
 
         attribute DOMString text;
@@ -71,4 +71,4 @@ module html {
             raises(EventException);
     };
 
-}
\ No newline at end of file
+}