New issue
Advanced search Search tips

Issue 860209 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Change MediaDevicesManager::EnumerationCallback into a OnceCallback

Project Member Reported by guidou@chromium.org, Jul 4

Issue description

It was originally defined as a Callback (before OnceCallback and RepeatingCallback).
It should be a OnceCallback.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/19f19073ad91cec7fd2fd90a57f55caec55301c3

commit 19f19073ad91cec7fd2fd90a57f55caec55301c3
Author: Guido Urdaneta <guidou@chromium.org>
Date: Wed Jul 04 21:20:16 2018

Change MediaDevicesManager::EnumerationCallback into a OnceCallback.

This CL also changes all the call sites to use base::BindOnce() instead of
base::Bind() or base::BindRepeating().

Drive-by: minor #include lint fix.

Bug: 860209
Change-Id: I112a926bc333c0d6252ac1fd43662b0c1ebaa7ef
Reviewed-on: https://chromium-review.googlesource.com/1125849
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572656}
[modify] https://crrev.com/19f19073ad91cec7fd2fd90a57f55caec55301c3/content/browser/renderer_host/media/audio_output_authorization_handler.cc
[modify] https://crrev.com/19f19073ad91cec7fd2fd90a57f55caec55301c3/content/browser/renderer_host/media/media_devices_dispatcher_host.cc
[modify] https://crrev.com/19f19073ad91cec7fd2fd90a57f55caec55301c3/content/browser/renderer_host/media/media_devices_manager.cc
[modify] https://crrev.com/19f19073ad91cec7fd2fd90a57f55caec55301c3/content/browser/renderer_host/media/media_devices_manager.h
[modify] https://crrev.com/19f19073ad91cec7fd2fd90a57f55caec55301c3/content/browser/renderer_host/media/media_devices_manager_unittest.cc
[modify] https://crrev.com/19f19073ad91cec7fd2fd90a57f55caec55301c3/content/browser/renderer_host/media/old_render_frame_audio_input_stream_factory.cc
[modify] https://crrev.com/19f19073ad91cec7fd2fd90a57f55caec55301c3/content/browser/renderer_host/media/render_frame_audio_input_stream_factory.cc

Sign in to add a comment