Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(154)

Unified Diff: third_party/WebKit/Source/core/html/HTMLMediaElement.cpp

Issue 1712903002: Remove prefixed EME. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove all prefixed-related code from the EME browser tests. Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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());
philipj_slow 2016/02/20 12:52:20 Drop they keySystem argument from WebMimeRegistry.
ddorwin 2016/02/22 18:30:46 Done.
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 {

Powered by Google App Engine
This is Rietveld 408576698