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 |