Diff
Modified: trunk/LayoutTests/ChangeLog (101087 => 101088)
--- trunk/LayoutTests/ChangeLog 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/ChangeLog 2011-11-23 18:12:59 UTC (rev 101088)
@@ -1,3 +1,22 @@
+2011-11-23 Sheriff Bot <webkit.review....@gmail.com>
+
+ Unreviewed, rolling out r101057.
+ http://trac.webkit.org/changeset/101057
+ https://bugs.webkit.org/show_bug.cgi?id=73027
+
+ Broke Chromium Windows build (Requested by aklein on #webkit).
+
+ * media/track/track-add-track-expected.txt:
+ * media/track/track-add-track.html:
+ * media/track/track-constants-expected.txt:
+ * media/track/track-constants.html:
+ * media/track/track-load-error-readyState-expected.txt:
+ * media/track/track-load-error-readyState.html:
+ * media/track/track-load-from-element-readyState-expected.txt:
+ * media/track/track-load-from-element-readyState.html:
+ * media/track/track-load-from-src-readyState-expected.txt:
+ * media/track/track-load-from-src-readyState.html:
+
2011-11-23 Philippe Normand <pnorm...@igalia.com>
Unreviewed, skip another svg test on GTK showing signed zero issues.
Modified: trunk/LayoutTests/media/track/track-add-track-expected.txt (101087 => 101088)
--- trunk/LayoutTests/media/track/track-add-track-expected.txt 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-add-track-expected.txt 2011-11-23 18:12:59 UTC (rev 101088)
@@ -5,7 +5,7 @@
RUN(video.appendChild(trackElement))
RUN(trackElement.src = '')
EXPECTED (video.textTracks.length == '1') OK
-EXPECTED (trackElement.readyState == '0') OK
+EXPECTED (tracks[0].readyState == '0') OK
EVENT(addtrack)
EXPECTED (event.target == '[object TextTrackList]') OK
EXPECTED (event instanceof window.TrackEvent == 'true') OK
@@ -14,10 +14,12 @@
** Test a TextTrack created with video.addTrack().
RUN(tracks.push(video.addTrack('captions', 'Caption Track', 'en')))
EXPECTED (video.textTracks.length == '2') OK
+EXPECTED (tracks[1].readyState == '0') OK
EVENT(addtrack)
EXPECTED (event.target == '[object TextTrackList]') OK
EXPECTED (event instanceof window.TrackEvent == 'true') OK
EXPECTED (event.track == '[object TextTrack]') OK
+EXPECTED (tracks[1].readyState == '0') OK
END OF TEST
Modified: trunk/LayoutTests/media/track/track-add-track.html (101087 => 101088)
--- trunk/LayoutTests/media/track/track-add-track.html 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-add-track.html 2011-11-23 18:12:59 UTC (rev 101088)
@@ -21,7 +21,9 @@
consoleWrite("<br>** Test a TextTrack created with video.addTrack().");
run("tracks.push(video.addTrack('captions', 'Caption Track', 'en'))");
testExpected("video.textTracks.length", 2);
+ testExpected("tracks[1].readyState", TextTrack.NONE);
} else {
+ testExpected("tracks[1].readyState", TextTrack.NONE);
consoleWrite("");
endTest();
}
@@ -42,7 +44,7 @@
run("trackElement.src = ''");
testExpected("video.textTracks.length", 1);
- testExpected("trackElement.readyState", HTMLTrackElement.NONE);
+ testExpected("tracks[0].readyState", TextTrack.NONE);
}
</script>
Modified: trunk/LayoutTests/media/track/track-constants-expected.txt (101087 => 101088)
--- trunk/LayoutTests/media/track/track-constants-expected.txt 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-constants-expected.txt 2011-11-23 18:12:59 UTC (rev 101088)
@@ -1,9 +1,9 @@
Test TextTrack constants.
-EXPECTED (HTMLTrackElement.NONE == '0') OK
-EXPECTED (HTMLTrackElement.LOADING == '1') OK
-EXPECTED (HTMLTrackElement.LOADED == '2') OK
-EXPECTED (HTMLTrackElement.ERROR == '3') OK
+EXPECTED (TextTrack.NONE == '0') OK
+EXPECTED (TextTrack.LOADING == '1') OK
+EXPECTED (TextTrack.LOADED == '2') OK
+EXPECTED (TextTrack.ERROR == '3') OK
EXPECTED (TextTrack.DISABLED == '0') OK
EXPECTED (TextTrack.HIDDEN == '1') OK
Modified: trunk/LayoutTests/media/track/track-constants.html (101087 => 101088)
--- trunk/LayoutTests/media/track/track-constants.html 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-constants.html 2011-11-23 18:12:59 UTC (rev 101088)
@@ -5,10 +5,10 @@
<script type="text/_javascript_">
function doTest ()
{
- testExpected("HTMLTrackElement.NONE", 0);
- testExpected("HTMLTrackElement.LOADING", 1);
- testExpected("HTMLTrackElement.LOADED", 2);
- testExpected("HTMLTrackElement.ERROR", 3);
+ testExpected("TextTrack.NONE", 0);
+ testExpected("TextTrack.LOADING", 1);
+ testExpected("TextTrack.LOADED", 2);
+ testExpected("TextTrack.ERROR", 3);
consoleWrite("");
testExpected("TextTrack.DISABLED", 0);
Modified: trunk/LayoutTests/media/track/track-load-error-readyState-expected.txt (101087 => 101088)
--- trunk/LayoutTests/media/track/track-load-error-readyState-expected.txt 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-load-error-readyState-expected.txt 2011-11-23 18:12:59 UTC (rev 101088)
@@ -1,6 +1,6 @@
Tests the error event on HTMLTrackElement and ERROR readyState on TextTrack.
EVENT(error)
-EXPECTED (track.readyState == '3') OK
+EXPECTED (track.track.readyState == '3') OK
END OF TEST
Modified: trunk/LayoutTests/media/track/track-load-error-readyState.html (101087 => 101088)
--- trunk/LayoutTests/media/track/track-load-error-readyState.html 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-load-error-readyState.html 2011-11-23 18:12:59 UTC (rev 101088)
@@ -11,7 +11,7 @@
{
consoleWrite("EVENT(error)");
track = document.getElementById('testTrackError');
- testExpected("track.readyState", HTMLTrackElement.ERROR);
+ testExpected("track.track.readyState", TextTrack.ERROR);
endTest();
}
Modified: trunk/LayoutTests/media/track/track-load-from-element-readyState-expected.txt (101087 => 101088)
--- trunk/LayoutTests/media/track/track-load-from-element-readyState-expected.txt 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-load-from-element-readyState-expected.txt 2011-11-23 18:12:59 UTC (rev 101088)
@@ -1,6 +1,6 @@
Tests the load event on HTMLTrackElement and LOADED readyState on TextTrack when src is set on the element.
EVENT(load)
-EXPECTED (track.readyState == '2') OK
+EXPECTED (track.track.readyState == '2') OK
END OF TEST
Modified: trunk/LayoutTests/media/track/track-load-from-element-readyState.html (101087 => 101088)
--- trunk/LayoutTests/media/track/track-load-from-element-readyState.html 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-load-from-element-readyState.html 2011-11-23 18:12:59 UTC (rev 101088)
@@ -11,7 +11,7 @@
{
consoleWrite("EVENT(load)");
track = document.getElementById('testTrackSrc');
- testExpected("track.readyState", HTMLTrackElement.LOADED);
+ testExpected("track.track.readyState", TextTrack.LOADED);
endTest();
}
Modified: trunk/LayoutTests/media/track/track-load-from-src-readyState-expected.txt (101087 => 101088)
--- trunk/LayoutTests/media/track/track-load-from-src-readyState-expected.txt 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-load-from-src-readyState-expected.txt 2011-11-23 18:12:59 UTC (rev 101088)
@@ -1,7 +1,7 @@
Tests the load event on HTMLTrackElement and LOADED readyState on TextTrack when src is set from _javascript_.
-EXPECTED (track.readyState == '0') OK
+EXPECTED (track.track.readyState == '0') OK
EVENT(load)
-EXPECTED (track.readyState == '2') OK
+EXPECTED (track.track.readyState == '2') OK
END OF TEST
Modified: trunk/LayoutTests/media/track/track-load-from-src-readyState.html (101087 => 101088)
--- trunk/LayoutTests/media/track/track-load-from-src-readyState.html 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/LayoutTests/media/track/track-load-from-src-readyState.html 2011-11-23 18:12:59 UTC (rev 101088)
@@ -16,12 +16,12 @@
function trackLoaded()
{
consoleWrite("EVENT(load)");
- testExpected("track.readyState", HTMLTrackElement.LOADED);
+ testExpected("track.track.readyState", TextTrack.LOADED);
endTest();
}
track = document.getElementById('testTrackEmpty');
- testExpected("track.readyState", HTMLTrackElement.NONE);
+ testExpected("track.track.readyState", TextTrack.NONE);
track.addEventListener("load", function () { trackLoaded(); }, true);
track.src = ""
Modified: trunk/Source/WebCore/ChangeLog (101087 => 101088)
--- trunk/Source/WebCore/ChangeLog 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/Source/WebCore/ChangeLog 2011-11-23 18:12:59 UTC (rev 101088)
@@ -1,3 +1,28 @@
+2011-11-23 Sheriff Bot <webkit.review....@gmail.com>
+
+ Unreviewed, rolling out r101057.
+ http://trac.webkit.org/changeset/101057
+ https://bugs.webkit.org/show_bug.cgi?id=73027
+
+ Broke Chromium Windows build (Requested by aklein on #webkit).
+
+ * html/HTMLTrackElement.cpp:
+ (WebCore::HTMLTrackElement::HTMLTrackElement):
+ (WebCore::HTMLTrackElement::didCompleteLoad):
+ (WebCore::HTMLTrackElement::textTrackReadyStateChanged):
+ * html/HTMLTrackElement.h:
+ * html/HTMLTrackElement.idl:
+ * html/LoadableTextTrack.cpp:
+ (WebCore::LoadableTextTrack::loadTimerFired):
+ (WebCore::LoadableTextTrack::cueLoadingStarted):
+ (WebCore::LoadableTextTrack::cueLoadingCompleted):
+ * html/TextTrack.cpp:
+ (WebCore::TextTrack::TextTrack):
+ (WebCore::TextTrack::setReadyState):
+ * html/TextTrack.h:
+ (WebCore::TextTrack::readyState):
+ * html/TextTrack.idl:
+
2011-11-23 Yury Semikhatsky <yu...@chromium.org>
Unreviewed. Build fix. Added missing ENABLE(WORKERS) guards.
Modified: trunk/Source/WebCore/html/HTMLTrackElement.cpp (101087 => 101088)
--- trunk/Source/WebCore/html/HTMLTrackElement.cpp 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/Source/WebCore/html/HTMLTrackElement.cpp 2011-11-23 18:12:59 UTC (rev 101088)
@@ -43,7 +43,6 @@
inline HTMLTrackElement::HTMLTrackElement(const QualifiedName& tagName, Document* document)
: HTMLElement(tagName, document)
- , m_readyState(HTMLTrackElement::NONE)
{
LOG(Media, "HTMLTrackElement::HTMLTrackElement - %p", this);
ASSERT(hasTagName(trackTag));
@@ -207,17 +206,14 @@
void HTMLTrackElement::didCompleteLoad(LoadableTextTrack*, bool loadingFailed)
{
- loadingFailed ? setReadyState(HTMLTrackElement::ERROR) : setReadyState(HTMLTrackElement::LOADED);
-
ExceptionCode ec = 0;
dispatchEvent(Event::create(loadingFailed ? eventNames().errorEvent : eventNames().loadEvent, false, false), ec);
}
-
-void HTMLTrackElement::setReadyState(ReadyState state)
+
+void HTMLTrackElement::textTrackReadyStateChanged(TextTrack* track)
{
- m_readyState = state;
if (HTMLMediaElement* parent = mediaElement())
- return parent->textTrackReadyStateChanged(m_track.get());
+ return parent->textTrackReadyStateChanged(track);
}
void HTMLTrackElement::textTrackKindChanged(TextTrack* track)
Modified: trunk/Source/WebCore/html/HTMLTrackElement.h (101087 => 101088)
--- trunk/Source/WebCore/html/HTMLTrackElement.h 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/Source/WebCore/html/HTMLTrackElement.h 2011-11-23 18:12:59 UTC (rev 101088)
@@ -53,17 +53,12 @@
bool isDefault() const;
void setIsDefault(bool);
-
- enum ReadyState { NONE = 0, LOADING = 1, LOADED = 2, ERROR = 3 };
- ReadyState readyState() const { return m_readyState; }
TextTrack* track();
void scheduleLoad();
virtual bool canLoadUrl(LoadableTextTrack*, const KURL&);
virtual void didCompleteLoad(LoadableTextTrack*, bool /* loadingFailed */);
-
- void setReadyState(ReadyState);
private:
HTMLTrackElement(const QualifiedName&, Document*);
@@ -84,6 +79,7 @@
HTMLMediaElement* mediaElement() const;
// TextTrackClient
+ virtual void textTrackReadyStateChanged(TextTrack*);
virtual void textTrackModeChanged(TextTrack*);
virtual void textTrackKindChanged(TextTrack*);
virtual void textTrackAddCues(TextTrack*, const TextTrackCueList*);
@@ -93,7 +89,6 @@
LoadableTextTrack* ensureTrack();
- HTMLTrackElement::ReadyState m_readyState;
RefPtr<LoadableTextTrack> m_track;
};
Modified: trunk/Source/WebCore/html/HTMLTrackElement.idl (101087 => 101088)
--- trunk/Source/WebCore/html/HTMLTrackElement.idl 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/Source/WebCore/html/HTMLTrackElement.idl 2011-11-23 18:12:59 UTC (rev 101088)
@@ -28,18 +28,12 @@
Conditional=VIDEO_TRACK,
EnabledAtRuntime=webkitVideoTrack
] HTMLTrackElement : HTMLElement {
- attribute [Reflect, URL] DOMString src;
- attribute DOMString kind;
- attribute DOMString srclang;
- attribute DOMString label;
- attribute [Reflect=default] boolean isDefault;
+ attribute [Reflect, URL] DOMString src;
+ attribute DOMString kind;
+ attribute DOMString srclang;
+ attribute DOMString label;
+ attribute [Reflect=default] boolean isDefault;
- const unsigned short NONE = 0;
- const unsigned short LOADING = 1;
- const unsigned short LOADED = 2;
- const unsigned short ERROR = 3;
- readonly attribute unsigned short readyState;
-
- readonly attribute TextTrack track;
+ readonly attribute TextTrack track;
};
}
Modified: trunk/Source/WebCore/html/LoadableTextTrack.cpp (101087 => 101088)
--- trunk/Source/WebCore/html/LoadableTextTrack.cpp 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/Source/WebCore/html/LoadableTextTrack.cpp 2011-11-23 18:12:59 UTC (rev 101088)
@@ -64,16 +64,13 @@
void LoadableTextTrack::loadTimerFired(Timer<LoadableTextTrack>*)
{
- if (!m_trackElement)
- return;
-
- m_trackElement->setReadyState(HTMLTrackElement::LOADING);
+ setReadyState(TextTrack::LOADING);
if (m_loader)
m_loader->cancelLoad();
- if (!m_trackElement->canLoadUrl(this, m_url)) {
- m_trackElement->setReadyState(HTMLTrackElement::ERROR);
+ if (!m_trackElement || !m_trackElement->canLoadUrl(this, m_url)) {
+ setReadyState(TextTrack::HTML_ERROR);
return;
}
@@ -103,18 +100,17 @@
{
ASSERT_UNUSED(loader, m_loader == loader);
- if (!m_trackElement)
- return;
- m_trackElement->setReadyState(HTMLTrackElement::LOADING);
+ setReadyState(TextTrack::LOADING);
}
void LoadableTextTrack::cueLoadingCompleted(TextTrackLoader* loader, bool loadingFailed)
{
ASSERT_UNUSED(loader, m_loader == loader);
- if (!m_trackElement)
- return;
- m_trackElement->didCompleteLoad(this, loadingFailed);
+ loadingFailed ? setReadyState(TextTrack::HTML_ERROR) : setReadyState(TextTrack::LOADED);
+
+ if (m_trackElement)
+ m_trackElement->didCompleteLoad(this, loadingFailed);
}
void LoadableTextTrack::fireCueChangeEvent()
Modified: trunk/Source/WebCore/html/TextTrack.cpp (101087 => 101088)
--- trunk/Source/WebCore/html/TextTrack.cpp 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/Source/WebCore/html/TextTrack.cpp 2011-11-23 18:12:59 UTC (rev 101088)
@@ -75,6 +75,7 @@
: TrackBase(context, TrackBase::TextTrack)
, m_label(label)
, m_language(language)
+ , m_readyState(TextTrack::NONE)
, m_mode(TextTrack::HIDDEN)
, m_client(client)
, m_trackType(type)
@@ -118,6 +119,13 @@
m_client->textTrackKindChanged(this);
}
+void TextTrack::setReadyState(ReadyState state)
+{
+ m_readyState = state;
+ if (m_client)
+ m_client->textTrackReadyStateChanged(this);
+}
+
void TextTrack::setMode(unsigned short mode, ExceptionCode& ec)
{
// 4.8.10.12.5 On setting the mode, if the new value is not either 0, 1, or 2,
Modified: trunk/Source/WebCore/html/TextTrack.h (101087 => 101088)
--- trunk/Source/WebCore/html/TextTrack.h 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/Source/WebCore/html/TextTrack.h 2011-11-23 18:12:59 UTC (rev 101088)
@@ -43,6 +43,7 @@
class TextTrackClient {
public:
virtual ~TextTrackClient() { }
+ virtual void textTrackReadyStateChanged(TextTrack*) = 0;
virtual void textTrackKindChanged(TextTrack*) = 0;
virtual void textTrackModeChanged(TextTrack*) = 0;
virtual void textTrackAddCues(TextTrack*, const TextTrackCueList*) = 0;
@@ -75,6 +76,9 @@
String language() const { return m_language; }
void setLanguage(const String& language) { m_language = language; }
+ enum ReadyState { NONE = 0, LOADING = 1, LOADED = 2, HTML_ERROR = 3 };
+ ReadyState readyState() const { return m_readyState; }
+
enum Mode { DISABLED = 0, HIDDEN = 1, SHOWING = 2 };
Mode mode() const { return m_mode; }
void setMode(unsigned short, ExceptionCode&);
@@ -103,12 +107,15 @@
protected:
TextTrack(ScriptExecutionContext*, TextTrackClient*, const String& kind, const String& label, const String& language, TextTrackType);
+ void setReadyState(ReadyState);
+
RefPtr<TextTrackCueList> m_cues;
private:
String m_kind;
String m_label;
String m_language;
+ TextTrack::ReadyState m_readyState;
TextTrack::Mode m_mode;
TextTrackClient* m_client;
TextTrackType m_trackType;
Modified: trunk/Source/WebCore/html/TextTrack.idl (101087 => 101088)
--- trunk/Source/WebCore/html/TextTrack.idl 2011-11-23 17:49:43 UTC (rev 101087)
+++ trunk/Source/WebCore/html/TextTrack.idl 2011-11-23 18:12:59 UTC (rev 101088)
@@ -34,6 +34,13 @@
readonly attribute DOMString label;
readonly attribute DOMString language;
+ const unsigned short NONE = 0;
+ const unsigned short LOADING = 1;
+ const unsigned short LOADED = 2;
+ // Reflect is used for TextTrack::ERROR because it conflicts with a windows define.
+ const [Reflect=HTML_ERROR] unsigned short ERROR = 3;
+ readonly attribute unsigned short readyState;
+
const unsigned short DISABLED = 0;
const unsigned short HIDDEN = 1;
const unsigned short SHOWING = 2;
