DialogBrowserTest implementation to invoke Content settings bubble dialogs.

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

 // Copyright (c) 2012 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/views/location_bar/location_bar_view.h"
 
 #include <algorithm>
 #include <map>
 
 #include "base/i18n/rtl.h"
@@ skipping 1025 matching lines @@
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // LocationBarView, private LocationBarTesting implementation:
 
 int LocationBarView::PageActionCount() {
   return page_action_views_.size();
 }
 
 int LocationBarView::PageActionVisibleCount() {
-  int result = 0;
+  size_t result = 0;

[tapted, 2017/02/03 00:35:41]

why this change? -- (style guide prefers int in general, but size_t is
acceptable when it makes things cleaner)

[kylix_rd, 2017/02/03 18:55:04]

Done.

[Peter Kasting, 2017/02/03 22:20:41]

See other comment on style guide not actually preferring size_t in general, but
it's definitely weird to change to a size_t here but not in the signature of the
function.

Changing the function as a whole to return a size_t would be OK, though, and
probably in keeping with the other APIs here that take size_ts.

   for (const auto& action_view : page_action_views_) {
     if (action_view->visible())
       ++result;
   }
   return result;
 }
 
 ExtensionAction* LocationBarView::GetPageAction(size_t index) {
   if (index < page_action_views_.size())
     return page_action_views_[index]->image_view()->extension_action();
@@ skipping 30 matching lines @@
   }
 
   NOTREACHED();
 }
 
 bool LocationBarView::GetBookmarkStarVisibility() {
   DCHECK(star_view_);
   return star_view_->visible();
 }
 
+int LocationBarView::ContentSettingImageModelCount() {
+  return content_setting_views_.size();
+}
+
+ContentSettingImageModel* LocationBarView::GetContentSettingImageModel(
+    size_t index) {
+  DCHECK(index >= 0 && index < content_setting_views_.size());
+  return content_setting_views_.at(index)->content_setting_image_model();
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // LocationBarView, private views::View implementation:
 
 const char* LocationBarView::GetClassName() const {
   return kViewClassName;
 }
 
 void LocationBarView::OnBoundsChanged(const gfx::Rect& previous_bounds) {
   OmniboxPopupView* popup = omnibox_view_->model()->popup_model()->view();
   if (popup->IsOpen())
@@ skipping 106 matching lines @@
 // LocationBarView, private TemplateURLServiceObserver implementation:
 
 void LocationBarView::OnTemplateURLServiceChanged() {
   template_url_service_->RemoveObserver(this);
   template_url_service_ = nullptr;
   // If the browser is no longer active, let's not show the info bubble, as this
   // would make the browser the active window again.
   if (omnibox_view_ && omnibox_view_->GetWidget()->IsActive())
     ShowFirstRunBubble();
 }
+
+////////////////////////////////////////////////////////////////////////////////
+// LocationBarView, private function(s) for testing:
+
+ContentSettingImageView*
+LocationBarView::GetContentSettingImageViewFromImageModel(

[tapted, 2017/02/03 00:35:40]

(move to TestApi class)

+    ContentSettingImageModel* image_model) {
+  for (auto i = content_setting_views_.begin();

[tapted, 2017/02/03 00:35:40]

for (ContentSettingImageView* view : content_setting_views_) {
  if (view->content_setting_image_model() == image_model)
    return view;

+       i < content_setting_views_.end(); i++) {
+    if ((*i)->content_setting_image_model() == image_model)
+      return *i;
+  }
+  NOTREACHED();
+  return nullptr;
+}
+
+