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

Side by Side Diff: chrome/browser/chromeos/arc/arc_session_manager_unittest.cc

Issue 2771803003: Don't start FRE from the login screen (Closed)
Patch Set: Improve comments Created 3 years, 8 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 <memory> 5 #include <memory>
6 #include <string> 6 #include <string>
7 #include <tuple> 7 #include <tuple>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 chromeos::LoginDisplay::Delegate* delegate) override { 82 chromeos::LoginDisplay::Delegate* delegate) override {
83 return nullptr; 83 return nullptr;
84 } 84 }
85 gfx::NativeWindow GetNativeWindow() const override { return nullptr; } 85 gfx::NativeWindow GetNativeWindow() const override { return nullptr; }
86 chromeos::OobeUI* GetOobeUI() const override { return nullptr; } 86 chromeos::OobeUI* GetOobeUI() const override { return nullptr; }
87 chromeos::WebUILoginView* GetWebUILoginView() const override { 87 chromeos::WebUILoginView* GetWebUILoginView() const override {
88 return nullptr; 88 return nullptr;
89 } 89 }
90 void BeforeSessionStart() override {} 90 void BeforeSessionStart() override {}
91 void Finalize() override {} 91 void Finalize() override {}
92 void OnCompleteLogin() override {}
93 void OpenProxySettings() override {} 92 void OpenProxySettings() override {}
94 void SetStatusAreaVisible(bool visible) override {} 93 void SetStatusAreaVisible(bool visible) override {}
95 chromeos::AutoEnrollmentController* GetAutoEnrollmentController() override {
96 return nullptr;
97 }
98 void StartWizard(chromeos::OobeScreen first_screen) override {} 94 void StartWizard(chromeos::OobeScreen first_screen) override {}
99 chromeos::WizardController* GetWizardController() override { return nullptr; } 95 chromeos::WizardController* GetWizardController() override { return nullptr; }
100 chromeos::AppLaunchController* GetAppLaunchController() override { 96 chromeos::AppLaunchController* GetAppLaunchController() override {
101 return nullptr; 97 return nullptr;
102 } 98 }
103 void StartUserAdding(const base::Closure& completion_callback) override {} 99 void StartUserAdding(const base::Closure& completion_callback) override {}
104 void CancelUserAdding() override {} 100 void CancelUserAdding() override {}
105 void StartSignInScreen(const chromeos::LoginScreenContext& context) override { 101 void StartSignInScreen(const chromeos::LoginScreenContext& context) override {
106 } 102 }
107 void OnPreferencesChanged() override {} 103 void OnPreferencesChanged() override {}
(...skipping 702 matching lines...) Expand 10 before | Expand all | Expand 10 after
810 // For managed case we handle closing outside of 806 // For managed case we handle closing outside of
811 // ArcPlayStoreEnabledPreferenceHandler. So it session turns to STOPPED. 807 // ArcPlayStoreEnabledPreferenceHandler. So it session turns to STOPPED.
812 EXPECT_EQ(ArcSessionManager::State::STOPPED, 808 EXPECT_EQ(ArcSessionManager::State::STOPPED,
813 arc_session_manager()->state()); 809 arc_session_manager()->state());
814 // Managed user's preference should not be overwritten. 810 // Managed user's preference should not be overwritten.
815 EXPECT_TRUE(IsArcPlayStoreEnabledForProfile(profile())); 811 EXPECT_TRUE(IsArcPlayStoreEnabledForProfile(profile()));
816 } 812 }
817 } 813 }
818 814
819 } // namespace arc 815 } // namespace arc
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/chromeos/login/enrollment/auto_enrollment_check_screen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698