| 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 {
|
|
|