cocoa browser: remove non-MD bookmark bar support

chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller_unittest.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 <Cocoa/Cocoa.h>
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/command_line.h"
 #include "base/mac/mac_util.h"
@@ skipping 1000 matching lines @@
 
   const BookmarkNode* parent = model->bookmark_bar_node();
   model->AddURL(parent, parent->child_count(),
                 ASCIIToUTF16("title"), GURL("http://one.com"));
 
   [bar_ loaded:model];
 
   // Make sure the internal bookmark button also is the correct height.
   NSArray* buttons = [bar_ buttons];
   EXPECT_GT([buttons count], 0u);
-  const bool kIsModeMaterial = ui::MaterialDesignController::IsModeMaterial();
 
   for (NSButton* button in buttons) {
-    if (kIsModeMaterial) {
-      EXPECT_FLOAT_EQ((chrome::kMinimumBookmarkBarHeight +
-                       bookmarks::kMaterialVisualHeightOffset) -
-                          2 * bookmarks::BookmarkVerticalPadding(),
-                      [button frame].size.height);
-    } else {
-      EXPECT_FLOAT_EQ(
-          (chrome::kMinimumBookmarkBarHeight + bookmarks::kVisualHeightOffset) -
-              2 * bookmarks::BookmarkVerticalPadding(),
-          [button frame].size.height);
-    }
+    EXPECT_FLOAT_EQ((chrome::kMinimumBookmarkBarHeight +
+                     bookmarks::kMaterialVisualHeightOffset) -
+                        2 * bookmarks::kBookmarkVerticalPadding,
+                    [button frame].size.height);
   }
 }
 
 TEST_F(BookmarkBarControllerTest, DropBookmarks) {
   const char* urls[] = {
     "http://qwantz.com",
     "http://xkcd.com",
     "javascript:alert('lolwut')",
     "file://localhost/tmp/local-file.txt"  // As if dragged from the desktop.
   };
@@ skipping 559 matching lines @@
   // |view_widths[1]| and |view_widths[4]| which will cause a visible button
   // transition from 1 -> 2 and 2 -> 3, respectively. With those two widths you
   // can easily compute all widths from 0 through 6. |view_widths[7]| is always
   // 200.0, and the remainder are the reverse of widths 0 through 6. When all
   // three buttons are visible, be sure to sanity check with frames (i.e. under
   // MD the first button should start at x=10, and there should be 16pt of space
   // between the buttons).
   //
   // The default font changed between OSX Mavericks, OSX Yosemite, and
   // OSX El Capitan, so this test requires different widths to trigger the
-  // appropriate results. Button widths and locations also changed with
-  // Material Design.
+  // appropriate results.
   CGFloat view_widths_el_capitan[] =
-      { 121.0, 122.0, 149.0, 150.0, 151.0, 152.0,
-        153.0, 200.0, 153.0, 152.0, 151.0, 150.0,
-        149.0, 122.0, 121.0 };
-  CGFloat view_widths_yosemite[] =
-      { 121.0, 122.0, 148.0, 149.0, 150.0, 151.0,
-        152.0, 200.0, 152.0, 151.0, 150.0, 149.0,
-        148.0, 122.0, 121.0 };
-  CGFloat view_widths_rest[] =
-      { 123.0, 124.0, 151.0, 152.0, 153.0, 154.0,
-        155.0, 200.0, 155.0, 154.0, 153.0, 152.0,
-        151.0, 124.0, 123.0 };
-  CGFloat material_view_widths_el_capitan[] =
       { 139.0, 140.0, 150.0, 151.0, 152.0, 153.0,
         154.0, 200.0, 154.0, 153.0, 152.0, 151.0,
         150.0, 140.0, 139.0 };
-  CGFloat material_view_widths_yosemite[] =
+  CGFloat view_widths_yosemite[] =
       { 140.0, 141.0, 150.0, 151.0, 152.0, 153.0,
         154.0, 200.0, 154.0, 153.0, 152.0, 151.0,
         150.0, 141.0, 140.0 };
-  CGFloat material_view_widths_rest[] =
+  CGFloat view_widths_rest[] =
       { 142.0, 143.0, 153.0, 154.0, 155.0, 156.0,
         157.0, 200.0, 157.0, 156.0, 155.0, 154.0,
         153.0, 143.0, 142.0 };
   CGFloat* view_widths = NULL;
-  bool is_mode_material = ui::MaterialDesignController::IsModeMaterial();
   if (base::mac::IsOS10_11()) {
-    view_widths = is_mode_material ? material_view_widths_el_capitan
-                                   : view_widths_el_capitan;
+    view_widths = view_widths_el_capitan;
   } else if (base::mac::IsOS10_10()) {
-    view_widths = is_mode_material ? material_view_widths_yosemite
-                                   : view_widths_yosemite;
+    view_widths = view_widths_yosemite;
   } else {
-    view_widths = is_mode_material ? material_view_widths_rest
-                                   : view_widths_rest;
+    view_widths = view_widths_rest;
   }
 
   BOOL off_the_side_button_is_hidden_results[] =
       { NO, NO, NO, NO, YES, YES, YES, YES, YES, YES, YES, NO, NO, NO, NO};
   int displayed_button_count_results[] =
       { 1, 2, 2, 2, 3, 3, 3, 3, 3, 3, 3, 2, 2, 2, 1 };
   for (unsigned int i = 0; i < arraysize(view_widths_yosemite); ++i) {
     NSRect frame = [[bar_ view] frame];
-    frame.size.width = view_widths[i] + bookmarks::BookmarkRightMargin();
+    frame.size.width = view_widths[i] + bookmarks::kBookmarkRightMargin;
     [[bar_ view] setFrame:frame];
     EXPECT_EQ(off_the_side_button_is_hidden_results[i],
               [bar_ offTheSideButtonIsHidden]);
     EXPECT_EQ(displayed_button_count_results[i], [bar_ displayedButtonCount]);
   }
 }
 
 TEST_F(BookmarkBarControllerTest, BookmarksWithAppsPageShortcut) {
   BookmarkModel* model = BookmarkModelFactory::GetForBrowserContext(profile());
   const BookmarkNode* root = model->bookmark_bar_node();
@@ skipping 437 matching lines @@
   ASSERT_TRUE([bar_ appsPageShortcutButtonIsHidden]);
 
   // Validate initial model.
   std::string actualModel = bookmarks::test::ModelStringFromNode(root);
   EXPECT_EQ(model_string, actualModel);
 
   // Test a series of points starting at the right edge of the bar.
   BookmarkButton* targetButton = [bar_ buttonWithTitleEqualTo:@"1b"];
   ASSERT_TRUE(targetButton);
   NSPoint targetPoint = [targetButton left];
-  CGFloat leftMarginIndicatorPosition = bookmarks::BookmarkLeftMargin() - 0.5 *
-                                        bookmarks::BookmarkHorizontalPadding();
+  CGFloat leftMarginIndicatorPosition = bookmarks::kBookmarkLeftMargin - 0.5 *
+                                        bookmarks::kBookmarkHorizontalPadding;
   const CGFloat baseOffset = targetPoint.x;
   CGFloat expected = leftMarginIndicatorPosition;
   CGFloat actual = [bar_ indicatorPosForDragToPoint:targetPoint];
   EXPECT_CGFLOAT_EQ(expected, actual);
   targetButton = [bar_ buttonWithTitleEqualTo:@"2f"];
   actual = [bar_ indicatorPosForDragToPoint:[targetButton right]];
   targetButton = [bar_ buttonWithTitleEqualTo:@"3b"];
   expected = [targetButton left].x - baseOffset + leftMarginIndicatorPosition;
   EXPECT_CGFLOAT_EQ(expected, actual);
   targetButton = [bar_ buttonWithTitleEqualTo:@"4b"];
   targetPoint = [targetButton right];
   targetPoint.x += 100;  // Somewhere off to the right.
-  CGFloat xDelta = 0.5 * bookmarks::BookmarkHorizontalPadding();
+  CGFloat xDelta = 0.5 * bookmarks::kBookmarkHorizontalPadding;
   expected = NSMaxX([targetButton frame]) + xDelta;
   actual = [bar_ indicatorPosForDragToPoint:targetPoint];
   EXPECT_CGFLOAT_EQ(expected, actual);
 }
 
 TEST_F(BookmarkBarControllerDragDropTest, PulseButton) {
   BookmarkModel* model = BookmarkModelFactory::GetForBrowserContext(profile());
   const BookmarkNode* root = model->bookmark_bar_node();
   GURL gurl("http://www.google.com");
   const BookmarkNode* node = model->AddURL(root, root->child_count(),
@@ skipping 63 matching lines @@
                              "2f3b ] 4b ");
   actual = bookmarks::test::ModelStringFromNode(root);
   EXPECT_EQ(expected, actual);
 
   // Verify that the other bookmark folder can't be deleted.
   BookmarkButton *otherButton = [bar_ otherBookmarksButton];
   EXPECT_FALSE([bar_ canDragBookmarkButtonToTrash:otherButton]);
 }
 
 }  // namespace