cocoa browser: remove non-material toolbar support

chrome/browser/ui/cocoa/toolbar/toolbar_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/toolbar/toolbar_controller.h"
 
 #include <algorithm>
 
 #include "base/mac/bundle_locations.h"
 #include "base/mac/foundation_util.h"
@@ skipping 62 matching lines @@
 using content::OpenURLParams;
 using content::Referrer;
 using content::WebContents;
 
 namespace {
 
 // Duration of the toolbar animation.
 const NSTimeInterval kToolBarAnimationDuration = 0.12;
 
 // The height of the location bar in Material Design.
 const CGFloat kMaterialDesignLocationBarHeight = 28;

[Evan Stade, 2016/09/27 19:34:51]

nit: remove references to MD

 
 // The padding between the top of the toolbar and the top of the
 // location bar.
 const CGFloat kMaterialDesignLocationBarPadding = 2;
 
 // The padding between Material Design elements and the edges of the toolbar.
 const CGFloat kMaterialDesignElementPadding = 4;
 
 // Toolbar buttons are 24x24 and centered in a 28x28 space, so there is a 2pt-
 // wide inset.
@@ skipping 129 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(NotificationBridge);
 };
 
 }  // namespace ToolbarControllerInternal
 
 @implementation ToolbarController
 
 @synthesize browser = browser_;
 
 + (CGFloat)locationBarHeight {
-  if (!ui::MaterialDesignController::IsModeMaterial()) {
-    return 29;
-  }
-
   return kMaterialDesignLocationBarHeight;
 }
 
 + (CGFloat)appMenuLeftPadding {
-  if (!ui::MaterialDesignController::IsModeMaterial()) {
-    return 3;
-  }
-
   return kMaterialDesignElementPadding;
 }
 
 + (CGFloat)materialDesignButtonInset {
   return kMaterialDesignButtonInset;
 }
 
 - (id)initWithCommands:(CommandUpdater*)commands
                profile:(Profile*)profile
                browser:(Browser*)browser
@@ skipping 40 matching lines @@
 
 - (void)viewDidLoad {
   // When linking and running on 10.10+, both -awakeFromNib and -viewDidLoad may
   // be called, don't initialize twice.
   if (locationBarView_) {
     DCHECK(base::mac::IsAtLeastOS10_10());
     return;
   }
 
   // Make Material Design layout adjustments to the NIB items.
-  bool isModeMaterial = ui::MaterialDesignController::IsModeMaterial();
-  if (isModeMaterial) {
-    ToolbarView* toolbarView = [self toolbarView];
-    NSRect toolbarBounds = [toolbarView bounds];
-    NSSize toolbarButtonSize = [ToolbarButton toolbarButtonSize];
+  ToolbarView* toolbarView = [self toolbarView];
+  NSRect toolbarBounds = [toolbarView bounds];
+  NSSize toolbarButtonSize = [ToolbarButton toolbarButtonSize];
 
-    // Set the toolbar height.
-    NSRect frame = [toolbarView frame];
-    frame.size.height = [self baseToolbarHeight];
-    [toolbarView setFrame:frame];
+  // Set the toolbar height.
+  NSRect frame = [toolbarView frame];
+  frame.size.height = [self baseToolbarHeight];
+  [toolbarView setFrame:frame];
 
-    NSRect backButtonFrame = [backButton_ frame];
-    backButtonFrame.origin.x =
-        kMaterialDesignElementPadding + kMaterialDesignButtonInset;
-    backButtonFrame.origin.y = NSMaxY(toolbarBounds) -
-        kMaterialDesignElementPadding - toolbarButtonSize.height;
-    backButtonFrame.size = toolbarButtonSize;
-    [backButton_ setFrame:backButtonFrame];
+  NSRect backButtonFrame = [backButton_ frame];
+  backButtonFrame.origin.x =
+      kMaterialDesignElementPadding + kMaterialDesignButtonInset;
+  backButtonFrame.origin.y = NSMaxY(toolbarBounds) -
+      kMaterialDesignElementPadding - toolbarButtonSize.height;
+  backButtonFrame.size = toolbarButtonSize;
+  [backButton_ setFrame:backButtonFrame];
 
-    NSRect forwardButtonFrame = [forwardButton_ frame];
-    forwardButtonFrame.origin.x =
-        NSMaxX(backButtonFrame) + 2 * kMaterialDesignButtonInset;
-    forwardButtonFrame.origin.y = backButtonFrame.origin.y;
-    forwardButtonFrame.size = toolbarButtonSize;
-    [forwardButton_ setFrame:forwardButtonFrame];
+  NSRect forwardButtonFrame = [forwardButton_ frame];
+  forwardButtonFrame.origin.x =
+      NSMaxX(backButtonFrame) + 2 * kMaterialDesignButtonInset;
+  forwardButtonFrame.origin.y = backButtonFrame.origin.y;
+  forwardButtonFrame.size = toolbarButtonSize;
+  [forwardButton_ setFrame:forwardButtonFrame];
 
-    NSRect reloadButtonFrame = [reloadButton_ frame];
-    reloadButtonFrame.origin.x =
-        NSMaxX(forwardButtonFrame) + 2 * kMaterialDesignButtonInset;
-    reloadButtonFrame.origin.y = forwardButtonFrame.origin.y;
-    reloadButtonFrame.size = toolbarButtonSize;
-    [reloadButton_ setFrame:reloadButtonFrame];
+  NSRect reloadButtonFrame = [reloadButton_ frame];
+  reloadButtonFrame.origin.x =
+      NSMaxX(forwardButtonFrame) + 2 * kMaterialDesignButtonInset;
+  reloadButtonFrame.origin.y = forwardButtonFrame.origin.y;
+  reloadButtonFrame.size = toolbarButtonSize;
+  [reloadButton_ setFrame:reloadButtonFrame];
 
-    NSRect homeButtonFrame = [homeButton_ frame];
-    homeButtonFrame.origin.x =
-        NSMaxX(reloadButtonFrame) + 2 * kMaterialDesignButtonInset;
-    homeButtonFrame.origin.y = reloadButtonFrame.origin.y;
-    homeButtonFrame.size = toolbarButtonSize;
-    [homeButton_ setFrame:homeButtonFrame];
+  NSRect homeButtonFrame = [homeButton_ frame];
+  homeButtonFrame.origin.x =
+      NSMaxX(reloadButtonFrame) + 2 * kMaterialDesignButtonInset;
+  homeButtonFrame.origin.y = reloadButtonFrame.origin.y;
+  homeButtonFrame.size = toolbarButtonSize;
+  [homeButton_ setFrame:homeButtonFrame];
 
-    // Replace the app button from the nib with an AppToolbarButton instance for
-    // Material Design.
-    AppToolbarButton* newMenuButton =
-        [[[AppToolbarButton alloc] initWithFrame:[appMenuButton_ frame]]
-            autorelease];
-    [newMenuButton setAutoresizingMask:[appMenuButton_ autoresizingMask]];
-    [[appMenuButton_ superview] addSubview:newMenuButton];
-    [appMenuButton_ removeFromSuperview];
-    appMenuButton_ = newMenuButton;
+  // Replace the app button from the nib with an AppToolbarButton instance for

[Evan Stade, 2016/09/27 19:34:51]

can you remove AppToolbarButtonCell now too?

+  // Material Design.

[Evan Stade, 2016/09/27 19:34:51]

nit: remove references to material design (since it's the only design)

+  AppToolbarButton* newMenuButton =
+      [[[AppToolbarButton alloc] initWithFrame:[appMenuButton_ frame]]
+          autorelease];
+  [newMenuButton setAutoresizingMask:[appMenuButton_ autoresizingMask]];
+  [[appMenuButton_ superview] addSubview:newMenuButton];
+  [appMenuButton_ removeFromSuperview];
+  appMenuButton_ = newMenuButton;
 
-    // Adjust the menu button's position.
-    NSRect menuButtonFrame = [appMenuButton_ frame];
-    CGFloat menuButtonFrameMaxX =
-        NSMaxX(toolbarBounds) - [ToolbarController appMenuLeftPadding];
-    menuButtonFrame.origin.x =
-        menuButtonFrameMaxX - kMaterialDesignButtonInset -
-            toolbarButtonSize.width;
-    menuButtonFrame.origin.y = homeButtonFrame.origin.y;
-    menuButtonFrame.size = toolbarButtonSize;
-    [appMenuButton_ setFrame:menuButtonFrame];
+  // Adjust the menu button's position.
+  NSRect menuButtonFrame = [appMenuButton_ frame];
+  CGFloat menuButtonFrameMaxX =
+      NSMaxX(toolbarBounds) - [ToolbarController appMenuLeftPadding];
+  menuButtonFrame.origin.x =
+      menuButtonFrameMaxX - kMaterialDesignButtonInset -
+          toolbarButtonSize.width;
+  menuButtonFrame.origin.y = homeButtonFrame.origin.y;
+  menuButtonFrame.size = toolbarButtonSize;
+  [appMenuButton_ setFrame:menuButtonFrame];
 
-    // Adjust the size and location on the location bar to take up the
-    // space between the reload and menu buttons.
-    NSRect locationBarFrame = [locationBar_ frame];
-    locationBarFrame.origin.x = NSMaxX(homeButtonFrame) +
-        kMaterialDesignButtonInset;
-    if (![homeButton_ isHidden]) {
-      // Ensure proper spacing between the home button and the location bar.
-      locationBarFrame.origin.x += kMaterialDesignElementPadding;
-    }
-    locationBarFrame.origin.y = NSMaxY(toolbarBounds) -
-        kMaterialDesignLocationBarPadding - kMaterialDesignLocationBarHeight;
-    locationBarFrame.size.width =
-        menuButtonFrame.origin.x -
-            locationBarFrame.origin.x;
-    locationBarFrame.size.height = kMaterialDesignLocationBarHeight;
-    [locationBar_ setFrame:locationBarFrame];
+  // Adjust the size and location on the location bar to take up the
+  // space between the reload and menu buttons.
+  NSRect locationBarFrame = [locationBar_ frame];
+  locationBarFrame.origin.x = NSMaxX(homeButtonFrame) +
+      kMaterialDesignButtonInset;
+  if (![homeButton_ isHidden]) {
+    // Ensure proper spacing between the home button and the location bar.
+    locationBarFrame.origin.x += kMaterialDesignElementPadding;
+  }
+  locationBarFrame.origin.y = NSMaxY(toolbarBounds) -
+      kMaterialDesignLocationBarPadding - kMaterialDesignLocationBarHeight;
+  locationBarFrame.size.width =
+      menuButtonFrame.origin.x -
+          locationBarFrame.origin.x;
+  locationBarFrame.size.height = kMaterialDesignLocationBarHeight;
+  [locationBar_ setFrame:locationBarFrame];
 
-    // Correctly position the extension buttons' container view.
-    NSRect containerFrame = [browserActionsContainerView_ frame];
-    containerFrame.size.width += kMaterialDesignButtonInset;
-    containerFrame.origin.y =
-        locationBarFrame.origin.y + kMaterialDesignContainerYOffset;
-    containerFrame.size.height = toolbarButtonSize.height;
-    [browserActionsContainerView_ setFrame:containerFrame];
-  } else {
-    [[backButton_ cell] setImageID:IDR_BACK
-                    forButtonState:image_button_cell::kDefaultState];
-    [[backButton_ cell] setImageID:IDR_BACK_H
-                    forButtonState:image_button_cell::kHoverState];
-    [[backButton_ cell] setImageID:IDR_BACK_P
-                    forButtonState:image_button_cell::kPressedState];
-    [[backButton_ cell] setImageID:IDR_BACK_D
-                    forButtonState:image_button_cell::kDisabledState];
-
-    [[forwardButton_ cell] setImageID:IDR_FORWARD
-                       forButtonState:image_button_cell::kDefaultState];
-    [[forwardButton_ cell] setImageID:IDR_FORWARD_H
-                       forButtonState:image_button_cell::kHoverState];
-    [[forwardButton_ cell] setImageID:IDR_FORWARD_P
-                       forButtonState:image_button_cell::kPressedState];
-    [[forwardButton_ cell] setImageID:IDR_FORWARD_D
-                       forButtonState:image_button_cell::kDisabledState];
-
-    [[reloadButton_ cell] setImageID:IDR_RELOAD
-                      forButtonState:image_button_cell::kDefaultState];
-    [[reloadButton_ cell] setImageID:IDR_RELOAD_H
-                      forButtonState:image_button_cell::kHoverState];
-    [[reloadButton_ cell] setImageID:IDR_RELOAD_P
-                      forButtonState:image_button_cell::kPressedState];
-
-    [[homeButton_ cell] setImageID:IDR_HOME
-                    forButtonState:image_button_cell::kDefaultState];
-    [[homeButton_ cell] setImageID:IDR_HOME_H
-                    forButtonState:image_button_cell::kHoverState];
-    [[homeButton_ cell] setImageID:IDR_HOME_P
-                    forButtonState:image_button_cell::kPressedState];
-
-    [[appMenuButton_ cell] setImageID:IDR_TOOLS
-                       forButtonState:image_button_cell::kDefaultState];
-    [[appMenuButton_ cell] setImageID:IDR_TOOLS_H
-                       forButtonState:image_button_cell::kHoverState];
-    [[appMenuButton_ cell] setImageID:IDR_TOOLS_P
-                       forButtonState:image_button_cell::kPressedState];
-
-    // Adjust the toolbar height if running on Retina - see the comment in
-    // -baseToolbarHeight.
-    CGFloat toolbarHeight = [self baseToolbarHeight];
-    ToolbarView* toolbarView = [self toolbarView];
-    NSRect toolbarFrame = [toolbarView frame];
-    if (toolbarFrame.size.height != toolbarHeight) {
-      toolbarFrame.size.height = toolbarHeight;
-      [toolbarView setFrame:toolbarFrame];
-    }
-  }
+  // Correctly position the extension buttons' container view.
+  NSRect containerFrame = [browserActionsContainerView_ frame];
+  containerFrame.size.width += kMaterialDesignButtonInset;
+  containerFrame.origin.y =
+      locationBarFrame.origin.y + kMaterialDesignContainerYOffset;
+  containerFrame.size.height = toolbarButtonSize.height;
+  [browserActionsContainerView_ setFrame:containerFrame];
 
   notificationBridge_.reset(
       new ToolbarControllerInternal::NotificationBridge(self));
   notificationBridge_->UpdateSeverity();
 
   [appMenuButton_ setOpenMenuOnClick:YES];
 
   [backButton_ setOpenMenuOnRightClick:YES];
   [forwardButton_ setOpenMenuOnRightClick:YES];
 
@@ skipping 15 matching lines @@
   // The |locationBarView_| is responsible for positioning the accessibility
   // views.
   std::vector<NSView*> accessibility_views =
       locationBarView_->GetDecorationAccessibilityViews();
   for (const auto& view : accessibility_views) {
     [[self toolbarView] addSubview:view
                         positioned:NSWindowAbove
                         relativeTo:locationBar_];
   }
 
-  if (!isModeMaterial) {
-    [locationBar_ setFont:[NSFont systemFontOfSize:[NSFont systemFontSize]]];
-  }
-
   // Register pref observers for the optional home and page/options buttons
   // and then add them to the toolbar based on those prefs.
   PrefService* prefs = profile_->GetPrefs();
   showHomeButton_.Init(
       prefs::kShowHomeButton, prefs,
       base::Bind(
           &ToolbarControllerInternal::NotificationBridge::OnPreferenceChanged,
           base::Unretained(notificationBridge_.get())));
   [self showOptionalHomeButton];
   [self installAppMenu];
@@ skipping 11 matching lines @@
                    button:forwardButton_]);
 
   trackingArea_.reset(
       [[CrTrackingArea alloc] initWithRect:NSZeroRect // Ignored
                                    options:NSTrackingMouseMoved |
                                            NSTrackingInVisibleRect |
                                            NSTrackingMouseEnteredAndExited |
                                            NSTrackingActiveAlways
                                      owner:self
                                   userInfo:nil]);
-  NSView* toolbarView = [self view];
-  [toolbarView addTrackingArea:trackingArea_.get()];
+  NSView* parentView = [self view];
+  [parentView addTrackingArea:trackingArea_.get()];
 
   // If the user has any Browser Actions installed, the container view for them
   // may have to be resized depending on the width of the toolbar frame.
-  [toolbarView setPostsFrameChangedNotifications:YES];
+  [parentView setPostsFrameChangedNotifications:YES];
   [[NSNotificationCenter defaultCenter]
       addObserver:self
          selector:@selector(toolbarFrameChanged)
              name:NSViewFrameDidChangeNotification
-           object:toolbarView];
+           object:parentView];
 
   // Set ViewIDs for toolbar elements which don't have their dedicated class.
   // ViewIDs of |toolbarView|, |reloadButton_|, |locationBar_| and
   // |browserActionsContainerView_| are handled by themselves.
   view_id_util::SetID(backButton_, VIEW_ID_BACK_BUTTON);
   view_id_util::SetID(forwardButton_, VIEW_ID_FORWARD_BUTTON);
   view_id_util::SetID(homeButton_, VIEW_ID_HOME_BUTTON);
   view_id_util::SetID(appMenuButton_, VIEW_ID_APP_MENU);
 
   [self addAccessibilityDescriptions];
@@ skipping 262 matching lines @@
   if (!hasToolbar_)
     return;
   BOOL hide = showHomeButton_.GetValue() ? NO : YES;
   if (hide == [homeButton_ isHidden])
     return;  // Nothing to do, view state matches pref state.
 
   // Always shift the text field by the width of the home button minus one pixel
   // since the frame edges of each button are right on top of each other. When
   // hiding the button, reverse the direction of the movement (to the left).
   CGFloat moveX = [homeButton_ frame].size.width;
-  if (!ui::MaterialDesignController::IsModeMaterial()) {
-    moveX -= 1.0;
-  } else {
-    // Ensure proper spacing between the home button and the location bar.
-    moveX += kMaterialDesignElementPadding;
-  }
+  // Ensure proper spacing between the home button and the location bar.
+  moveX += kMaterialDesignElementPadding;
   if (hide)
     moveX *= -1;  // Reverse the direction of the move.
 
   [locationBar_ setFrame:[self adjustRect:[locationBar_ frame]
                                  byAmount:moveX]];
   [homeButton_ setHidden:hide];
 }
 
 // Install the app menu buttons. Calling this repeatedly is inexpensive so it
 // can be done every time the buttons are shown.
 - (void)installAppMenu {
   if (appMenuController_.get())
     return;
 
   appMenuController_.reset(
       [[AppMenuController alloc] initWithBrowser:browser_]);
   [appMenuController_ setUseWithPopUpButtonCell:YES];
   [appMenuButton_ setAttachedMenu:[appMenuController_ menu]];
 }
 
 - (void)updateAppMenuButtonSeverity:(AppMenuIconPainter::Severity)severity
                            iconType:(AppMenuIconController::IconType)iconType
                             animate:(BOOL)animate {
-  if (!ui::MaterialDesignController::IsModeMaterial()) {
-    AppToolbarButtonCell* cell =
-        base::mac::ObjCCastStrict<AppToolbarButtonCell>([appMenuButton_ cell]);
-    [cell setSeverity:severity shouldAnimate:animate];
-    return;
-  }
   AppToolbarButton* appMenuButton =
       base::mac::ObjCCastStrict<AppToolbarButton>(appMenuButton_);
   [appMenuButton setSeverity:severity iconType:iconType shouldAnimate:animate];
 }
 
 - (void)prefChanged:(const std::string&)prefName {
   if (prefName == prefs::kShowHomeButton) {
     [self showOptionalHomeButton];
   }
 }
@@ skipping 72 matching lines @@
 
   if ([browserActionsContainerView_ isHidden]) {
     CGFloat edgeXPos = [appMenuButton_ frame].origin.x;
     leftDistance = edgeXPos - locationBarXPos -
         [ToolbarController appMenuLeftPadding];
   } else {
     leftDistance = NSMinX([browserActionsContainerView_ animationEndFrame]) -
         locationBarXPos;
     // Equalize the distance between the location bar and the first extension
     // button, and the distance between the location bar and home/reload button.
-    if (ui::MaterialDesignController::IsModeMaterial()) {
-      leftDistance -= kMaterialDesignButtonInset;
-    }
+    leftDistance -= kMaterialDesignButtonInset;
   }
   if (leftDistance != 0.0)
     [self adjustLocationSizeBy:leftDistance animate:animate];
   else
     [locationBar_ stopAnimation];
 }
 
 - (void)maintainMinimumLocationBarWidth {
   CGFloat locationBarWidth = NSWidth([locationBar_ frame]);
   locationBarAtMinSize_ = locationBarWidth <= kMinimumLocationBarWidth;
@@ skipping 10 matching lines @@
     return;
 
   if ([browserActionsContainerView_ isAnimating]) {
     // If the browser actions container is animating, we need to stop it first,
     // because the frame it's animating for could be incorrect with the new
     // bounds (if, for instance, the bookmark bar was added).
     // This will advance to the end of the animation, so we also need to adjust
     // it afterwards.
     [browserActionsContainerView_ stopAnimation];
     NSRect containerFrame = [browserActionsContainerView_ frame];
-    if (!ui::MaterialDesignController::IsModeMaterial()) {
-      CGFloat elementTopPadding =
-          kMaterialDesignElementPadding + kMaterialDesignButtonInset;
-      // Pre-Material Design, this value is calculated from the values in
-      // Toolbar.xib: the height of the toolbar (35) minus the height of the
-      // child elements (29) minus the y-origin of the elements (4).
-      elementTopPadding = 2;
-      containerFrame.origin.y =
-          NSHeight([[self view] frame]) - NSHeight(containerFrame) -
-          elementTopPadding;
-    } else {
-      containerFrame.origin.y =
-          [locationBar_ frame].origin.y + kMaterialDesignContainerYOffset;
-    }
+    containerFrame.origin.y =
+        [locationBar_ frame].origin.y + kMaterialDesignContainerYOffset;
     [browserActionsContainerView_ setFrame:containerFrame];
     [self pinLocationBarToLeftOfBrowserActionsContainerAndAnimate:NO];
   }
 
   [self maintainMinimumLocationBarWidth];
 
   if (locationBarAtMinSize_) {
     // Once the grippy is pinned, leave it until it is explicity un-pinned.
     [browserActionsContainerView_ setGrippyPinned:YES];
     NSRect containerFrame = [browserActionsContainerView_ frame];
@@ skipping 84 matching lines @@
 - (NSPoint)saveCreditCardBubblePoint {
   return locationBarView_->GetSaveCreditCardBubblePoint();
 }
 
 - (NSPoint)translateBubblePoint {
   return locationBarView_->GetTranslateBubblePoint();
 }
 
 - (CGFloat)baseToolbarHeight {
   // Height of the toolbar in pixels when the bookmark bar is closed.
-  const bool kIsModeMaterial = ui::MaterialDesignController::IsModeMaterial();
-  const CGFloat kBaseToolbarHeightNormal = kIsModeMaterial ? 37 : 35;
-
-  // Not all lines are drawn at 2x normal height when running on Retina, which
-  // causes the toolbar controls to be visually 1pt too high within the toolbar
-  // area. It's not possible to adjust the control y-positions by 0.5pt and have
-  // them appear 0.5pt lower (they are still drawn at their original locations),
-  // so instead shave off 1pt from the bottom of the toolbar. Note that there's
-  // an offsetting change in -[BookmarkBarController preferredHeight] to
-  // maintain the proper spacing between bookmark icons and toolbar items. See
-  // https://crbug.com/326245 .
-  const CGFloat kLineWidth = [[self view] cr_lineWidth];
-  const BOOL kIsRetina = (kLineWidth < 1);
-  BOOL reduceHeight = NO;
-
-  // Only adjust the height if Retina and not Material Design.
-  reduceHeight = kIsRetina && !kIsModeMaterial;
-
-  return reduceHeight ? kBaseToolbarHeightNormal - 1
-                      : kBaseToolbarHeightNormal;
+  const CGFloat kBaseToolbarHeightNormal = 37;
+  return kBaseToolbarHeightNormal;
 }
 
 - (CGFloat)desiredHeightForCompression:(CGFloat)compressByHeight {
   // With no toolbar, just ignore the compression.
   if (!hasToolbar_)
     return NSHeight([locationBar_ frame]);
 
   return [self baseToolbarHeight] - compressByHeight;
 }
 
@@ skipping 82 matching lines @@
 - (void)hideDropURLsIndicatorInView:(NSView*)view {
   // Do nothing.
 }
 
 // (URLDropTargetController protocol)
 - (BOOL)isUnsupportedDropData:(id<NSDraggingInfo>)info {
   return drag_util::IsUnsupportedDropData(profile_, info);
 }
 
 @end