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

Side by Side Diff: content/browser/media/session/mock_media_session_player_observer.cc

Issue 2416853005: Fixing naming issues in MediaSession (Closed)
Patch Set: addressed nits 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "content/browser/media/session/mock_media_session_observer.h" 5 #include "content/browser/media/session/mock_media_session_player_observer.h"
6 6
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 namespace content { 9 namespace content {
10 10
11 MockMediaSessionObserver::MockMediaSessionObserver() = default; 11 MockMediaSessionPlayerObserver::MockMediaSessionPlayerObserver() = default;
12 12
13 MockMediaSessionObserver::~MockMediaSessionObserver() = default; 13 MockMediaSessionPlayerObserver::~MockMediaSessionPlayerObserver() = default;
14 14
15 void MockMediaSessionObserver::OnSuspend(int player_id) { 15 void MockMediaSessionPlayerObserver::OnSuspend(int player_id) {
16 EXPECT_GE(player_id, 0); 16 EXPECT_GE(player_id, 0);
17 EXPECT_GT(players_.size(), static_cast<size_t>(player_id)); 17 EXPECT_GT(players_.size(), static_cast<size_t>(player_id));
18 18
19 ++received_suspend_calls_; 19 ++received_suspend_calls_;
20 players_[player_id].is_playing_ = false; 20 players_[player_id].is_playing_ = false;
21 } 21 }
22 22
23 void MockMediaSessionObserver::OnResume(int player_id) { 23 void MockMediaSessionPlayerObserver::OnResume(int player_id) {
24 EXPECT_GE(player_id, 0); 24 EXPECT_GE(player_id, 0);
25 EXPECT_GT(players_.size(), static_cast<size_t>(player_id)); 25 EXPECT_GT(players_.size(), static_cast<size_t>(player_id));
26 26
27 ++received_resume_calls_; 27 ++received_resume_calls_;
28 players_[player_id].is_playing_ = true; 28 players_[player_id].is_playing_ = true;
29 } 29 }
30 30
31 void MockMediaSessionObserver::OnSetVolumeMultiplier( 31 void MockMediaSessionPlayerObserver::OnSetVolumeMultiplier(
32 int player_id, double volume_multiplier) { 32 int player_id,
33 double volume_multiplier) {
33 EXPECT_GE(player_id, 0); 34 EXPECT_GE(player_id, 0);
34 EXPECT_GT(players_.size(), static_cast<size_t>(player_id)); 35 EXPECT_GT(players_.size(), static_cast<size_t>(player_id));
35 36
36 EXPECT_GE(volume_multiplier, 0.0f); 37 EXPECT_GE(volume_multiplier, 0.0f);
37 EXPECT_LE(volume_multiplier, 1.0f); 38 EXPECT_LE(volume_multiplier, 1.0f);
38 39
39 players_[player_id].volume_multiplier_ = volume_multiplier; 40 players_[player_id].volume_multiplier_ = volume_multiplier;
40 } 41 }
41 42
42 int MockMediaSessionObserver::StartNewPlayer() { 43 int MockMediaSessionPlayerObserver::StartNewPlayer() {
43 players_.push_back(MockPlayer(true, 1.0f)); 44 players_.push_back(MockPlayer(true, 1.0f));
44 return players_.size() - 1; 45 return players_.size() - 1;
45 } 46 }
46 47
47 bool MockMediaSessionObserver::IsPlaying(size_t player_id) { 48 bool MockMediaSessionPlayerObserver::IsPlaying(size_t player_id) {
48 EXPECT_GT(players_.size(), player_id); 49 EXPECT_GT(players_.size(), player_id);
49 return players_[player_id].is_playing_; 50 return players_[player_id].is_playing_;
50 } 51 }
51 52
52 double MockMediaSessionObserver::GetVolumeMultiplier(size_t player_id) { 53 double MockMediaSessionPlayerObserver::GetVolumeMultiplier(size_t player_id) {
53 EXPECT_GT(players_.size(), player_id); 54 EXPECT_GT(players_.size(), player_id);
54 return players_[player_id].volume_multiplier_; 55 return players_[player_id].volume_multiplier_;
55 } 56 }
56 57
57 void MockMediaSessionObserver::SetPlaying(size_t player_id, bool playing) { 58 void MockMediaSessionPlayerObserver::SetPlaying(size_t player_id,
59 bool playing) {
58 EXPECT_GT(players_.size(), player_id); 60 EXPECT_GT(players_.size(), player_id);
59 players_[player_id].is_playing_ = playing; 61 players_[player_id].is_playing_ = playing;
60 } 62 }
61 63
62 int MockMediaSessionObserver::received_suspend_calls() const { 64 int MockMediaSessionPlayerObserver::received_suspend_calls() const {
63 return received_suspend_calls_; 65 return received_suspend_calls_;
64 } 66 }
65 67
66 int MockMediaSessionObserver::received_resume_calls() const { 68 int MockMediaSessionPlayerObserver::received_resume_calls() const {
67 return received_resume_calls_; 69 return received_resume_calls_;
68 } 70 }
69 71
70 } // namespace content 72 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698