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

Unified Diff: Source/web/WebMediaPlayerClientImpl.cpp

Issue 388803005: Eliminate MediaPlayer abstraction(seek, duration, poster APIs) (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: updated as per review Created 6 years, 5 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/WebMediaPlayerClientImpl.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/WebMediaPlayerClientImpl.cpp
diff --git a/Source/web/WebMediaPlayerClientImpl.cpp b/Source/web/WebMediaPlayerClientImpl.cpp
index d6c86149a983df26be0d86e9de9b6d69f41fd29d..9b6c0e952b45cbc7cf9533c6c17cb9d7414d8349 100644
--- a/Source/web/WebMediaPlayerClientImpl.cpp
+++ b/Source/web/WebMediaPlayerClientImpl.cpp
@@ -225,38 +225,6 @@ void WebMediaPlayerClientImpl::load(WebMediaPlayer::LoadType loadType, const WTF
m_webMediaPlayer->load(loadType, kurl, corsMode);
}
-double WebMediaPlayerClientImpl::duration() const
-{
- if (m_webMediaPlayer)
- return m_webMediaPlayer->duration();
- return 0.0;
-}
-
-double WebMediaPlayerClientImpl::currentTime() const
-{
- if (m_webMediaPlayer)
- return m_webMediaPlayer->currentTime();
- return 0.0;
-}
-
-void WebMediaPlayerClientImpl::seek(double time)
-{
- if (m_webMediaPlayer)
- m_webMediaPlayer->seek(time);
-}
-
-bool WebMediaPlayerClientImpl::seeking() const
-{
- if (m_webMediaPlayer)
- return m_webMediaPlayer->seeking();
- return false;
-}
-
-void WebMediaPlayerClientImpl::setPoster(const KURL& poster)
-{
- if (m_webMediaPlayer)
- m_webMediaPlayer->setPoster(WebURL(poster));
-}
void WebMediaPlayerClientImpl::paint(GraphicsContext* context, const IntRect& rect)
{
« no previous file with comments | « Source/web/WebMediaPlayerClientImpl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698