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

Unified Diff: chrome/browser/ui/ash/chrome_new_window_delegate.cc

Issue 1350653004: [sessions] Properly namespace recently-componentized TabRestore code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix Mac Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/sessions/tab_restore_service_factory.cc ('k') | chrome/browser/ui/browser.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/ash/chrome_new_window_delegate.cc
diff --git a/chrome/browser/ui/ash/chrome_new_window_delegate.cc b/chrome/browser/ui/ash/chrome_new_window_delegate.cc
index 1c2eb93fcffd520657d337ad3ba94552fd948d03..34c5a9cfc9c310a291e28274f3b1c29800a561e6 100644
--- a/chrome/browser/ui/ash/chrome_new_window_delegate.cc
+++ b/chrome/browser/ui/ash/chrome_new_window_delegate.cc
@@ -19,7 +19,8 @@
namespace {
void RestoreTabUsingProfile(Profile* profile) {
- TabRestoreService* service = TabRestoreServiceFactory::GetForProfile(profile);
+ sessions::TabRestoreService* service =
+ TabRestoreServiceFactory::GetForProfile(profile);
service->RestoreMostRecentEntry(NULL, chrome::HOST_DESKTOP_TYPE_ASH);
}
@@ -38,29 +39,31 @@ ChromeNewWindowDelegate::~ChromeNewWindowDelegate() {}
// this waits for it. Once the TabRestoreService finishes loading the tab is
// restored.
class ChromeNewWindowDelegate::TabRestoreHelper
- : public TabRestoreServiceObserver {
+ : public sessions::TabRestoreServiceObserver {
public:
TabRestoreHelper(ChromeNewWindowDelegate* delegate,
Profile* profile,
- TabRestoreService* service)
- : delegate_(delegate),
- profile_(profile),
- tab_restore_service_(service) {
+ sessions::TabRestoreService* service)
+ : delegate_(delegate), profile_(profile), tab_restore_service_(service) {
tab_restore_service_->AddObserver(this);
}
~TabRestoreHelper() override { tab_restore_service_->RemoveObserver(this); }
- TabRestoreService* tab_restore_service() { return tab_restore_service_; }
+ sessions::TabRestoreService* tab_restore_service() {
+ return tab_restore_service_;
+ }
- void TabRestoreServiceChanged(TabRestoreService* service) override {}
+ void TabRestoreServiceChanged(sessions::TabRestoreService* service) override {
+ }
- void TabRestoreServiceDestroyed(TabRestoreService* service) override {
+ void TabRestoreServiceDestroyed(
+ sessions::TabRestoreService* service) override {
// This destroys us.
delegate_->tab_restore_helper_.reset();
}
- void TabRestoreServiceLoaded(TabRestoreService* service) override {
+ void TabRestoreServiceLoaded(sessions::TabRestoreService* service) override {
RestoreTabUsingProfile(profile_);
// This destroys us.
delegate_->tab_restore_helper_.reset();
@@ -69,7 +72,7 @@ class ChromeNewWindowDelegate::TabRestoreHelper
private:
ChromeNewWindowDelegate* delegate_;
Profile* profile_;
- TabRestoreService* tab_restore_service_;
+ sessions::TabRestoreService* tab_restore_service_;
DISALLOW_COPY_AND_ASSIGN(TabRestoreHelper);
};
@@ -111,7 +114,7 @@ void ChromeNewWindowDelegate::RestoreTab() {
profile = ProfileManager::GetActiveUserProfile();
if (profile->IsOffTheRecord())
return;
- TabRestoreService* service =
+ sessions::TabRestoreService* service =
TabRestoreServiceFactory::GetForProfile(profile);
if (!service)
return;
« no previous file with comments | « chrome/browser/sessions/tab_restore_service_factory.cc ('k') | chrome/browser/ui/browser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698