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

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

Issue 2798083002: Record user interactions with MediaSession by action type (Closed)
Patch Set: addressed style nits Created 3 years, 8 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 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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/media_session_service_impl.h" 5 #include "content/browser/media/session/media_session_service_impl.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/run_loop.h" 8 #include "base/run_loop.h"
9 #include "content/browser/media/session/media_session_impl.h" 9 #include "content/browser/media/session/media_session_impl.h"
10 #include "content/browser/media/session/media_session_player_observer.h" 10 #include "content/browser/media/session/media_session_player_observer.h"
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 explicit MockMediaSessionPlayerObserver(RenderFrameHost* rfh) 60 explicit MockMediaSessionPlayerObserver(RenderFrameHost* rfh)
61 : render_frame_host_(rfh) {} 61 : render_frame_host_(rfh) {}
62 62
63 ~MockMediaSessionPlayerObserver() override = default; 63 ~MockMediaSessionPlayerObserver() override = default;
64 64
65 void OnSuspend(int player_id) override {} 65 void OnSuspend(int player_id) override {}
66 void OnResume(int player_id) override {} 66 void OnResume(int player_id) override {}
67 void OnSetVolumeMultiplier(int player_id, double volume_multiplier) override { 67 void OnSetVolumeMultiplier(int player_id, double volume_multiplier) override {
68 } 68 }
69 69
70 RenderFrameHost* GetRenderFrameHost() const override { 70 RenderFrameHost* render_frame_host() const override {
71 return render_frame_host_; 71 return render_frame_host_;
72 } 72 }
73 73
74 private: 74 private:
75 RenderFrameHost* render_frame_host_; 75 RenderFrameHost* render_frame_host_;
76 }; 76 };
77 77
78 void NavigateToURLAndWaitForFinish(Shell* window, const GURL& url) { 78 void NavigateToURLAndWaitForFinish(Shell* window, const GURL& url) {
79 base::RunLoop run_loop; 79 base::RunLoop run_loop;
80 MockWebContentsObserver observer(window->web_contents(), 80 MockWebContentsObserver observer(window->web_contents(),
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
187 fragment_change_url = GURL(fragment_change_url.spec() + "#some-anchor"); 187 fragment_change_url = GURL(fragment_change_url.spec() + "#some-anchor");
188 NavigateToURLAndWaitForFinish(shell(), fragment_change_url); 188 NavigateToURLAndWaitForFinish(shell(), fragment_change_url);
189 189
190 EXPECT_EQ(blink::mojom::MediaSessionPlaybackState::PLAYING, 190 EXPECT_EQ(blink::mojom::MediaSessionPlaybackState::PLAYING,
191 GetService()->playback_state()); 191 GetService()->playback_state());
192 EXPECT_TRUE(GetService()->metadata().has_value()); 192 EXPECT_TRUE(GetService()->metadata().has_value());
193 EXPECT_EQ(1u, GetService()->actions().size()); 193 EXPECT_EQ(1u, GetService()->actions().size());
194 } 194 }
195 195
196 } // namespace content 196 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698