DialogBrowserTest implementation to invoke Content settings bubble dialogs.

chrome/browser/ui/views/location_bar/content_setting_bubble_dialog_browsertest.cc

+// Copyright 2017 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 "base/memory/ptr_util.h"
+#include "base/time/time.h"
+#include "chrome/browser/content_settings/tab_specific_content_settings.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/content_settings/content_setting_image_model.h"
+#include "chrome/browser/ui/location_bar/location_bar.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/test/test_browser_dialog.h"
+#include "chrome/browser/ui/views/content_setting_bubble_contents.h"
+#include "chrome/browser/ui/views/frame/browser_view.h"

[tapted, 2017/02/07 01:23:31]

is it possible to drop this? (it isn't linked on Mac)

you might just need to substitute #include "chrome/browser/ui/browser_window.h"
which is cross-platform

[kylix_rd, 2017/02/07 18:25:21]

Done.

+#include "chrome/test/base/in_process_browser_test.h"
+#include "components/content_settings/core/common/content_settings_types.h"
+#include "ui/events/event.h"
+#include "ui/events/event_constants.h"
+#include "ui/views/view.h"
+#include "ui/views/widget/widget.h"
+#include "url/gurl.h"
+
+class ContentSettingBubbleDialogTest : public DialogBrowserTest {
+ public:
+  ContentSettingBubbleDialogTest() {}
+
+  void ShowDialogBubble(ContentSettingsType content_type);
+  void ShowDialog(const std::string& name) override;
+ private:

[tapted, 2017/02/07 01:23:31]

nit: blank line before

[kylix_rd, 2017/02/07 18:25:21]

Done.

+  DISALLOW_COPY_AND_ASSIGN(ContentSettingBubbleDialogTest);
+};
+
+void ContentSettingBubbleDialogTest::ShowDialogBubble(
+    ContentSettingsType content_type) {
+  content::WebContents* web_contents =
+      browser()->tab_strip_model()->GetActiveWebContents();
+  TabSpecificContentSettings* content_settings =
+      TabSpecificContentSettings::FromWebContents(web_contents);
+  switch (content_type) {
+    case CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC:
+    case CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA:
+      content_settings->OnMediaStreamPermissionSet(
+          GURL::EmptyGURL(),
+          content_type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC
+              ? TabSpecificContentSettings::MICROPHONE_ACCESSED
+              : TabSpecificContentSettings::CAMERA_ACCESSED,
+          std::string(), std::string(), std::string(), std::string());
+      break;
+    case CONTENT_SETTINGS_TYPE_GEOLOCATION:
+      content_settings->OnGeolocationPermissionSet(GURL::EmptyGURL(), false);
+      break;
+    default:
+      // For all other content_types passed in, mark them as blocked.
+      content_settings->OnContentBlocked(content_type);
+      break;
+  }
+  browser()->window()->UpdateToolbar(web_contents);
+  LocationBarTesting* location_bar_testing =
+      browser()->window()->GetLocationBar()->GetLocationBarForTesting();
+  EXPECT_TRUE(location_bar_testing->TestContentSettingImagePressed(
+      ContentSettingImageModel::GetContentSettingImageModelIndexForTesting(
+          content_type)));
+}
+
+void ContentSettingBubbleDialogTest::ShowDialog(const std::string& name) {
+  static constexpr struct {

[tapted, 2017/02/07 01:23:31]

nit: no need for static (I think that will reserve space in the data segment,
making the binary slightly bigger .. or the compiler could just optimise it all
away too since it is constexpr)

+    const char* name;
+    ContentSettingsType content_type;
+  } content_settings_values[] = {
+      {"cookies", CONTENT_SETTINGS_TYPE_COOKIES},
+      {"images", CONTENT_SETTINGS_TYPE_IMAGES},
+      {"javascript", CONTENT_SETTINGS_TYPE_JAVASCRIPT},
+      {"plugins", CONTENT_SETTINGS_TYPE_PLUGINS},
+      {"popups", CONTENT_SETTINGS_TYPE_POPUPS},
+      {"geolocation", CONTENT_SETTINGS_TYPE_GEOLOCATION},
+      {"ppapi_broker", CONTENT_SETTINGS_TYPE_PPAPI_BROKER},
+      {"mixed_script", CONTENT_SETTINGS_TYPE_MIXEDSCRIPT},
+      {"mediastream_mic", CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC},
+      {"mediastream_camera", CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA},
+      {"protocol_handlers", CONTENT_SETTINGS_TYPE_PROTOCOL_HANDLERS},
+      {"automatic_downloads", CONTENT_SETTINGS_TYPE_AUTOMATIC_DOWNLOADS},
+      {"midi_sysex", CONTENT_SETTINGS_TYPE_MIDI_SYSEX},
+  };
+  for (auto content_settings : content_settings_values) {
+    if (name == content_settings.name) {
+      ShowDialogBubble(content_settings.content_type);
+      return;
+    }
+  }
+  ADD_FAILURE();

[tapted, 2017/02/07 01:23:31]

nit: maybe comment here like " // Not reached."? I guess I haven't seen this
pattern enough to immediately realise that's what it indicates here.

+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest, InvokeDialog_cookies) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest, InvokeDialog_images) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_javascript) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest, InvokeDialog_plugins) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest, InvokeDialog_popups) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_geolocation) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_ppapi_broker) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_mixed_script) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_mediastream_mic) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_mediastream_camera) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_protocol_handlers) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_automatic_downloads) {
+  RunDialog();
+}
+
+IN_PROC_BROWSER_TEST_F(ContentSettingBubbleDialogTest,
+                       InvokeDialog_midi_sysex) {
+  RunDialog();
+}