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

Side by Side Diff: chrome/browser/ui/startup/startup_tab_provider.h

Issue 2623253002: Adding check to ensure onboarding content is not shown to supervised users. (Closed)
Patch Set: Fixing Win double define Created 3 years, 11 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/ui/startup/startup_tab_provider.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 #ifndef CHROME_BROWSER_UI_STARTUP_STARTUP_TAB_PROVIDER_H_ 5 #ifndef CHROME_BROWSER_UI_STARTUP_STARTUP_TAB_PROVIDER_H_
6 #define CHROME_BROWSER_UI_STARTUP_STARTUP_TAB_PROVIDER_H_ 6 #define CHROME_BROWSER_UI_STARTUP_STARTUP_TAB_PROVIDER_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/gtest_prod_util.h" 10 #include "base/gtest_prod_util.h"
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 56
57 // The static Check*TabPolicy methods below enforce the policies relevant to 57 // The static Check*TabPolicy methods below enforce the policies relevant to
58 // the respective Get*Tabs methods, but do not gather or interact with any 58 // the respective Get*Tabs methods, but do not gather or interact with any
59 // system state relating to making those policy decisions. 59 // system state relating to making those policy decisions.
60 60
61 // Determines which tabs should be shown according to onboarding/first 61 // Determines which tabs should be shown according to onboarding/first
62 // run policy. 62 // run policy.
63 static StartupTabs CheckStandardOnboardingTabPolicy( 63 static StartupTabs CheckStandardOnboardingTabPolicy(
64 bool is_first_run, 64 bool is_first_run,
65 bool has_seen_welcome_page, 65 bool has_seen_welcome_page,
66 bool is_signed_in); 66 bool is_signed_in,
67 bool is_supervised_user);
Peter Kasting 2017/01/12 18:28:19 Random muse: It seems like there are now a number
67 68
68 #if defined(OS_WIN) 69 #if defined(OS_WIN)
69 // Determines which tabs should be shown according to onboarding/first run 70 // Determines which tabs should be shown according to onboarding/first run
70 // policy, including promo content specific to Windows 10. 71 // policy, including promo content specific to Windows 10.
71 static StartupTabs CheckWin10OnboardingTabPolicy( 72 static StartupTabs CheckWin10OnboardingTabPolicy(
72 bool is_first_run, 73 bool is_first_run,
73 bool has_seen_welcome_page, 74 bool has_seen_welcome_page,
74 bool has_seen_win10_promo, 75 bool has_seen_win10_promo,
75 bool is_signed_in, 76 bool is_signed_in,
76 bool set_default_browser_allowed, 77 bool set_default_browser_allowed,
77 bool is_default_browser); 78 bool is_default_browser,
79 bool is_supervised_user);
78 #endif 80 #endif
79 81
80 // Processes first run URLs specified in Master Preferences file, replacing 82 // Processes first run URLs specified in Master Preferences file, replacing
81 // any "magic word" URL hosts with appropriate URLs. 83 // any "magic word" URL hosts with appropriate URLs.
82 static StartupTabs CheckMasterPrefsTabPolicy( 84 static StartupTabs CheckMasterPrefsTabPolicy(
83 bool is_first_run, 85 bool is_first_run,
84 const std::vector<GURL>& first_run_tabs); 86 const std::vector<GURL>& first_run_tabs);
85 87
86 // Determines which tabs should be shown as a result of the presence/absence 88 // Determines which tabs should be shown as a result of the presence/absence
87 // of a Reset Trigger on this profile. 89 // of a Reset Trigger on this profile.
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 StartupTabs GetPreferencesTabs(const base::CommandLine& command_line, 133 StartupTabs GetPreferencesTabs(const base::CommandLine& command_line,
132 Profile* profile) const override; 134 Profile* profile) const override;
133 StartupTabs GetNewTabPageTabs(const base::CommandLine& command_line, 135 StartupTabs GetNewTabPageTabs(const base::CommandLine& command_line,
134 Profile* profile) const override; 136 Profile* profile) const override;
135 137
136 private: 138 private:
137 DISALLOW_COPY_AND_ASSIGN(StartupTabProviderImpl); 139 DISALLOW_COPY_AND_ASSIGN(StartupTabProviderImpl);
138 }; 140 };
139 141
140 #endif // CHROME_BROWSER_UI_STARTUP_STARTUP_TAB_PROVIDER_H_ 142 #endif // CHROME_BROWSER_UI_STARTUP_STARTUP_TAB_PROVIDER_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/ui/startup/startup_tab_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698