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

Unified Diff: content/browser/media/session/media_session_service_impl.cc

Issue 2723803002: Fix two notification update issues on page navigation (Closed)
Patch Set: fixed tests Created 3 years, 10 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/browser/media/session/media_session_service_impl.cc
diff --git a/content/browser/media/session/media_session_service_impl.cc b/content/browser/media/session/media_session_service_impl.cc
index cf452e2204df634b8f8e29dc8b873006c40c9484..f7cf58d65d15965a7dadd110d3b5240f4fb9e46d 100644
--- a/content/browser/media/session/media_session_service_impl.cc
+++ b/content/browser/media/session/media_session_service_impl.cc
@@ -43,6 +43,12 @@ RenderFrameHost* MediaSessionServiceImpl::GetRenderFrameHost() {
render_frame_routing_id_);
}
+void MediaSessionServiceImpl::Reset() {
+ playback_state_ = blink::mojom::MediaSessionPlaybackState::NONE;
whywhat 2017/02/28 21:26:56 nit: I wonder if you should actually call SetPlayb
Zhiqiang Zhang (Slow) 2017/02/28 22:13:10 Done. Also I think the actions should be cleared i
+ metadata_ = base::nullopt;
+ actions_.clear();
+}
+
void MediaSessionServiceImpl::SetClient(
blink::mojom::MediaSessionClientPtr client) {
client_ = std::move(client);

Powered by Google App Engine
This is Rietveld 408576698