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

Unified Diff: Source/web/WebMediaPlayerClientImpl.h

Issue 157423003: Remove the dependency on encryptedmedia from HTMLMediaElement. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fix compilation error on linux Created 6 years, 9 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
« no previous file with comments | « Source/web/AssertMatchingEnums.cpp ('k') | Source/web/WebMediaPlayerClientImpl.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/WebMediaPlayerClientImpl.h
diff --git a/Source/web/WebMediaPlayerClientImpl.h b/Source/web/WebMediaPlayerClientImpl.h
index 58528d2a2b1e5137bcf3c678c3fd61647980fcec..94501c924a9304cf6d66dfee6b73ae31efe092f5 100644
--- a/Source/web/WebMediaPlayerClientImpl.h
+++ b/Source/web/WebMediaPlayerClientImpl.h
@@ -48,6 +48,7 @@
namespace WebCore {
class AudioSourceProviderClient;
+class HTMLMediaElement;
}
namespace blink {
@@ -66,9 +67,6 @@ public:
virtual ~WebMediaPlayerClientImpl();
- // Returns the encapsulated blink::WebMediaPlayer.
- WebMediaPlayer* mediaPlayer() const;
-
// WebMediaPlayerClient methods:
virtual void networkStateChanged() OVERRIDE;
virtual void readyStateChanged() OVERRIDE;
@@ -80,10 +78,13 @@ public:
virtual double volume() const OVERRIDE;
virtual void playbackStateChanged() OVERRIDE;
virtual WebMediaPlayer::Preload preload() const OVERRIDE;
+
+ // WebEncryptedMediaPlayerClient methods:
virtual void keyAdded(const WebString& keySystem, const WebString& sessionId) OVERRIDE;
virtual void keyError(const WebString& keySystem, const WebString& sessionId, MediaKeyErrorCode, unsigned short systemCode) OVERRIDE;
virtual void keyMessage(const WebString& keySystem, const WebString& sessionId, const unsigned char* message, unsigned messageLength, const WebURL& defaultURL) OVERRIDE;
virtual void keyNeeded(const WebString& contentType, const unsigned char* initData, unsigned initDataLength) OVERRIDE;
+
virtual bool needsWebLayerForVideo() const OVERRIDE;
virtual void setWebLayer(WebLayer*) OVERRIDE;
virtual void addTextTrack(WebInbandTextTrack*) OVERRIDE;
@@ -93,8 +94,8 @@ public:
virtual void requestSeek(double) OVERRIDE;
// MediaPlayer methods:
+ virtual WebMediaPlayer* webMediaPlayer() const OVERRIDE;
virtual void load(WebMediaPlayer::LoadType, const WTF::String& url) OVERRIDE;
-
virtual void play() OVERRIDE;
virtual void pause() OVERRIDE;
virtual void prepareToPlay() OVERRIDE;
@@ -136,17 +137,14 @@ public:
virtual WebCore::AudioSourceProvider* audioSourceProvider() OVERRIDE;
#endif
- virtual WebCore::MediaPlayer::MediaKeyException generateKeyRequest(const String& keySystem, const unsigned char* initData, unsigned initDataLength) OVERRIDE;
- virtual WebCore::MediaPlayer::MediaKeyException addKey(const String& keySystem, const unsigned char* key, unsigned keyLength, const unsigned char* initData, unsigned initDataLength, const String& sessionId) OVERRIDE;
- virtual WebCore::MediaPlayer::MediaKeyException cancelKeyRequest(const String& keySystem, const String& sessionId) OVERRIDE;
- virtual void setContentDecryptionModule(WebContentDecryptionModule*) OVERRIDE;
-
private:
explicit WebMediaPlayerClientImpl(WebCore::MediaPlayerClient*);
void startDelayedLoad();
void loadInternal();
+ WebCore::HTMLMediaElement& mediaElement() const;
+
#if OS(ANDROID)
// FIXME: This path "only works" on Android. It is a workaround for the problem that Skia could not handle Android's GL_TEXTURE_EXTERNAL_OES
// texture internally. It should be removed and replaced by the normal paint path.
@@ -164,7 +162,6 @@ private:
double m_volume;
bool m_muted;
double m_rate;
- WebContentDecryptionModule* m_cdm;
#if ENABLE(WEB_AUDIO)
// AudioClientImpl wraps an AudioSourceProviderClient.
« no previous file with comments | « Source/web/AssertMatchingEnums.cpp ('k') | Source/web/WebMediaPlayerClientImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698