Alternate NTP: Limit width of tab titles in recent tabs menu

chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc

 // 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.
 
 #include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h"
 
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/sessions/session_types.h"
 #include "chrome/browser/sessions/session_types_test_helper.h"
 #include "chrome/browser/sync/glue/session_model_associator.h"
@@ skipping 100 matching lines @@
   base::Time timestamp = base::Time::Now();
   const base::TimeDelta time_delta = base::TimeDelta::FromMinutes(10);
 
   RecentTabsBuilderTestHelper recent_tabs_builder;
 
   // Create 1st session : 1 window, 3 tabs
   recent_tabs_builder.AddSession();
   recent_tabs_builder.AddWindow(0);
   for (int i = 0; i < 3; ++i) {
     timestamp -= time_delta;
-    recent_tabs_builder.AddTabWithTimestamp(0, 0, timestamp);
+    recent_tabs_builder.AddTabWithInfo(0, 0, timestamp, string16());
   }
 
   // Create 2nd session : 2 windows, 1 tab in 1st window, 2 tabs in 2nd window
   recent_tabs_builder.AddSession();
   recent_tabs_builder.AddWindow(1);
   recent_tabs_builder.AddWindow(1);
   timestamp -= time_delta;
-  recent_tabs_builder.AddTabWithTimestamp(1, 0, timestamp);
+  recent_tabs_builder.AddTabWithInfo(1, 0, timestamp, string16());
   timestamp -= time_delta;
-  recent_tabs_builder.AddTabWithTimestamp(1, 1, timestamp);
+  recent_tabs_builder.AddTabWithInfo(1, 1, timestamp, string16());
   timestamp -= time_delta;
-  recent_tabs_builder.AddTabWithTimestamp(1, 1, timestamp);
+  recent_tabs_builder.AddTabWithInfo(1, 1, timestamp, string16());
 
   recent_tabs_builder.RegisterRecentTabs(&associator_);
 
   // Verify that data is populated correctly in RecentTabsSubMenuModel.
   // Expected menu:
   // - first inserted tab is most recent and hence is top
   // Menu index  Menu items
   // --------------------------------------
   // 0           Reopen closed tab
   // 1           <separator>
@@ skipping 86 matching lines @@
 
   TestRecentTabsSubMenuModel model(NULL, browser(), &associator_, true);
   int num_items = model.GetItemCount();
   EXPECT_EQ(7, num_items);
 
   std::vector<string16> tab_titles =
       recent_tabs_builder.GetTabTitlesSortedByRecency();
   for (int i = 0; i < 4; ++i)
     EXPECT_EQ(tab_titles[i], model.GetLabelAt(i + 3));
 }
+
+TEST_F(RecentTabsSubMenuModelTest, MaxWidth) {
+  // Create 1 sessions with 1 window and 1 tab.

[kuan, 2012/11/21 15:52:43]

nit: s/sessions/session/

[sail, 2012/11/21 19:28:30]

Done.

+  RecentTabsBuilderTestHelper recent_tabs_builder;
+  recent_tabs_builder.AddSession();
+  recent_tabs_builder.AddWindow(0);
+  recent_tabs_builder.AddTab(0, 0);
+  recent_tabs_builder.RegisterRecentTabs(&associator_);
+
+  // Menu index  Menu items
+  // --------------------------------------
+  // 0           Reopen closed tab
+  // 1           <separator>
+  // 2           <section header for 1st session>
+  // 3           <the only tab of the only window of session 1>
+
+  TestRecentTabsSubMenuModel model(NULL, browser(), &associator_, true);
+  EXPECT_EQ(4, model.GetItemCount());
+  EXPECT_EQ(-1, model.GetMaxWidthForItemAtIndex(0));
+  EXPECT_NE(-1, model.GetMaxWidthForItemAtIndex(1));
+  EXPECT_NE(-1, model.GetMaxWidthForItemAtIndex(2));
+  EXPECT_NE(-1, model.GetMaxWidthForItemAtIndex(3));
+}
+
+TEST_F(RecentTabsSubMenuModelTest, MaxWidthNoDevices) {
+  // Expected menu:
+  // Menu index  Menu items
+  // --------------------------------------
+  // 0           Reopen closed tab
+  // 1           <separator>
+  // 2           No tabs from other Devices
+
+  TestRecentTabsSubMenuModel model(NULL, browser(), NULL, false);
+  EXPECT_EQ(3, model.GetItemCount());
+  EXPECT_EQ(-1, model.GetMaxWidthForItemAtIndex(0));
+  EXPECT_NE(-1, model.GetMaxWidthForItemAtIndex(1));
+  EXPECT_EQ(-1, model.GetMaxWidthForItemAtIndex(2));
+}