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

Side by Side Diff: content/renderer/media/media_devices_listener_impl.cc

Issue 2851173004: Eliminate bind callback that doesn't take a BindSourceInfo parameter. (Closed)
Patch Set: . Created 3 years, 7 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/renderer/media/media_devices_listener_impl.h" 5 #include "content/renderer/media/media_devices_listener_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/memory/ptr_util.h" 9 #include "base/memory/ptr_util.h"
10 #include "content/public/renderer/render_frame.h" 10 #include "content/public/renderer/render_frame.h"
11 #include "content/renderer/media/media_devices_event_dispatcher.h" 11 #include "content/renderer/media/media_devices_event_dispatcher.h"
12 #include "mojo/public/cpp/bindings/strong_binding.h" 12 #include "mojo/public/cpp/bindings/strong_binding.h"
13 13
14 namespace content { 14 namespace content {
15 15
16 // static 16 // static
17 void MediaDevicesListenerImpl::Create( 17 void MediaDevicesListenerImpl::Create(
18 int render_frame_id, 18 int render_frame_id,
19 const service_manager::BindSourceInfo& source_info,
19 ::mojom::MediaDevicesListenerRequest request) { 20 ::mojom::MediaDevicesListenerRequest request) {
20 mojo::MakeStrongBinding( 21 mojo::MakeStrongBinding(
21 base::MakeUnique<MediaDevicesListenerImpl>(render_frame_id), 22 base::MakeUnique<MediaDevicesListenerImpl>(render_frame_id),
22 std::move(request)); 23 std::move(request));
23 } 24 }
24 25
25 MediaDevicesListenerImpl::MediaDevicesListenerImpl(int render_frame_id) 26 MediaDevicesListenerImpl::MediaDevicesListenerImpl(int render_frame_id)
26 : render_frame_id_(render_frame_id) {} 27 : render_frame_id_(render_frame_id) {}
27 28
28 MediaDevicesListenerImpl::~MediaDevicesListenerImpl() = default; 29 MediaDevicesListenerImpl::~MediaDevicesListenerImpl() = default;
29 30
30 void MediaDevicesListenerImpl::OnDevicesChanged( 31 void MediaDevicesListenerImpl::OnDevicesChanged(
31 MediaDeviceType type, 32 MediaDeviceType type,
32 uint32_t subscription_id, 33 uint32_t subscription_id,
33 const MediaDeviceInfoArray& device_infos) { 34 const MediaDeviceInfoArray& device_infos) {
34 DCHECK(thread_checker_.CalledOnValidThread()); 35 DCHECK(thread_checker_.CalledOnValidThread());
35 36
36 base::WeakPtr<MediaDevicesEventDispatcher> event_dispatcher = 37 base::WeakPtr<MediaDevicesEventDispatcher> event_dispatcher =
37 MediaDevicesEventDispatcher::GetForRenderFrame( 38 MediaDevicesEventDispatcher::GetForRenderFrame(
38 RenderFrame::FromRoutingID(render_frame_id_)); 39 RenderFrame::FromRoutingID(render_frame_id_));
39 if (event_dispatcher) 40 if (event_dispatcher)
40 event_dispatcher->DispatchDevicesChangedEvent(type, device_infos); 41 event_dispatcher->DispatchDevicesChangedEvent(type, device_infos);
41 } 42 }
42 43
43 } // namespace content 44 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698