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

Unified Diff: content/renderer/media/webmediaplayer_ms_unittest.cc

Issue 2490783002: Refactor WebMediaPlayerDelegate interface. (Closed)
Patch Set: Fix did_play_ to only be set once per play. Created 3 years, 11 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: content/renderer/media/webmediaplayer_ms_unittest.cc
diff --git a/content/renderer/media/webmediaplayer_ms_unittest.cc b/content/renderer/media/webmediaplayer_ms_unittest.cc
index bed8922e20487a5d1cb34a4b4c21c7a9ad67ffe3..ba1343e0c7d8476e38996761cdd6ffe764faa0a8 100644
--- a/content/renderer/media/webmediaplayer_ms_unittest.cc
+++ b/content/renderer/media/webmediaplayer_ms_unittest.cc
@@ -53,17 +53,17 @@ class FakeWebMediaPlayerDelegate
}
void DidPlay(int delegate_id,
- bool has_video,
bool has_audio,
- bool is_remote,
+ bool has_video,
media::MediaContentType type) override {
EXPECT_EQ(delegate_id_, delegate_id);
EXPECT_FALSE(playing_);
playing_ = true;
+ has_video_ = has_video;
is_gone_ = false;
}
- void DidPause(int delegate_id, bool reached_end_of_stream) override {
+ void DidPause(int delegate_id) override {
EXPECT_EQ(delegate_id_, delegate_id);
EXPECT_TRUE(playing_);
EXPECT_FALSE(is_gone_);
@@ -75,9 +75,28 @@ class FakeWebMediaPlayerDelegate
is_gone_ = true;
}
- bool IsHidden() override { return is_hidden_; }
+ void SetIdle(int delegate_id, bool is_idle) override {
+ EXPECT_EQ(delegate_id_, delegate_id);
+ is_idle_ = is_idle;
+ }
+
+ bool IsIdle(int delegate_id) override {
+ EXPECT_EQ(delegate_id_, delegate_id);
+ return is_idle_;
+ }
+
+ void ClearStaleFlag(int delegate_id) override {
+ EXPECT_EQ(delegate_id_, delegate_id);
+ }
+
+ bool IsStale(int delegate_id) override {
+ EXPECT_EQ(delegate_id_, delegate_id);
+ return false;
+ }
- bool IsPlayingBackgroundVideo() override { return false; }
+ bool IsFrameHidden() override { return is_hidden_; }
+ bool IsFrameClosed() override { return false; }
+ bool IsBackgroundVideoPlaybackAllowed() override { return false; }
void set_hidden(bool is_hidden) { is_hidden_ = is_hidden; }
@@ -85,8 +104,10 @@ class FakeWebMediaPlayerDelegate
int delegate_id_ = 1234;
Observer* observer_ = nullptr;
bool playing_ = false;
+ bool has_video_ = false;
bool is_hidden_ = false;
bool is_gone_ = true;
+ bool is_idle_ = false;
DISALLOW_COPY_AND_ASSIGN(FakeWebMediaPlayerDelegate);
};

Powered by Google App Engine
This is Rietveld 408576698