[Mac] Website settings: Add cookie info & permission icons, plus other visual fixes.

chrome/browser/ui/cocoa/website_settings_bubble_controller.mm

 // 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.
 
 #import "chrome/browser/ui/cocoa/website_settings_bubble_controller.h"
 
+#include "base/string_number_conversions.h"
 #include "base/sys_string_conversions.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #import "chrome/browser/ui/cocoa/info_bubble_view.h"
 #import "chrome/browser/ui/cocoa/info_bubble_window.h"
 #import "chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.h"
 #include "chrome/browser/ui/tab_contents/tab_contents.h"
 #include "content/public/browser/cert_store.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "grit/ui_resources.h"
 #import "third_party/GTM/AppKit/GTMUILocalizerAndLayoutTweaker.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 
 namespace {
 
 // The width of the window, in view coordinates. The height will be determined
 // by the content.
 const CGFloat kWindowWidth = 380;
 
 // Spacing in between sections.
 const CGFloat kVerticalSpacing = 10;
 
 // Padding between the window frame and content.
 const CGFloat kFramePadding = 10;
 
 // Spacing between the optional headline and description text views.
 const CGFloat kHeadlineSpacing = 2;
 
-// Spacing between the image and the text.
-const CGFloat kImageSpacing = 10;
+// Spacing between images on the Connection tab and the text.
+const CGFloat kConnectionImageSpacing = 10;
 
-// Square size of the image.
-const CGFloat kImageSize = 30;
+// Square size of the images on the Connections tab.
+const CGFloat kConnectionImageSize = 30;
 
-// The X position of the text fields. Variants for with and without an image.
-const CGFloat kTextXPositionNoImage = kFramePadding;
-const CGFloat kTextXPosition = kTextXPositionNoImage + kImageSize +
-    kImageSpacing;
+// Square size of the permission images.
+const CGFloat kPermissionImageSize = 19;
 
-// Width of the text fields.
-const CGFloat kTextWidth = kWindowWidth - (kImageSize + kImageSpacing +
-    kFramePadding * 2);
+// Vertical adjustment for the permission images.
+// They have an extra pixel of padding on the bottom edge.
+const CGFloat kPermissionImageYAdjust = 1;
+
+// Spacing between a permission image and the text.
+const CGFloat kPermissionImageSpacing = 3;
 
 // The amount of padding given to tab view contents.
 const CGFloat kTabViewContentsPadding = kFramePadding;
 
 // The spacing between individual items in the Permissions tab.
 const CGFloat kPermissionsTabSpacing = 8;
 
 // The extra space to the left of the first tab in the tab strip.
 const CGFloat kTabStripXPadding = 19;
 
@@ skipping 16 matching lines @@
 const CGFloat kTabLabelXPadding = 12;
 
 // In the permission changing menu, the order of the menu items (which
 // correspond to different content settings).
 const ContentSetting kPermissionsMenuSettings[] = {
   CONTENT_SETTING_ALLOW,
   CONTENT_SETTING_BLOCK,
   CONTENT_SETTING_DEFAULT
 };
 
+// Return the text color to use for the indentity status when the site's
+// identity has been verified.
+NSColor* IdentityVerifiedTextColor() {
+  // RGB components are specified using integer RGB [0-255] values for easy
+  // comparison to other platforms.
+  return [NSColor colorWithCalibratedRed:0x07/255.0
+                                   green:0x95/255.0
+                                    blue:0
+                                   alpha:1.0];
+}
+
 }  // namespace
 
 // A simple container to hold all the contents of the website settings bubble.
 // It uses a flipped coordinate system to make text layout easier.
 @interface WebsiteSettingsContentView : NSView
 @end
 @implementation WebsiteSettingsContentView
 - (BOOL)isFlipped {
   return YES;
 }
@@ skipping 146 matching lines @@
 }
 
 - (void)setPresenter:(WebsiteSettings*)presenter {
   presenter_.reset(presenter);
 }
 
 // Create the subviews for the website settings bubble.
 - (void)initializeContents {
   // Keeps track of the position that the next control should be drawn at.
   NSPoint controlOrigin = NSMakePoint(
-      kTextXPositionNoImage,
+      kFramePadding,
       kFramePadding + info_bubble::kBubbleArrowHeight);
 
   // Create the container view that uses flipped coordinates.
   NSRect contentFrame = NSMakeRect(0, 0, kWindowWidth, 300);
   contentView_.reset(
       [[WebsiteSettingsContentView alloc] initWithFrame:contentFrame]);
 
   // Create a text field (empty for now) to show the site identity.
   identityField_ = [self addText:string16()
                         withSize:[NSFont systemFontSize]
@@ skipping 51 matching lines @@
   [segmentedControl_ setSelectedSegment:0];
   [contentView_ addSubview:segmentedControl_];
 
   NSRect tabFrame = NSMakeRect(0, 0, kWindowWidth, 300);
   tabView_.reset([[NSTabView alloc] initWithFrame:tabFrame]);
   [tabView_ setTabViewType:NSNoTabsNoBorder];
   [tabView_ setDrawsBackground:NO];
   [tabView_ setControlSize:NSSmallControlSize];
   [contentView_ addSubview:tabView_.get()];
 
-  permissionsContentView_ = [self addPermissionsTabToTabView:tabView_];
+  permissionsTabContentView_ = [self addPermissionsTabToTabView:tabView_];
   [self addConnectionTabToTabView:tabView_];
 
   // Replace the window's content.
   [[[self window] contentView] setSubviews:
       [NSArray arrayWithObject:contentView_.get()]];
 
   [self performLayout];
 }
 
 // Create the contents of the Permissions tab and add it to the given tab view.
 // Returns a weak reference to the tab view item's view.
 - (NSView*)addPermissionsTabToTabView:(NSTabView*)tabView {
   scoped_nsobject<NSTabViewItem> item([[NSTabViewItem alloc] init]);
   [tabView_ addTabViewItem:item.get()];
   scoped_nsobject<NSView> contentView([[WebsiteSettingsContentView alloc]
       initWithFrame:[tabView_ contentRect]]);
   [item setView:contentView.get()];
+
+  // Initialize the two containers that hold the controls. The initial frames
+  // are arbitrary, and will be adjusted after the controls are laid out.
+  cookiesView_ = [[WebsiteSettingsContentView alloc]

[Robert Sesek, 2012/08/13 17:44:57]

These are leaked.

+      initWithFrame:[tabView_ contentRect]];
+  permissionsView_ = [[WebsiteSettingsContentView alloc]
+      initWithFrame:[tabView_ contentRect]];
+
+  [contentView addSubview:cookiesView_];
+  [contentView addSubview:permissionsView_];
+
   return contentView.get();
 }
 
 // Create the contents of the Connection tab and add it to the given tab view.
 // Returns a weak reference to the tab view item's view.
 - (NSView*)addConnectionTabToTabView:(NSTabView*)tabView {
   scoped_nsobject<NSTabViewItem> item([[NSTabViewItem alloc] init]);
   scoped_nsobject<NSView> contentView([[WebsiteSettingsContentView alloc]
       initWithFrame:[tabView_ contentRect]]);
 
   // Place all the text at the same position. It will be adjusted in
   // performLayout.
   NSPoint textPosition = NSMakePoint(
-      kTabViewContentsPadding + kImageSize + kImageSpacing,
+      kTabViewContentsPadding + kConnectionImageSize + kConnectionImageSpacing,
       kTabViewContentsPadding);
+  NSSize imageSize = NSMakeSize(kConnectionImageSize, kConnectionImageSize);
 
-  identityStatusIcon_ =
-      [self addImageToView:contentView atOffset:kTabViewContentsPadding];
+  identityStatusIcon_ = [self addImageWithSize:imageSize
+                                        toView:contentView
+                                      atOffset:kTabViewContentsPadding];
   identityStatusDescriptionField_ =
       [self addText:string16()
            withSize:[NSFont smallSystemFontSize]
                bold:NO
              toView:contentView.get()
             atPoint:textPosition];
   separatorAfterIdentity_ = [self addSeparatorToView:contentView];
 
-  connectionStatusIcon_ =
-      [self addImageToView:contentView atOffset:kTabViewContentsPadding];
+  connectionStatusIcon_ = [self addImageWithSize:imageSize
+                                          toView:contentView
+                                        atOffset:kTabViewContentsPadding];
   connectionStatusDescriptionField_ =
       [self addText:string16()
            withSize:[NSFont smallSystemFontSize]
                bold:NO
              toView:contentView.get()
             atPoint:textPosition];
   separatorAfterConnection_ = [self addSeparatorToView:contentView];
 
-  firstVisitIcon_ =
-      [self addImageToView:contentView atOffset:kTabViewContentsPadding];
+  firstVisitIcon_ = [self addImageWithSize:imageSize
+                                    toView:contentView
+                                  atOffset:kTabViewContentsPadding];
   firstVisitHeaderField_ =
       [self addText:l10n_util::GetStringUTF16(IDS_PAGE_INFO_SITE_INFO_TITLE)
            withSize:[NSFont smallSystemFontSize]
                bold:YES
              toView:contentView.get()
             atPoint:textPosition];
   firstVisitDescriptionField_ =
       [self addText:string16()
            withSize:[NSFont smallSystemFontSize]
                bold:NO
@@ skipping 17 matching lines @@
 
 // Layout all of the controls in the window. This should be called whenever
 // the content has changed.
 - (void)performLayout {
   // Place the identity status immediately below the identity.
   [self sizeTextFieldHeightToFit:identityField_];
   [self sizeTextFieldHeightToFit:identityStatusField_];
   CGFloat yPos = NSMaxY([identityField_ frame]) + kHeadlineSpacing;
   yPos = [self setYPositionOfView:identityStatusField_ to:yPos];
 
-  // Lay out the connection tab.
+  // Lay out the Permissions tab.
+  NSRect permissionsViewFrame = [permissionsView_ frame];
+  permissionsViewFrame.origin.y = NSMaxY([cookiesView_ frame]);
+  [permissionsView_ setFrame:permissionsViewFrame];
+
+  // Lay out the Connection tab.
 
   // Lay out the identity status section.
   [self sizeTextFieldHeightToFit:identityStatusDescriptionField_];
   yPos = std::max(NSMaxY([identityStatusDescriptionField_ frame]),
                   NSMaxY([identityStatusIcon_ frame]));
   yPos = [self setYPositionOfView:separatorAfterIdentity_
                                to:yPos + kVerticalSpacing];
   yPos += kVerticalSpacing;
 
   // Lay out the connection status section.
@@ skipping 17 matching lines @@
   // Adjust the tab view size and place it below the identity status.
 
   NSRect segmentedControlFrame = [segmentedControl_ frame];
   segmentedControlFrame.origin.y =
       NSMaxY([identityStatusField_ frame]);
   [segmentedControl_ setFrame:segmentedControlFrame];
 
   NSRect tabViewFrame = [tabView_ frame];
   tabViewFrame.origin.y = NSMaxY(segmentedControlFrame);
 
+  // Determine the height of the tab contents.
+
   CGFloat connectionTabHeight = std::max(
       NSMaxY([firstVisitDescriptionField_ frame]),
       NSMaxY([firstVisitIcon_ frame ]));
   connectionTabHeight += kVerticalSpacing;
 
+  CGFloat permissionsTabHeight =
+      NSHeight([cookiesView_ frame]) + NSHeight([permissionsView_ frame]);
+
   CGFloat tabContentHeight = std::max(connectionTabHeight,
-                                      permissionsTabHeight_);
+                                      permissionsTabHeight);
   tabViewFrame.size.height = tabContentHeight +
       NSHeight(tabViewFrame) - NSHeight([tabView_ contentRect]);
   [tabView_ setFrame:tabViewFrame];
 
   // Adjust the contentView to fit everything.
   [contentView_ setFrame:NSMakeRect(
       0, 0, kWindowWidth, NSMaxY([tabView_ frame]))];
 
   // Now adjust and position the window.
 
@@ skipping 57 matching lines @@
 // Create a new text field and add it to the given array of subviews.
 // The array will retain a reference to the object.
 - (NSTextField*)addText:(const string16&)text
                withSize:(CGFloat)fontSize
                    bold:(BOOL)bold
                  toView:(NSView*)view
                 atPoint:(NSPoint)point {
   // Size the text to take up the full available width, with some padding.
   // The height is arbitrary as it will be adjusted later.
   CGFloat width = NSWidth([view frame]) - point.x - kFramePadding;
-  NSRect frame = NSMakeRect(point.x, point.y, width, kImageSize);
+  NSRect frame = NSMakeRect(point.x, point.y, width, 100);
   scoped_nsobject<NSTextField> textField(
      [[NSTextField alloc] initWithFrame:frame]);
   [self configureTextFieldAsLabel:textField.get()];
   [textField setStringValue:base::SysUTF16ToNSString(text)];
   NSFont* font = bold ? [NSFont boldSystemFontOfSize:fontSize]
                       : [NSFont systemFontOfSize:fontSize];
   [textField setFont:font];
   [self sizeTextFieldHeightToFit:textField];
   [view addSubview:textField.get()];
   return textField.get();
 }
 
 // Add an image as a subview of the given view, placed at a pre-determined x
 // position and the given y position. Return the new NSImageView.
-- (NSImageView*)addImageToView:(NSView*)view
-                      atOffset:(CGFloat)offset {
-  NSRect frame = NSMakeRect(kFramePadding, offset, kImageSize, kImageSize);
+- (NSImageView*)addImageWithSize:(NSSize)size
+                          toView:(NSView*)view
+                        atOffset:(CGFloat)offset {
+  NSRect frame = NSMakeRect(kFramePadding, offset, size.width, size.height);
   scoped_nsobject<NSImageView> imageView(
       [[NSImageView alloc] initWithFrame:frame]);
   [imageView setImageFrameStyle:NSImageFrameNone];
   [view addSubview:imageView.get()];
   return imageView.get();
 }
 
 // Add a separator as a subview of the given view. Return the new view.
 - (NSView*)addSeparatorToView:(NSView*)view {
   // Take up almost the full width of the container's frame.
@@ skipping 98 matching lines @@
   if (presenter_.get())
     presenter_->OnSitePermissionChanged(type, newSetting);
 }
 
 // Adds a new row to the UI listing the permissions. Returns the amount of
 // vertical space that was taken up by the row.
 - (CGFloat)addPermission:
     (const WebsiteSettingsUI::PermissionInfo&)permissionInfo
                   toView:(NSView*)view
                  atPoint:(NSPoint)point {
+  // TODO(dubroy): Remove this check by refactoring GetPermissionIcon to take
+  // the PermissionInfo object as its argument.
+  ContentSetting setting = permissionInfo.setting == CONTENT_SETTING_DEFAULT ?
+      permissionInfo.default_setting : permissionInfo.setting;
+  NSImage* image = WebsiteSettingsUI::GetPermissionIcon(
+      permissionInfo.type, setting).ToNSImage();
+  NSImageView* imageView = [self addImageWithSize:[image size]
+                                           toView:view
+                                         atOffset:point.y];
+  [imageView setImage:image];
+  point.x += kPermissionImageSize + kPermissionImageSpacing;
+
   string16 labelText =
       WebsiteSettingsUI::PermissionTypeToUIString(permissionInfo.type) +
       ASCIIToUTF16(":");
 
   NSTextField* label = [self addText:labelText
                             withSize:[NSFont smallSystemFontSize]
                                 bold:NO
                               toView:view
                              atPoint:point];
 
   // Shrink the label to fit the text width.
   NSSize requiredSize = [[label cell] cellSizeForBounds:[label frame]];
   [label setFrameSize:requiredSize];
 
   NSPoint popUpPosition = NSMakePoint(NSMaxX([label frame]), point.y);
   NSPopUpButton* button = [self addPopUpButtonForPermission:permissionInfo
                                                      toView:view
                                                     atPoint:popUpPosition];
 
   // Adjust the vertical position of the button so that its title text is
   // aligned with the label. Assumes that the text is the same size in both.
   NSRect titleRect = [[button cell] titleRectForBounds:[button bounds]];
   popUpPosition.y -= titleRect.origin.y;
   [button setFrameOrigin:popUpPosition];
 
+  // Align the icon with the text.
+  [self alignPermissionIcon:imageView withTextField:label];
+
   return std::max(NSHeight([label frame]), NSHeight([button frame]));
 }
 
+// Align an image with a text field by vertically centering the image on
+// the cap height of the first line of text.
+- (void)alignPermissionIcon:(NSImageView*)imageView
+              withTextField:(NSTextField*)textField {
+  NSFont* font = [textField font];
+
+  // Calculate the offset from the top of the text field.
+  CGFloat capHeight = [font capHeight];
+  CGFloat offset = (kPermissionImageSize - capHeight) / 2 -
+      ([font ascender] - capHeight) - kPermissionImageYAdjust;
+
+  NSRect frame = [imageView frame];
+  frame.origin.y -= offset;
+  [imageView setFrame:frame];
+}
+
+- (CGFloat)addCookieInfo:
+    (const WebsiteSettingsUI::CookieInfo&)cookieInfo
+                  toView:(NSView*)view
+                 atPoint:(NSPoint)point {
+  NSImage* image = WebsiteSettingsUI::GetPermissionIcon(
+      CONTENT_SETTINGS_TYPE_COOKIES, CONTENT_SETTING_ALLOW).ToNSImage();
+  NSImageView* imageView = [self addImageWithSize:[image size]
+                                           toView:view
+                                         atOffset:point.y];
+  [imageView setImage:image];
+  point.x += kPermissionImageSize + kPermissionImageSpacing;
+
+  string16 labelText = l10n_util::GetStringFUTF16(
+      IDS_WEBSITE_SETTINGS_SITE_DATA_STATS_LINE,
+      UTF8ToUTF16(cookieInfo.cookie_source),
+      base::IntToString16(cookieInfo.allowed),
+      base::IntToString16(cookieInfo.blocked));
+
+  NSTextField* label = [self addText:labelText
+                            withSize:[NSFont smallSystemFontSize]
+                                bold:NO
+                              toView:view
+                             atPoint:point];
+
+  // Shrink the label to fit the text width.
+  NSSize requiredSize = [[label cell] cellSizeForBounds:[label frame]];
+  [label setFrameSize:requiredSize];
+
+  // Align the icon with the text.
+  [self alignPermissionIcon:imageView withTextField:label];
+
+  return NSHeight([label frame]);
+}
+
 // Set the content of the identity and identity status fields.
 - (void)setIdentityInfo:(const WebsiteSettingsUI::IdentityInfo&)identityInfo {
   [identityField_ setStringValue:
       base::SysUTF8ToNSString(identityInfo.site_identity)];
   [identityStatusField_ setStringValue:
       base::SysUTF16ToNSString(identityInfo.GetIdentityStatusText())];
 
+  WebsiteSettings::SiteIdentityStatus status = identityInfo.identity_status;
+  if (status == WebsiteSettings::SITE_IDENTITY_STATUS_CERT ||
+      status == WebsiteSettings::SITE_IDENTITY_STATUS_DNSSEC_CERT ||
+      status == WebsiteSettings::SITE_IDENTITY_STATUS_EV_CERT) {
+    [identityStatusField_ setTextColor:IdentityVerifiedTextColor()];
+  }
+
   [identityStatusIcon_ setImage:WebsiteSettingsUI::GetIdentityIcon(
       identityInfo.identity_status).ToNSImage()];
   [identityStatusDescriptionField_ setStringValue:
       base::SysUTF8ToNSString(identityInfo.identity_status_description)];
 
   [connectionStatusIcon_ setImage:WebsiteSettingsUI::GetConnectionIcon(
       identityInfo.connection_status).ToNSImage()];
   [connectionStatusDescriptionField_ setStringValue:
       base::SysUTF8ToNSString(identityInfo.connection_status_description)];
 
   [self performLayout];
 }
 
+- (void)setCookieInfo:(const CookieInfoList&)cookieInfoList {
+  [cookiesView_ setSubviews:[NSArray array]];
+  NSPoint controlOrigin = NSMakePoint(kFramePadding, kFramePadding);
+
+  NSTextField* header = [self addText:ASCIIToUTF16("Cookies")
+                             withSize:[NSFont smallSystemFontSize]
+                                 bold:YES
+                               toView:cookiesView_
+                              atPoint:controlOrigin];
+  [self sizeTextFieldHeightToFit:header];
+  controlOrigin.y += NSHeight([header frame]) + kPermissionsTabSpacing;
+
+  for (CookieInfoList::const_iterator it = cookieInfoList.begin();
+       it != cookieInfoList.end();
+       ++it) {
+    CGFloat rowHeight = [self addCookieInfo:*it
+                                     toView:cookiesView_
+                                    atPoint:controlOrigin];
+    controlOrigin.y += rowHeight + kPermissionsTabSpacing;
+  }
+  [cookiesView_ setFrameSize:NSMakeSize(kWindowWidth, controlOrigin.y)];
+}
+
+- (void)setPermissionInfo:(const PermissionInfoList&)permissionInfoList {
+  [permissionsView_ setSubviews:[NSArray array]];
+  NSPoint controlOrigin = NSMakePoint(kFramePadding, kFramePadding);
+
+  NSTextField* header = [self addText:ASCIIToUTF16("Permissions")
+                             withSize:[NSFont smallSystemFontSize]
+                                 bold:YES
+                               toView:permissionsView_
+                              atPoint:controlOrigin];
+  [self sizeTextFieldHeightToFit:header];
+  controlOrigin.y += NSHeight([header frame]) + kPermissionsTabSpacing;
+
+  for (PermissionInfoList::const_iterator permission =
+           permissionInfoList.begin();
+       permission != permissionInfoList.end();
+       ++permission) {
+    CGFloat rowHeight = [self addPermission:*permission
+                                     toView:permissionsView_
+                                    atPoint:controlOrigin];
+    controlOrigin.y += rowHeight + kPermissionsTabSpacing;
+  }
+  [permissionsView_ setFrameSize:NSMakeSize(kWindowWidth, controlOrigin.y)];
+}
+
 - (void)setFirstVisit:(const string16&)firstVisit {
   [firstVisitIcon_ setImage:
       WebsiteSettingsUI::GetFirstVisitIcon(firstVisit).ToNSImage()];
   [firstVisitDescriptionField_ setStringValue:
       base::SysUTF16ToNSString(firstVisit)];
   [self performLayout];
 }
 
-- (void)setPermissionInfo:(const PermissionInfoList&)permissionInfoList {
-  [permissionsContentView_ setSubviews:[NSArray array]];
-  NSPoint controlOrigin = NSMakePoint(kFramePadding, kFramePadding);
-
-  for (PermissionInfoList::const_iterator permission =
-           permissionInfoList.begin();
-       permission != permissionInfoList.end();
-       ++permission) {
-    CGFloat rowHeight = [self addPermission:*permission
-                                     toView:permissionsContentView_
-                                    atPoint:controlOrigin];
-    controlOrigin.y += rowHeight + kPermissionsTabSpacing;
-  }
-  permissionsTabHeight_ = controlOrigin.y;
-}
-
 @end
 
 WebsiteSettingsUIBridge::WebsiteSettingsUIBridge()
   : bubble_controller_(nil) {
 }
 
 WebsiteSettingsUIBridge::~WebsiteSettingsUIBridge() {
 }
 
 void WebsiteSettingsUIBridge::set_bubble_controller(
@@ skipping 22 matching lines @@
       tab_contents->content_settings(),
       tab_contents->infobar_tab_helper(),
       url,
       ssl,
       content::CertStore::GetInstance());
   [bubble_controller setPresenter:presenter];
 
   [bubble_controller showWindow:nil];
 }
 
+void WebsiteSettingsUIBridge::SetIdentityInfo(
+    const WebsiteSettingsUI::IdentityInfo& identity_info) {
+  [bubble_controller_ setIdentityInfo:identity_info];
+}
+
 void WebsiteSettingsUIBridge::SetCookieInfo(
     const CookieInfoList& cookie_info_list) {
+  [bubble_controller_ setCookieInfo:cookie_info_list];
 }
 
 void WebsiteSettingsUIBridge::SetPermissionInfo(
     const PermissionInfoList& permission_info_list) {
   [bubble_controller_ setPermissionInfo:permission_info_list];
 }
 
-void WebsiteSettingsUIBridge::SetIdentityInfo(
-    const WebsiteSettingsUI::IdentityInfo& identity_info) {
-  [bubble_controller_ setIdentityInfo:identity_info];
-}
-
 void WebsiteSettingsUIBridge::SetFirstVisit(const string16& first_visit) {
   [bubble_controller_ setFirstVisit:first_visit];
 }