Dynamic updating recent menu for tabs from other devices.

chrome/browser/ui/cocoa/app_menu/app_menu_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 "chrome/browser/ui/cocoa/app_menu/app_menu_controller.h"
 #include "base/command_line.h"
 #include "base/mac/scoped_nsobject.h"
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/run_loop.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/sync/profile_sync_service_factory.h"
+#include "chrome/browser/sync/profile_sync_test_util.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_list_observer.h"
-#import "chrome/browser/ui/cocoa/app_menu/app_menu_controller.h"
 #include "chrome/browser/ui/cocoa/test/cocoa_profile_test.h"
 #include "chrome/browser/ui/cocoa/test/run_loop_testing.h"
 #import "chrome/browser/ui/cocoa/toolbar/toolbar_controller.h"
 #import "chrome/browser/ui/cocoa/view_resizer_pong.h"
 #include "chrome/browser/ui/sync/browser_synced_window_delegates_getter.h"
 #include "chrome/browser/ui/toolbar/app_menu_model.h"
 #include "chrome/browser/ui/toolbar/recent_tabs_builder_test_helper.h"
 #include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/grit/generated_resources.h"
 #include "chrome/grit/theme_resources.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/browser_sync/profile_sync_service.h"
+#include "components/browser_sync/profile_sync_service_mock.h"
 #include "components/sync/base/sync_prefs.h"
 #include "components/sync/device_info/local_device_info_provider_mock.h"
 #include "components/sync/driver/sync_client.h"
 #include "components/sync/model/fake_sync_change_processor.h"
 #include "components/sync/model/sync_error_factory_mock.h"
 #include "components/sync_sessions/fake_sync_sessions_client.h"
 #include "components/sync_sessions/sessions_sync_manager.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/gtest_mac.h"
 #include "testing/platform_test.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 
+using testing::_;
+using testing::Invoke;
+using testing::Return;
+
 namespace {
 
 class MockAppMenuModel : public AppMenuModel {
  public:
   MockAppMenuModel() : AppMenuModel() {}
   ~MockAppMenuModel() {}
   MOCK_METHOD2(ExecuteCommand, void(int command_id, int event_flags));
 };
 
 class DummyRouter : public sync_sessions::LocalSessionEventRouter {
  public:
   ~DummyRouter() override {}
   void StartRoutingTo(
       sync_sessions::LocalSessionEventHandler* handler) override {}
   void Stop() override {}
 };
 
 class AppMenuControllerTest : public CocoaProfileTest {
  public:
-  AppMenuControllerTest()
-      : local_device_(new syncer::LocalDeviceInfoProviderMock(
-            "AppMenuControllerTest",
-            "Test Machine",
-            "Chromium 10k",
-            "Chrome 10k",
-            sync_pb::SyncEnums_DeviceType_TYPE_LINUX,
-            "device_id")) {}
+  AppMenuControllerTest() {
+    TestingProfile::TestingFactories factories;
+    factories.push_back(std::make_pair(ProfileSyncServiceFactory::GetInstance(),
+                                       BuildMockProfileSyncService));
+    AddTestingFactories(factories);
+  }
 
   void SetUp() override {
     CocoaProfileTest::SetUp();
     ASSERT_TRUE(browser());
 
+    local_device_ = base::MakeUnique<syncer::LocalDeviceInfoProviderMock>(
+        "AppMenuControllerTest", "Test Machine", "Chromium 10k", "Chrome 10k",
+        sync_pb::SyncEnums_DeviceType_TYPE_LINUX, "device_id");
+
     controller_.reset([[AppMenuController alloc] initWithBrowser:browser()]);
-    fake_model_.reset(new MockAppMenuModel);
 
-    sync_prefs_.reset(new syncer::SyncPrefs(profile()->GetPrefs()));
-    dummy_router_.reset(new DummyRouter());
-    manager_.reset(new sync_sessions::SessionsSyncManager(
+    fake_model_ = base::MakeUnique<MockAppMenuModel>();
+
+    sync_prefs_ = base::MakeUnique<syncer::SyncPrefs>(profile()->GetPrefs());
+
+    mock_sync_service_ = static_cast<browser_sync::ProfileSyncServiceMock*>(
+        ProfileSyncServiceFactory::GetInstance()->GetForProfile(profile()));
+
+    manager_ = base::MakeUnique<sync_sessions::SessionsSyncManager>(
         ProfileSyncServiceFactory::GetForProfile(profile())
             ->GetSyncClient()
             ->GetSyncSessionsClient(),
-        sync_prefs_.get(), local_device_.get(), dummy_router_.get(),
-        base::Closure(), base::Closure()));
+        sync_prefs_.get(), local_device_.get(), &dummy_router_, base::Closure(),
+        base::Closure());
+
     manager_->MergeDataAndStartSyncing(
         syncer::SESSIONS, syncer::SyncDataList(),
         std::unique_ptr<syncer::SyncChangeProcessor>(
             new syncer::FakeSyncChangeProcessor),
         std::unique_ptr<syncer::SyncErrorFactory>(
             new syncer::SyncErrorFactoryMock));
   }
 
   void RegisterRecentTabs(RecentTabsBuilderTestHelper* helper) {
     helper->ExportToSessionsSyncManager(manager_.get());
   }
 
-  sync_sessions::OpenTabsUIDelegate* GetOpenTabsDelegate() {
-    return manager_.get();
-  }
-
   void TearDown() override {
     fake_model_.reset();
     controller_.reset();
     manager_.reset();
+    sync_prefs_.reset();
+    local_device_.reset();
     CocoaProfileTest::TearDown();
   }
 
+  void EnableSync() {
+    EXPECT_CALL(*mock_sync_service_, IsSyncActive())
+        .WillRepeatedly(Return(true));
+    EXPECT_CALL(*mock_sync_service_,
+                IsDataTypeControllerRunning(syncer::SESSIONS))
+        .WillRepeatedly(Return(true));
+    EXPECT_CALL(*mock_sync_service_,
+                IsDataTypeControllerRunning(syncer::PROXY_TABS))
+        .WillRepeatedly(Return(true));
+    EXPECT_CALL(*mock_sync_service_, GetOpenTabsUIDelegateMock())
+        .WillRepeatedly(Return(manager_.get()));
+  }
+
   AppMenuController* controller() {
     return controller_.get();
   }
 
   base::scoped_nsobject<AppMenuController> controller_;
 
   std::unique_ptr<MockAppMenuModel> fake_model_;
 
  private:
+  std::unique_ptr<syncer::LocalDeviceInfoProviderMock> local_device_;
+  DummyRouter dummy_router_;
   std::unique_ptr<syncer::SyncPrefs> sync_prefs_;
-  std::unique_ptr<DummyRouter> dummy_router_;
+  browser_sync::ProfileSyncServiceMock* mock_sync_service_ = nullptr;
   std::unique_ptr<sync_sessions::SessionsSyncManager> manager_;
-  std::unique_ptr<syncer::LocalDeviceInfoProviderMock> local_device_;
 };
 
 TEST_F(AppMenuControllerTest, Initialized) {
   EXPECT_TRUE([controller() menu]);
   EXPECT_GE([[controller() menu] numberOfItems], 5);
 }
 
 TEST_F(AppMenuControllerTest, DispatchSimple) {
   base::scoped_nsobject<NSButton> button([[NSButton alloc] init]);
   [button setTag:IDC_ZOOM_PLUS];
 
   // Set fake model to test dispatching.
   EXPECT_CALL(*fake_model_, ExecuteCommand(IDC_ZOOM_PLUS, 0));
   [controller() setModel:fake_model_.get()];
 
   [controller() dispatchAppMenuCommand:button.get()];
   chrome::testing::NSRunLoopRunAllPending();
 }
 
 TEST_F(AppMenuControllerTest, RecentTabsFavIcon) {
+  EnableSync();
+
   RecentTabsBuilderTestHelper recent_tabs_builder;
   recent_tabs_builder.AddSession();
   recent_tabs_builder.AddWindow(0);
   recent_tabs_builder.AddTab(0, 0);
   RegisterRecentTabs(&recent_tabs_builder);
 
-  RecentTabsSubMenuModel recent_tabs_sub_menu_model(
-      NULL, browser(), GetOpenTabsDelegate());
+  RecentTabsSubMenuModel recent_tabs_sub_menu_model(nullptr, browser());
   fake_model_->AddSubMenuWithStringId(
       IDC_RECENT_TABS_MENU, IDS_RECENT_TABS_MENU,
       &recent_tabs_sub_menu_model);
 
   [controller() setModel:fake_model_.get()];
   NSMenu* menu = [controller() menu];
   [controller() updateRecentTabsSubmenu];
 
   NSString* title = l10n_util::GetNSStringWithFixup(IDS_RECENT_TABS_MENU);
   NSMenu* recent_tabs_menu = [[menu itemWithTitle:title] submenu];
   EXPECT_TRUE(recent_tabs_menu);
   EXPECT_EQ(6, [recent_tabs_menu numberOfItems]);
 
   // Send a icon changed event and verify that the icon is updated.
   gfx::Image icon(ResourceBundle::GetSharedInstance().GetNativeImageNamed(
       IDR_BOOKMARKS_FAVICON));
   recent_tabs_sub_menu_model.SetIcon(3, icon);
   EXPECT_NSNE(icon.ToNSImage(), [[recent_tabs_menu itemAtIndex:3] image]);
   recent_tabs_sub_menu_model.GetMenuModelDelegate()->OnIconChanged(3);
   EXPECT_TRUE([[recent_tabs_menu itemAtIndex:3] image]);
   EXPECT_NSEQ(icon.ToNSImage(), [[recent_tabs_menu itemAtIndex:3] image]);
 
   controller_.reset();
   fake_model_.reset();
 }
 
 TEST_F(AppMenuControllerTest, RecentTabsElideTitle) {
+  EnableSync();
+
   // Add 1 session with 1 window and 2 tabs.
   RecentTabsBuilderTestHelper recent_tabs_builder;
   recent_tabs_builder.AddSession();
   recent_tabs_builder.AddWindow(0);
   base::string16 tab1_short_title = base::ASCIIToUTF16("Short");
   recent_tabs_builder.AddTabWithInfo(0, 0, base::Time::Now(), tab1_short_title);
   base::string16 tab2_long_title = base::ASCIIToUTF16(
       "Very very very very very very very very very very very very long");
   recent_tabs_builder.AddTabWithInfo(0, 0,
       base::Time::Now() - base::TimeDelta::FromMinutes(10), tab2_long_title);
   RegisterRecentTabs(&recent_tabs_builder);
 
-  RecentTabsSubMenuModel recent_tabs_sub_menu_model(
-      NULL, browser(), GetOpenTabsDelegate());
+  RecentTabsSubMenuModel recent_tabs_sub_menu_model(nullptr, browser());
   fake_model_->AddSubMenuWithStringId(
       IDC_RECENT_TABS_MENU, IDS_RECENT_TABS_MENU,
       &recent_tabs_sub_menu_model);
 
   [controller() setModel:fake_model_.get()];
   NSMenu* menu = [controller() menu];
   [controller() updateRecentTabsSubmenu];
 
   NSString* title = l10n_util::GetNSStringWithFixup(IDS_RECENT_TABS_MENU);
   NSMenu* recent_tabs_menu = [[menu itemWithTitle:title] submenu];
@@ skipping 61 matching lines @@
   BrowserRemovedObserver observer;
   // This is normally called by ToolbarController, but since |controller_| is
   // not owned by one, call it here.
   [controller_ browserWillBeDestroyed];
   CloseBrowserWindow();
   observer.WaitUntilBrowserRemoved();
   // |controller_| is released in TearDown().
 }
 
 }  // namespace