Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(439)

Side by Side Diff: chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc

Issue 1155443009: [Sync] Rename SyncActive to IsSyncActive. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More non-android cases. Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h" 5 #include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 #include "base/prefs/scoped_user_pref_update.h" 9 #include "base/prefs/scoped_user_pref_update.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 657 matching lines...) Expand 10 before | Expand all | Expand 10 after
668 // Remove all local window items. 668 // Remove all local window items.
669 local_window_items_.clear(); 669 local_window_items_.clear();
670 } 670 }
671 671
672 browser_sync::OpenTabsUIDelegate* 672 browser_sync::OpenTabsUIDelegate*
673 RecentTabsSubMenuModel::GetOpenTabsUIDelegate() { 673 RecentTabsSubMenuModel::GetOpenTabsUIDelegate() {
674 if (!open_tabs_delegate_) { 674 if (!open_tabs_delegate_) {
675 ProfileSyncService* service = ProfileSyncServiceFactory::GetInstance()-> 675 ProfileSyncService* service = ProfileSyncServiceFactory::GetInstance()->
676 GetForProfile(browser_->profile()); 676 GetForProfile(browser_->profile());
677 // Only return the delegate if it exists and it is done syncing sessions. 677 // Only return the delegate if it exists and it is done syncing sessions.
678 if (service && service->SyncActive()) 678 if (service && service->IsSyncActive())
679 open_tabs_delegate_ = service->GetOpenTabsUIDelegate(); 679 open_tabs_delegate_ = service->GetOpenTabsUIDelegate();
680 } 680 }
681 return open_tabs_delegate_; 681 return open_tabs_delegate_;
682 } 682 }
683 683
684 void RecentTabsSubMenuModel::TabRestoreServiceChanged( 684 void RecentTabsSubMenuModel::TabRestoreServiceChanged(
685 TabRestoreService* service) { 685 TabRestoreService* service) {
686 ClearLocalEntries(); 686 ClearLocalEntries();
687 687
688 BuildLocalEntries(); 688 BuildLocalEntries();
689 689
690 ui::MenuModelDelegate* menu_model_delegate = GetMenuModelDelegate(); 690 ui::MenuModelDelegate* menu_model_delegate = GetMenuModelDelegate();
691 if (menu_model_delegate) 691 if (menu_model_delegate)
692 menu_model_delegate->OnMenuStructureChanged(); 692 menu_model_delegate->OnMenuStructureChanged();
693 } 693 }
694 694
695 void RecentTabsSubMenuModel::TabRestoreServiceDestroyed( 695 void RecentTabsSubMenuModel::TabRestoreServiceDestroyed(
696 TabRestoreService* service) { 696 TabRestoreService* service) {
697 TabRestoreServiceChanged(service); 697 TabRestoreServiceChanged(service);
698 } 698 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/sync/one_click_signin_sync_observer_unittest.cc ('k') | chrome/browser/ui/webui/foreign_session_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698