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

Side by Side Diff: content/renderer/media/webmediaplayer_ms_unittest.cc

Issue 1766783003: Expand suspension of idle media players to all platforms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@notify_pause
Patch Set: Cleanup. Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stddef.h> 5 #include <stddef.h>
6 6
7 #include "content/public/renderer/media_stream_renderer_factory.h" 7 #include "content/public/renderer/media_stream_renderer_factory.h"
8 #include "content/renderer/media/webmediaplayer_ms.h" 8 #include "content/renderer/media/webmediaplayer_ms.h"
9 #include "content/renderer/media/webmediaplayer_ms_compositor.h" 9 #include "content/renderer/media/webmediaplayer_ms_compositor.h"
10 #include "content/renderer/render_frame_impl.h" 10 #include "content/renderer/render_frame_impl.h"
(...skipping 700 matching lines...) Expand 10 before | Expand all | Expand 10 after
711 player_.OnPause(); 711 player_.OnPause();
712 EXPECT_TRUE(player_.paused()); 712 EXPECT_TRUE(player_.paused());
713 713
714 // A hidden player should start playing after being shown again. 714 // A hidden player should start playing after being shown again.
715 delegate_.set_hidden(false); 715 delegate_.set_hidden(false);
716 player_.OnShown(); 716 player_.OnShown();
717 EXPECT_FALSE(player_.paused()); 717 EXPECT_FALSE(player_.paused());
718 718
719 // A hidden event should not pause the player. 719 // A hidden event should not pause the player.
720 delegate_.set_hidden(true); 720 delegate_.set_hidden(true);
721 player_.OnHidden(false); 721 player_.OnHidden();
722 EXPECT_FALSE(player_.paused()); 722 EXPECT_FALSE(player_.paused());
723 723
724 // OnPause() should pause the player. 724 // OnPause() should pause the player.
725 player_.OnPause(); 725 player_.OnPause();
726 EXPECT_TRUE(player_.paused()); 726 EXPECT_TRUE(player_.paused());
727 727
728 // A user generated pause() should clear the automatic resumption. 728 // A user generated pause() should clear the automatic resumption.
729 player_.pause(); 729 player_.pause();
730 delegate_.set_hidden(false); 730 delegate_.set_hidden(false);
731 player_.OnShown(); 731 player_.OnShown();
732 EXPECT_TRUE(player_.paused()); 732 EXPECT_TRUE(player_.paused());
733 733
734 // A user generated play() should start playback. 734 // A user generated play() should start playback.
735 player_.play(); 735 player_.play();
736 EXPECT_FALSE(player_.paused()); 736 EXPECT_FALSE(player_.paused());
737 737
738 // An OnHidden() with forced suspension should pause playback. 738 // An OnSuspend() without forced suspension should do nothing.
739 delegate_.set_hidden(true); 739 player_.OnSuspend(false);
740 player_.OnHidden(true); 740 EXPECT_FALSE(player_.paused());
741
742 // An OnSuspend() with forced suspension should pause playback.
743 player_.OnSuspend(true);
741 EXPECT_TRUE(player_.paused()); 744 EXPECT_TRUE(player_.paused());
742 745
743 // OnShown() should restart after a forced suspension. 746 // OnShown() should restart after a forced suspension.
744 delegate_.set_hidden(false);
745 player_.OnShown(); 747 player_.OnShown();
746 EXPECT_FALSE(player_.paused()); 748 EXPECT_FALSE(player_.paused());
747 EXPECT_CALL(*this, DoSetWebLayer(false)); 749 EXPECT_CALL(*this, DoSetWebLayer(false));
748 } 750 }
749 #endif 751 #endif
750 752
751 } // namespace content 753 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698