Index: third_party/WebKit/Source/core/html/HTMLMediaElement.cpp |
diff --git a/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp b/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp |
index efeaceaddd63eecdb4b7d2f1a3ec83db944fcc60..54026bb8e4d85b2811627023fdc6324f52d5594a 100644 |
--- a/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp |
+++ b/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp |
@@ -213,7 +213,7 @@ static const AtomicString& VideoKindToString(WebMediaPlayerClient::VideoTrackKin |
return emptyAtom; |
} |
-static bool canLoadURL(const KURL& url, const ContentType& contentType, const String& keySystem) |
+static bool canLoadURL(const KURL& url, const ContentType& contentType) |
{ |
DEFINE_STATIC_LOCAL(const String, codecs, ("codecs")); |
@@ -234,7 +234,7 @@ static bool canLoadURL(const KURL& url, const ContentType& contentType, const St |
// when used with parameters, e.g. "application/octet-stream;codecs=theora", is a type that the user agent knows |
// it cannot render. |
if (contentMIMEType != "application/octet-stream" || contentTypeCodecs.isEmpty()) { |
- WebMimeRegistry::SupportsType supported = Platform::current()->mimeRegistry()->supportsMediaMIMEType(contentMIMEType, contentTypeCodecs, keySystem.lower()); |
+ WebMimeRegistry::SupportsType supported = Platform::current()->mimeRegistry()->supportsMediaMIMEType(contentMIMEType, contentTypeCodecs, String()); |
return supported > WebMimeRegistry::IsNotSupported; |
} |
@@ -247,7 +247,7 @@ void HTMLMediaElement::recordAutoplayMetric(AutoplayMetrics metric) |
autoplayHistogram.count(metric); |
} |
-WebMimeRegistry::SupportsType HTMLMediaElement::supportsType(const ContentType& contentType, const String& keySystem) |
+WebMimeRegistry::SupportsType HTMLMediaElement::supportsType(const ContentType& contentType) |
{ |
DEFINE_STATIC_LOCAL(const String, codecs, ("codecs")); |
@@ -258,7 +258,6 @@ WebMimeRegistry::SupportsType HTMLMediaElement::supportsType(const ContentType& |
// The codecs string is not lower-cased because MP4 values are case sensitive |
// per http://tools.ietf.org/html/rfc4281#page-7. |
String typeCodecs = contentType.parameter(codecs); |
- String system = keySystem.lower(); |
if (type.isEmpty()) |
return WebMimeRegistry::IsNotSupported; |
@@ -268,7 +267,7 @@ WebMimeRegistry::SupportsType HTMLMediaElement::supportsType(const ContentType& |
if (type == "application/octet-stream") |
return WebMimeRegistry::IsNotSupported; |
- return Platform::current()->mimeRegistry()->supportsMediaMIMEType(type, typeCodecs, system); |
+ return Platform::current()->mimeRegistry()->supportsMediaMIMEType(type, typeCodecs, String()); |
} |
URLRegistry* HTMLMediaElement::s_mediaStreamRegistry = 0; |
@@ -636,9 +635,9 @@ HTMLMediaElement::NetworkState HTMLMediaElement::networkState() const |
return m_networkState; |
} |
-String HTMLMediaElement::canPlayType(const String& mimeType, const String& keySystem) const |
+String HTMLMediaElement::canPlayType(const String& mimeType) const |
{ |
- WebMimeRegistry::SupportsType support = supportsType(ContentType(mimeType), keySystem); |
+ WebMimeRegistry::SupportsType support = supportsType(ContentType(mimeType)); |
String canPlay; |
// 4.8.10.3 |
@@ -654,7 +653,7 @@ String HTMLMediaElement::canPlayType(const String& mimeType, const String& keySy |
break; |
} |
- WTF_LOG(Media, "HTMLMediaElement::canPlayType(%p, %s, %s) -> %s", this, mimeType.utf8().data(), keySystem.utf8().data(), canPlay.utf8().data()); |
+ WTF_LOG(Media, "HTMLMediaElement::canPlayType(%p, %s) -> %s", this, mimeType.utf8().data(), canPlay.utf8().data()); |
return canPlay; |
} |
@@ -876,11 +875,10 @@ void HTMLMediaElement::selectMediaResource() |
return; |
} |
- // No type or key system information is available when the url comes |
- // from the 'src' attribute so MediaPlayer |
+ // No type is available when the url comes from the 'src' attribute so MediaPlayer |
// will have to pick a media engine based on the file extension. |
ContentType contentType((String())); |
- loadResource(mediaURL, contentType, String()); |
+ loadResource(mediaURL, contentType); |
WTF_LOG(Media, "HTMLMediaElement::selectMediaResource(%p), using 'src' attribute url", this); |
return; |
} |
@@ -892,8 +890,7 @@ void HTMLMediaElement::selectMediaResource() |
void HTMLMediaElement::loadNextSourceChild() |
{ |
ContentType contentType((String())); |
- String keySystem; |
- KURL mediaURL = selectNextSourceChild(&contentType, &keySystem, Complain); |
+ KURL mediaURL = selectNextSourceChild(&contentType, Complain); |
if (!mediaURL.isValid()) { |
waitForSourceChange(); |
return; |
@@ -903,15 +900,15 @@ void HTMLMediaElement::loadNextSourceChild() |
resetMediaPlayerAndMediaSource(); |
m_loadState = LoadingFromSourceElement; |
- loadResource(mediaURL, contentType, keySystem); |
+ loadResource(mediaURL, contentType); |
} |
-void HTMLMediaElement::loadResource(const KURL& url, ContentType& contentType, const String& keySystem) |
+void HTMLMediaElement::loadResource(const KURL& url, ContentType& contentType) |
{ |
ASSERT(isMainThread()); |
ASSERT(isSafeToLoadURL(url, Complain)); |
- WTF_LOG(Media, "HTMLMediaElement::loadResource(%p, %s, %s, %s)", this, urlForLoggingMedia(url).utf8().data(), contentType.raw().utf8().data(), keySystem.utf8().data()); |
+ WTF_LOG(Media, "HTMLMediaElement::loadResource(%p, %s, %s)", this, urlForLoggingMedia(url).utf8().data(), contentType.raw().utf8().data()); |
LocalFrame* frame = document().frame(); |
if (!frame) { |
@@ -963,7 +960,7 @@ void HTMLMediaElement::loadResource(const KURL& url, ContentType& contentType, c |
} |
} |
- if (attemptLoad && canLoadURL(url, contentType, keySystem)) { |
+ if (attemptLoad && canLoadURL(url, contentType)) { |
ASSERT(!webMediaPlayer()); |
if (!m_havePreparedToPlay && effectivePreloadType() == WebMediaPlayer::PreloadNone) { |
@@ -2519,7 +2516,7 @@ bool HTMLMediaElement::havePotentialSourceChild() |
RefPtrWillBeRawPtr<HTMLSourceElement> currentSourceNode = m_currentSourceNode; |
RefPtrWillBeRawPtr<Node> nextNode = m_nextChildNodeToConsider; |
- KURL nextURL = selectNextSourceChild(0, 0, DoNothing); |
+ KURL nextURL = selectNextSourceChild(0, DoNothing); |
m_currentSourceNode = currentSourceNode; |
m_nextChildNodeToConsider = nextNode; |
@@ -2527,7 +2524,7 @@ bool HTMLMediaElement::havePotentialSourceChild() |
return nextURL.isValid(); |
} |
-KURL HTMLMediaElement::selectNextSourceChild(ContentType* contentType, String* keySystem, InvalidURLAction actionIfInvalid) |
+KURL HTMLMediaElement::selectNextSourceChild(ContentType* contentType, InvalidURLAction actionIfInvalid) |
{ |
#if !LOG_DISABLED |
// Don't log if this was just called to find out if there are any valid <source> elements. |
@@ -2548,7 +2545,6 @@ KURL HTMLMediaElement::selectNextSourceChild(ContentType* contentType, String* k |
Node* node; |
HTMLSourceElement* source = 0; |
String type; |
- String system; |
bool lookingForStartNode = m_nextChildNodeToConsider; |
bool canUseSourceElement = false; |
@@ -2578,15 +2574,14 @@ KURL HTMLMediaElement::selectNextSourceChild(ContentType* contentType, String* k |
goto checkAgain; |
type = source->type(); |
- // FIXME(82965): Add support for keySystem in <source> and set system from source. |
if (type.isEmpty() && mediaURL.protocolIsData()) |
type = mimeTypeFromDataURL(mediaURL); |
- if (!type.isEmpty() || !system.isEmpty()) { |
+ if (!type.isEmpty()) { |
#if !LOG_DISABLED |
if (shouldLog) |
- WTF_LOG(Media, "HTMLMediaElement::selectNextSourceChild(%p) - 'type' is '%s' - key system is '%s'", this, type.utf8().data(), system.utf8().data()); |
+ WTF_LOG(Media, "HTMLMediaElement::selectNextSourceChild(%p) - 'type' is '%s'", this, type.utf8().data()); |
#endif |
- if (!supportsType(ContentType(type), system)) |
+ if (!supportsType(ContentType(type))) |
goto checkAgain; |
} |
@@ -2605,8 +2600,6 @@ checkAgain: |
if (canUseSourceElement) { |
if (contentType) |
*contentType = ContentType(type); |
- if (keySystem) |
- *keySystem = system; |
m_currentSourceNode = source; |
m_nextChildNodeToConsider = source->nextSibling(); |
} else { |