Change popup construction logic so that folders with empty name is no longer used for excluding the root node from display, rather, user is_root() for that purpose. A folder with an empty name is now shown as a blank line in the popup.

BUG=28313
TEST=Create a new folder with a blank name and add an URL to that folder. Select the new bookmark. Bring up the bubble and click either 'Edit' or 'Close'. It should not crash at this point.  Also, click on the popup and a blank line should appear representing the newly created folder.

Review URL: http://codereview.chromium.org/418027

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32681 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 36 insertions, 0 deletions

diff --git a/chrome/browser/cocoa/bookmark_bubble_controller_unittest.mm b/chrome/browser/cocoa/bookmark_bubble_controller_unittest.mm
index 76b8d38..9f037da 100644
--- a/chrome/browser/cocoa/bookmark_bubble_controller_unittest.mm
+++ b/chrome/browser/cocoa/bookmark_bubble_controller_unittest.mm
@@ -129,6 +129,42 @@ TEST_F(BookmarkBubbleControllerTest, TestFillInFolder) {
   EXPECT_FALSE([titles containsObject:@"title2"]);
 }
 
+// Confirm ability to handle folders with blank name.
+TEST_F(BookmarkBubbleControllerTest, TestFolderWithBlankName) {
+  // Create some folders, including a nested folder
+  BookmarkModel* model = GetBookmarkModel();
+  EXPECT_TRUE(model);
+  const BookmarkNode* bookmarkBarNode = model->GetBookmarkBarNode();
+  EXPECT_TRUE(bookmarkBarNode);
+  const BookmarkNode* node1 = model->AddGroup(bookmarkBarNode, 0, L"one");
+  EXPECT_TRUE(node1);
+  const BookmarkNode* node2 = model->AddGroup(bookmarkBarNode, 1, L"");
+  EXPECT_TRUE(node2);
+  const BookmarkNode* node3 = model->AddGroup(bookmarkBarNode, 2, L"three");
+  EXPECT_TRUE(node3);
+  const BookmarkNode* node2_1 =
+      model->AddURL(node2, 0, L"title1", GURL("http://www.google.com"));
+  EXPECT_TRUE(node2_1);
+
+  BookmarkBubbleController* controller = ControllerForNode(node1);
+  EXPECT_TRUE(controller);
+
+  // One of the items should be blank and its node should be node2.
+  NSArray* items = [[controller folderPopUpButton] itemArray];
+  EXPECT_EQ(6U, [items count]);
+  BOOL blankFolderFound = NO;
+  for (NSMenuItem* item in [[controller folderPopUpButton] itemArray]) {
+    if ([[item title] length] == 0 &&
+        static_cast<const BookmarkNode*>([[item representedObject]
+                                          pointerValue]) == node2) {
+      blankFolderFound = YES;
+      break;
+    }
+  }
+  EXPECT_TRUE(blankFolderFound);
+}
+
+
 // Click on edit; bubble gets closed.
 TEST_F(BookmarkBubbleControllerTest, TestEdit) {
   BookmarkModel* model = GetBookmarkModel();