Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 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 | |
| 3 // found in the LICENSE file. | |
| 4 | |
| 5 #include "content/renderer/media/android/renderer_media_session_manager.h" | |
| 6 | |
| 7 #include "base/logging.h" | |
| 8 #include "content/renderer/media/android/webmediasession_android.h" | |
| 9 | |
| 10 namespace content { | |
| 11 | |
| 12 const int kDefaultMediaSession = 0; // ID of the default media session | |
|
philipj_slow
2015/11/16 14:06:21
Perhaps append ID to this name?
davve
2015/11/16 14:34:26
Done.
| |
| 13 | |
| 14 RendererMediaSessionManager::RendererMediaSessionManager( | |
| 15 RenderFrame* render_frame) | |
| 16 : RenderFrameObserver(render_frame), | |
| 17 next_session_id_(kDefaultMediaSession + 1) {} | |
| 18 | |
| 19 RendererMediaSessionManager::~RendererMediaSessionManager() { | |
| 20 DCHECK(sessions_.empty()) | |
| 21 << "RendererMediaSessionManager is owned by RenderFrameImpl and is " | |
| 22 "destroyed only after all media sessions are destroyed."; | |
| 23 } | |
| 24 | |
| 25 int RendererMediaSessionManager::RegisterMediaSession( | |
| 26 WebMediaSessionAndroid* session) { | |
| 27 sessions_[next_session_id_] = session; | |
| 28 return next_session_id_++; | |
|
philipj_slow
2015/11/16 14:06:21
Maybe CHECK that this doesn't overflow so that it
davve
2015/11/16 14:34:26
Adding a check sounds good, but I'm also curious w
| |
| 29 } | |
| 30 | |
| 31 void RendererMediaSessionManager::UnregisterMediaSession(int session_id) { | |
| 32 sessions_.erase(session_id); | |
| 33 } | |
| 34 | |
| 35 } // namespace content | |
| OLD | NEW |