Adding device selection menus to the content setting bubble

chrome/browser/ui/content_settings/content_setting_media_menu_model.cc

+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/content_settings/content_setting_media_menu_model.h"
+
+#include "base/utf_string_conversions.h"
+#include "chrome/browser/media/media_capture_devices_dispatcher.h"
+#include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/content_settings/content_setting_bubble_model.h"
+#include "chrome/common/pref_names.h"
+
+
+ContentSettingMediaMenuModel::ContentSettingMediaMenuModel(
+    Profile* profile,
+    content::MediaStreamType type,
+    ContentSettingBubbleModel* bubble_model,
+    Observer* observer)
+    : ALLOW_THIS_IN_INITIALIZER_LIST(ui::SimpleMenuModel(this)),
+      profile_(profile),
+      type_(type),
+      media_bubble_model_(bubble_model),
+      observer_(observer) {
+  DCHECK_EQ(CONTENT_SETTINGS_TYPE_MEDIASTREAM,
+            media_bubble_model_->content_type());
+  BuildMenu();

[markusheintz_, 2013/02/06 11:03:26]

I guess we can inline BuildMenu here. I don't see that it is called anywhere
else.

[no longer working on chromium, 2013/02/06 13:31:52]

We can do this. But from the readability perspective, it is good to wrap the
code into a private function to keep the constructor small. I hope it is fine to
keep it this way, but please let me know if you strongly want it to be inline, I
can change it.

[markusheintz_, 2013/02/06 14:43:19]

SGTM.

+}
+
+ContentSettingMediaMenuModel::~ContentSettingMediaMenuModel() {
+}
+
+bool ContentSettingMediaMenuModel::IsCommandIdChecked(int command_id) const {
+  return false;
+}
+
+bool ContentSettingMediaMenuModel::IsCommandIdEnabled(int command_id) const {
+  return true;
+}
+
+bool ContentSettingMediaMenuModel::GetAcceleratorForCommandId(

[markusheintz_, 2013/02/06 11:03:26]

Is it possible to open navigate and close the menu without a mouse and the
keyboard only? If not let's file a bug for this because this would be an
accessibility issue.

[no longer working on chromium, 2013/02/06 13:31:52]

Not sure here, I just filed a bug 174615.
Do you know if we can use the keyboard to open the bubble?

[markusheintz_, 2013/02/06 14:43:19]

Got to double check this. Thanks for filing the bug.

+    int command_id,
+    ui::Accelerator* accelerator) {
+  return false;
+}
+
+void ContentSettingMediaMenuModel::ExecuteCommand(int command_id) {
+  CommandMap::const_iterator it = commands_.find(command_id);
+  DCHECK(it != commands_.end());
+  media_bubble_model_->OnMeiaMenuClicked(type_, it->second.id);
+
+  if (observer_)
+    observer_->UpdateMenuLabel(type_, it->second.name);
+}
+
+void ContentSettingMediaMenuModel::BuildMenu() {
+  PrefService* prefs = profile_->GetPrefs();
+  MediaCaptureDevicesDispatcher* dispatcher =
+      MediaCaptureDevicesDispatcher::GetInstance();
+  content::MediaStreamDevices devices;
+  std::string default_device;
+  if (type_ == content::MEDIA_DEVICE_AUDIO_CAPTURE) {
+    devices = dispatcher->GetAudioCaptureDevices();
+    default_device = prefs->GetString(prefs::kDefaultAudioCaptureDevice);
+  } else if (type_ == content::MEDIA_DEVICE_VIDEO_CAPTURE) {
+    devices = dispatcher->GetVideoCaptureDevices();
+    default_device = prefs->GetString(prefs::kDefaultVideoCaptureDevice);
+  } else {
+    NOTREACHED();
+  }
+
+  for (size_t i = 0; i < devices.size(); ++i) {
+    int command_id = commands_.size();
+    commands_.insert(std::make_pair(command_id, devices[i]));
+    AddItem(i, UTF8ToUTF16(devices[i].name));
+  }
+}