https://cs.chromium.org/chromium/src/content/public/browser/bluetooth_chooser.h?dr=CSs&sq=package:chromium&rcl=1482150271&l=89 It seems that this function is not called anywhere. Maybe it can be removed?
SGTM
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f33d69fc38520101b4dfeed7213eec7be237e52b commit f33d69fc38520101b4dfeed7213eec7be237e52b Author: juncai <juncai@chromium.org> Date: Wed Mar 15 17:16:25 2017 Remove content::BluetoothChooser::RemoveDevice() function content::BluetoothChooser::RemoveDevice() function is not used, this CL removes it. BUG= 675720 Review-Url: https://codereview.chromium.org/2753613002 Cr-Commit-Position: refs/heads/master@{#457114} [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/chrome/android/java/src/org/chromium/chrome/browser/BluetoothChooserDialog.java [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/chrome/browser/ui/android/bluetooth_chooser_android.cc [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/chrome/browser/ui/android/bluetooth_chooser_android.h [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/chrome/browser/ui/bluetooth/bluetooth_chooser_desktop.cc [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/chrome/browser/ui/bluetooth/bluetooth_chooser_desktop.h [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/chrome/browser/ui/bluetooth/chrome_extension_bluetooth_chooser.cc [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/chrome/browser/ui/bluetooth/chrome_extension_bluetooth_chooser.h [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/content/public/browser/bluetooth_chooser.h [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/content/shell/browser/layout_test/layout_test_bluetooth_chooser_factory.cc [modify] https://crrev.com/f33d69fc38520101b4dfeed7213eec7be237e52b/content/shell/browser/layout_test/layout_test_first_device_bluetooth_chooser.h
Comment 1 by juncai@chromium.org
, Jan 10 2017Status: Assigned (was: Available)