Add new option to open history page to show more other devices

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 163 matching lines @@
   RecentTabsBuilderTestHelper recent_tabs_builder;
   for (int s = 0; s < 4; ++s) {
     recent_tabs_builder.AddSession();
     recent_tabs_builder.AddWindow(s);
     recent_tabs_builder.AddTab(s, 0);
   }
   recent_tabs_builder.RegisterRecentTabs(&associator_);
 
   // Verify that data is populated correctly in RecentTabsSubMenuModel.
   // Expected menu:
   // - max sessions is 3, so only 3 most-recent sessions will show

[Alexei Svitkine (slow), 2013/03/19 17:09:33]

Expand this comment to explain why More devices... should be shown in this case.

[MAD, 2013/03/19 17:46:48]

Done.

   // 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 3>
   // 4           <separator>
   // 5           <section header for 2nd session>
   // 6           <the only tab of the only window of session 2>
   // 7           <separator>
   // 8           <section header for 3rd session>
   // 9           <the only tab of the only window of session 1>
+  // 10          <separator>
+  // 11          <More devices...>
 
   TestRecentTabsSubMenuModel model(NULL, browser(), &associator_, true);
   int num_items = model.GetItemCount();
-  EXPECT_EQ(10, num_items);
+  EXPECT_EQ(12, num_items);
 
   std::vector<string16> tab_titles =
       recent_tabs_builder.GetTabTitlesSortedByRecency();
   EXPECT_EQ(tab_titles[0], model.GetLabelAt(3));
   EXPECT_EQ(tab_titles[1], model.GetLabelAt(6));
   EXPECT_EQ(tab_titles[2], model.GetLabelAt(9));
 }
 
 TEST_F(RecentTabsSubMenuModelTest, MaxTabsPerSessionAndRecency) {
   // Create a session: 2 windows with 5 tabs each.
   RecentTabsBuilderTestHelper recent_tabs_builder;
   recent_tabs_builder.AddSession();
   for (int w = 0; w < 2; ++w) {
     recent_tabs_builder.AddWindow(0);
     for (int t = 0; t < 5; ++t)
       recent_tabs_builder.AddTab(0, w);
   }
   recent_tabs_builder.RegisterRecentTabs(&associator_);
 
   // Verify that data is populated correctly in RecentTabsSubMenuModel.
   // Expected menu:
   // - max tabs per session  is 4, so only 4 most-recent tabs will show,
   //   independent of which window they came from

[Alexei Svitkine (slow), 2013/03/19 17:09:33]

Expand this comment to explain why More devices... should be shown in this case.

[MAD, 2013/03/19 17:46:48]

Done.

   // Menu index  Menu items
   // --------------------------------------
   // 0           Reopen closed tab
   // 1           <separator>
   // 2           <section header for session>
   // 3-6         <4 most-recent tabs of session>
+  // 7           <separator>
+  // 8           <More devices...>
 
   TestRecentTabsSubMenuModel model(NULL, browser(), &associator_, true);
   int num_items = model.GetItemCount();
-  EXPECT_EQ(7, num_items);
+  EXPECT_EQ(9, 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 session with 1 window and 1 tab.
   RecentTabsBuilderTestHelper recent_tabs_builder;
@@ skipping 24 matching lines @@
   // 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));
 }