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

Unified Diff: media/mojo/clients/mojo_renderer_unittest.cc

Issue 2386443002: media: Use native BufferingState in media mojo interfaces (Closed)
Patch Set: media: Use native media enums in media mojo interfaces Created 4 years, 2 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 | « media/mojo/clients/mojo_renderer.cc ('k') | media/mojo/common/media_type_converters.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/mojo/clients/mojo_renderer_unittest.cc
diff --git a/media/mojo/clients/mojo_renderer_unittest.cc b/media/mojo/clients/mojo_renderer_unittest.cc
index e57ccefba0529121386f95f1d8cb465464ba5fad..296949b8ec2d89d95bad6806c4dce424ea773390 100644
--- a/media/mojo/clients/mojo_renderer_unittest.cc
+++ b/media/mojo/clients/mojo_renderer_unittest.cc
@@ -372,6 +372,21 @@ TEST_F(MojoRendererTest, GetMediaTime) {
Destroy();
}
+TEST_F(MojoRendererTest, OnBufferingStateChange) {
+ Initialize();
+ Play();
+
+ EXPECT_CALL(renderer_client_, OnBufferingStateChange(BUFFERING_HAVE_ENOUGH))
+ .Times(1);
+ remote_renderer_client_->OnBufferingStateChange(BUFFERING_HAVE_ENOUGH);
+
+ EXPECT_CALL(renderer_client_, OnBufferingStateChange(BUFFERING_HAVE_NOTHING))
+ .Times(1);
+ remote_renderer_client_->OnBufferingStateChange(BUFFERING_HAVE_NOTHING);
+
+ base::RunLoop().RunUntilIdle();
+}
+
TEST_F(MojoRendererTest, OnEnded) {
Initialize();
Play();
« no previous file with comments | « media/mojo/clients/mojo_renderer.cc ('k') | media/mojo/common/media_type_converters.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698