[Mac] Bookmark button cell updates

chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell_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 "chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell.h"
 #include "base/mac/scoped_nsobject.h"
+#include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
-#import "chrome/browser/ui/cocoa/bookmarks/bookmark_button_cell.h"
 #include "chrome/browser/ui/cocoa/test/cocoa_profile_test.h"
 #import "chrome/browser/ui/cocoa/test/cocoa_test_helper.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
 #include "ui/base/material_design/material_design_controller.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image.h"
 #include "ui/resources/grit/ui_resources.h"
@@ skipping 95 matching lines @@
   base::scoped_nsobject<BookmarkButtonCell> cell(
       [[BookmarkButtonCell alloc] initTextCell:@"Testing"]);
 
   const BookmarkNode* node = model->bookmark_bar_node();
   [cell setBookmarkNode:node];
   EXPECT_EQ(node, [cell bookmarkNode]);
 
   node = model->other_node();
   [cell setBookmarkNode:node];
   EXPECT_EQ(node, [cell bookmarkNode]);
+  EXPECT_EQ(node->GetTitle(), base::SysNSStringToUTF16([cell title]));
 }
 
 TEST_F(BookmarkButtonCellTest, BookmarkMouseForwarding) {
   base::scoped_nsobject<BookmarkButtonCell> cell(
       [[BookmarkButtonCell alloc] initTextCell:@"Testing"]);
   base::scoped_nsobject<ButtonRemembersMouseEnterExit> button(
       [[ButtonRemembersMouseEnterExit alloc]
           initWithFrame:NSMakeRect(0, 0, 50, 50)]);
   [button setCell:cell.get()];
   EXPECT_EQ(0, button.get()->enters_);
@@ skipping 72 matching lines @@
 
   EXPECT_NE([bookmark_cell verticalTextOffset],
             [gradient_cell verticalTextOffset]);
 }
 
 TEST_F(BookmarkButtonCellTest, OffTheSideCell) {
   Class offTheSideButtonCellClass = NSClassFromString(@"OffTheSideButtonCell");
   EXPECT_TRUE([[BookmarkButtonCell offTheSideButtonCell]
       isKindOfClass:offTheSideButtonCellClass]);
 }
+
+TEST_F(BookmarkButtonCellTest, CellWidthForNode) {
+  BookmarkModel* model = BookmarkModelFactory::GetForBrowserContext(profile());
+
+  const BookmarkNode* bar = model->bookmark_bar_node();
+  const BookmarkNode* node_a =
+      model->AddURL(bar, bar->child_count(), base::ASCIIToUTF16("A cell"),
+                    GURL("http://www.google.com"));
+  const BookmarkNode* node_b =
+      model->AddURL(bar, bar->child_count(),
+                    base::ASCIIToUTF16("This cell has a longer title "),
+                    GURL("http://www.google.com/a"));
+  ResourceBundle& rb = ResourceBundle::GetSharedInstance();
+  base::scoped_nsobject<NSImage> image(
+      rb.GetNativeImageNamed(IDR_DEFAULT_FAVICON).CopyNSImage());
+  CGFloat width_for_node_a =
+      [BookmarkButtonCell cellWidthForNode:node_a image:image];
+  CGFloat width_for_node_b =
+      [BookmarkButtonCell cellWidthForNode:node_b image:image];
+  base::scoped_nsobject<BookmarkButtonCell> bookmark_cell(
+      [[BookmarkButtonCell alloc] initForNode:node_a
+                                         text:nil
+                                        image:image
+                               menuController:nil]);
+  EXPECT_CGFLOAT_EQ([bookmark_cell cellSize].width, width_for_node_a);

[Elly Fong-Jones, 2017/04/04 15:04:36]

what is this actually testing?

[lgrey, 2017/04/04 15:12:27]

|width_for_node_a| comes from the class method, |cellSize| is from the
instantiated cell. Basically: is the width we said it would be the width it
ended up being?

[Elly Fong-Jones, 2017/04/05 11:26:37]

Got it - can you add a comment to that effect?

[lgrey, 2017/04/05 15:21:56]

Done.

+  [bookmark_cell setBookmarkNode:node_b image:image];
+  EXPECT_CGFLOAT_EQ([bookmark_cell cellSize].width, width_for_node_b);
+}
 }  // namespace