Address some TODOs for ChooserBubbleDelegate class.
This patch did the following changes:
1. Changed
ChooserBubbleDelegate::Observer::OnOptionAdded(int index)
ChooserBubbleDelegate::Observer::OnOptionRemoved(int index)
to
ChooserBubbleDelegate::Observer::OnOptionAdded(size_t index)
ChooserBubbleDelegate::Observer::OnOptionRemoved(size_t index)
2. Changed
ChooserBubbleDelegate::GetOptions()
function to two functions:
ChooserBubbleDelegate::NumOptions
ChooserBubbleDelegate::GetOption(size_t index)
to improve class encapsulation.
3. Changed
ChooserBubbleDelegate::Select(int index)
to:
ChooserBubbleDelegate::Select(size_t index)
4. For USB/Bluetooth bubble delegate class, use a vector
containing a pair of device information instead of two
vectors.
5. Changed code that uses the above functions.
BUG=492204, 535633, 535635
Review URL: https://codereview.chromium.org/1545773002
Cr-Commit-Position: refs/heads/master@{#367476}
8 files changed