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

Side by Side Diff: chrome/browser/chromeos/login/session/user_session_manager.cc

Issue 2438063002: user_manager: Remove deps on session_manager (Closed)
Patch Set: Created 4 years, 2 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/chromeos/login/session/user_session_manager.h" 5 #include "chrome/browser/chromeos/login/session/user_session_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 1727 matching lines...) Expand 10 before | Expand all | Expand 10 after
1738 new QuickUnlockNotificationController(profile); 1738 new QuickUnlockNotificationController(profile);
1739 quick_unlock_notification_handler_.insert( 1739 quick_unlock_notification_handler_.insert(
1740 std::make_pair(profile, qu_feature_notification_controller)); 1740 std::make_pair(profile, qu_feature_notification_controller));
1741 } 1741 }
1742 1742
1743 // Mark login host for deletion after browser starts. This 1743 // Mark login host for deletion after browser starts. This
1744 // guarantees that the message loop will be referenced by the 1744 // guarantees that the message loop will be referenced by the
1745 // browser before it is dereferenced by the login host. 1745 // browser before it is dereferenced by the login host.
1746 if (login_host) 1746 if (login_host)
1747 login_host->Finalize(); 1747 login_host->Finalize();
1748 user_manager::UserManager::Get()->SessionStarted(); 1748 g_browser_process->platform_part()->SessionManager()->SessionStarted();
achuithb 2016/10/20 17:22:31 What's the difference between this and session_man
xiyuan 2016/10/20 17:26:50 They are the same. Line 1138 uses platform_part()-
achuithb 2016/10/20 17:39:44 Sure, if the current way is more consistent with t
1749 chromeos::BootTimesRecorder::Get()->LoginDone( 1749 chromeos::BootTimesRecorder::Get()->LoginDone(
1750 user_manager::UserManager::Get()->IsCurrentUserNew()); 1750 user_manager::UserManager::Get()->IsCurrentUserNew());
1751 1751
1752 // Check to see if this profile should show EndOfLife Notification and show 1752 // Check to see if this profile should show EndOfLife Notification and show
1753 // the message accordingly. 1753 // the message accordingly.
1754 if (ShouldShowEolNotification(profile)) 1754 if (ShouldShowEolNotification(profile))
1755 CheckEolStatus(profile); 1755 CheckEolStatus(profile);
1756 } 1756 }
1757 1757
1758 void UserSessionManager::RespectLocalePreferenceWrapper( 1758 void UserSessionManager::RespectLocalePreferenceWrapper(
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
1883 ->browser_policy_connector_chromeos() 1883 ->browser_policy_connector_chromeos()
1884 ->IsEnterpriseManaged()) { 1884 ->IsEnterpriseManaged()) {
1885 return false; 1885 return false;
1886 } 1886 }
1887 1887
1888 // Do not show end of life notification if this is a guest session 1888 // Do not show end of life notification if this is a guest session
1889 return !profile->IsGuestSession(); 1889 return !profile->IsGuestSession();
1890 } 1890 }
1891 1891
1892 } // namespace chromeos 1892 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698