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

Unified Diff: chrome/browser/ui/startup/startup_browser_creator_impl.cc

Issue 2578843002: Implement Win 10 onboarding content logic (Closed)
Patch Set: Created 4 years 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
Index: chrome/browser/ui/startup/startup_browser_creator_impl.cc
diff --git a/chrome/browser/ui/startup/startup_browser_creator_impl.cc b/chrome/browser/ui/startup/startup_browser_creator_impl.cc
index b088b9649494180310c986925cd95a41dc6064cb..12315fa95ec2151b7f9e6f83390b9c36d35f3602 100644
--- a/chrome/browser/ui/startup/startup_browser_creator_impl.cc
+++ b/chrome/browser/ui/startup/startup_browser_creator_impl.cc
@@ -279,18 +279,6 @@ void AppendTabs(const StartupTabs& from, StartupTabs* to) {
to->insert(to->end(), from.begin(), from.end());
}
-// Determines whether the Consolidated startup flow should be used, based on
-// OS, OS version, and the kUseConsolidatedStartupFlow Feature.
-bool UseConsolidatedFlow() {
-#if defined(OS_WIN)
- // TODO(tmartino): Enable for Windows 10+ once relevant Win 10-specific logic
- // is added to StartupTabProvider.
- if (base::win::GetVersion() >= base::win::VERSION_WIN10)
- return false;
-#endif // defined(OS_WIN)
- return base::FeatureList::IsEnabled(features::kUseConsolidatedStartupFlow);
-}
-
} // namespace
namespace internals {
@@ -377,7 +365,7 @@ bool StartupBrowserCreatorImpl::Launch(Profile* profile,
RecordLaunchModeHistogram(urls_to_open.empty() ?
LM_TO_BE_DECIDED : LM_WITH_URLS);
- if (UseConsolidatedFlow())
+ if (base::FeatureList::IsEnabled(features::kUseConsolidatedStartupFlow))
ProcessLaunchUrlsUsingConsolidatedFlow(process_startup, urls_to_open);
else
ProcessLaunchURLs(process_startup, urls_to_open);

Powered by Google App Engine
This is Rietveld 408576698