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

Unified Diff: Source/modules/mediasource/HTMLVideoElementMediaSource.cpp

Issue 291873002: Eliminate MediaPlayer & MediaPlayerClient abstractions (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Removed one more unused method Created 6 years, 7 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/modules/mediasource/HTMLVideoElementMediaSource.cpp
diff --git a/Source/modules/mediasource/HTMLVideoElementMediaSource.cpp b/Source/modules/mediasource/HTMLVideoElementMediaSource.cpp
index 8d3c75d8181015bf7db27df8248bde7162e20531..bef36dbf62a3620955d7f18fd2ba2a21a6e659c7 100644
--- a/Source/modules/mediasource/HTMLVideoElementMediaSource.cpp
+++ b/Source/modules/mediasource/HTMLVideoElementMediaSource.cpp
@@ -36,6 +36,8 @@
namespace WebCore {
+using blink::WebMediaPlayer;
acolwell GONE FROM CHROMIUM 2014/05/20 16:40:21 nit: No need for using since there is only one ins
+
HTMLVideoElementMediaSource::HTMLVideoElementMediaSource() { }
HTMLVideoElementMediaSource::~HTMLVideoElementMediaSource() { }
@@ -45,11 +47,11 @@ VideoPlaybackQuality* HTMLVideoElementMediaSource::getVideoPlaybackQuality(HTMLV
unsigned total = 0;
unsigned dropped = 0;
unsigned corrupted = 0;
- MediaPlayer* player = videoElement.player();
- if (player) {
- total = player->decodedFrameCount();
- dropped = player->droppedFrameCount();
- corrupted = player->corruptedFrameCount();
+ WebMediaPlayer* webMediaPlayer = videoElement.webMediaPlayer();
+ if (webMediaPlayer) {
+ total = webMediaPlayer->decodedFrameCount();
+ dropped = webMediaPlayer->droppedFrameCount();
+ corrupted = webMediaPlayer->corruptedFrameCount();
}
return VideoPlaybackQuality::create(videoElement.document(), total, dropped, corrupted);

Powered by Google App Engine
This is Rietveld 408576698