Dynamic updating recent menu for tabs from other devices.

components/browser_sync/profile_sync_service_mock.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 "components/browser_sync/profile_sync_service_mock.h"
 
 #include <utility>
 
 namespace browser_sync {
 
 ProfileSyncServiceMock::ProfileSyncServiceMock(InitParams init_params)
     : ProfileSyncService(std::move(init_params)) {
   ON_CALL(*this, IsSyncRequested()).WillByDefault(testing::Return(true));
 }
 
 ProfileSyncServiceMock::ProfileSyncServiceMock(InitParams* init_params)
     : ProfileSyncServiceMock(std::move(*init_params)) {}
 
 ProfileSyncServiceMock::~ProfileSyncServiceMock() {}
 
+sync_sessions::OpenTabsUIDelegate*
+ProfileSyncServiceMock::GetOpenTabsUIDelegate() {
+  // If delegate is not mocked then we use usual one

[Peter Kasting, 2017/03/23 04:48:35]

Nit: Remove this comment, it adds nothing to the code.

+  sync_sessions::OpenTabsUIDelegate* tabs = GetOpenTabsUIDelegateMock();

[Peter Kasting, 2017/03/23 04:48:35]

Nit: Call this |mock| or |delegate| or |mock_delegate| rather than |tabs|.

+  if (!tabs)
+    return ProfileSyncService::GetOpenTabsUIDelegate();
+  return tabs;

[Peter Kasting, 2017/03/23 04:48:35]

Nit: Simpler:

  return tabs ? tabs : ProfileSyncService::GetOpenTabsUIDelegate();

+}
+
 }  // namespace browser_sync