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

Unified Diff: Source/platform/graphics/media/MediaPlayer.h

Issue 157423003: Remove the dependency on encryptedmedia from HTMLMediaElement. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Address review comments. Created 6 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: Source/platform/graphics/media/MediaPlayer.h
diff --git a/Source/platform/graphics/media/MediaPlayer.h b/Source/platform/graphics/media/MediaPlayer.h
index db3d3b030f93f5d026a0fe064ccebf98bd261376..5c95724530ffdac21103645be1202d39f441e8eb 100644
--- a/Source/platform/graphics/media/MediaPlayer.h
+++ b/Source/platform/graphics/media/MediaPlayer.h
@@ -92,12 +92,6 @@ public:
// the movie size has changed
virtual void mediaPlayerSizeChanged() = 0;
- enum MediaKeyErrorCode { UnknownError = 1, ClientError, ServiceError, OutputError, HardwareChangeError, DomainError };
- virtual void mediaPlayerKeyAdded(const String& /* keySystem */, const String& /* sessionId */) = 0;
- virtual void mediaPlayerKeyError(const String& /* keySystem */, const String& /* sessionId */, MediaKeyErrorCode, unsigned short /* systemCode */) = 0;
- virtual void mediaPlayerKeyMessage(const String& /* keySystem */, const String& /* sessionId */, const unsigned char* /* message */, unsigned /* messageLength */, const KURL& /* defaultURL */) = 0;
- virtual bool mediaPlayerKeyNeeded(const String& /* contentType */, const unsigned char* /* initData */, unsigned /* initDataLength */) = 0;
-
virtual CORSMode mediaPlayerCORSMode() const = 0;
virtual void mediaPlayerSetWebLayer(blink::WebLayer*) = 0;
@@ -191,12 +185,7 @@ public:
#if ENABLE(WEB_AUDIO)
virtual AudioSourceProvider* audioSourceProvider() = 0;
#endif
-
- enum MediaKeyException { NoError, InvalidPlayerState, KeySystemNotSupported, InvalidAccess };
- virtual MediaKeyException addKey(const String&, const unsigned char*, unsigned, const unsigned char*, unsigned, const String&) = 0;
- virtual MediaKeyException generateKeyRequest(const String&, const unsigned char*, unsigned) = 0;
- virtual MediaKeyException cancelKeyRequest(const String&, const String&) = 0;
- virtual void setContentDecryptionModule(blink::WebContentDecryptionModule*) = 0;
+ virtual blink::WebMediaPlayer* webMediaPlayer() const = 0;
ddorwin 2014/02/27 00:37:59 This seems like a layering violation. Why can't th
acolwell GONE FROM CHROMIUM 2014/02/27 01:11:11 This layer is going away. There is no point in kee
};
}

Powered by Google App Engine
This is Rietveld 408576698