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

Unified Diff: chrome/browser/ui/webui/options/browser_options_handler.cc

Issue 1657933003: Fixes the interactive default browser UX for policy setting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: grt comments Created 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/options/browser_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/browser_options_handler.cc b/chrome/browser/ui/webui/options/browser_options_handler.cc
index 8337d62518f9d9915ae8d4d2ddd757383dd9223b..7462092287b638fe83bdf248eff0b9a48af247f6 100644
--- a/chrome/browser/ui/webui/options/browser_options_handler.cc
+++ b/chrome/browser/ui/webui/options/browser_options_handler.cc
@@ -184,7 +184,8 @@ BrowserOptionsHandler::BrowserOptionsHandler()
#endif // defined(OS_CHROMEOS)
signin_observer_(this),
weak_ptr_factory_(this) {
- default_browser_worker_ = new ShellIntegration::DefaultBrowserWorker(this);
+ default_browser_worker_ =
+ new ShellIntegration::DefaultBrowserWorker(this, false);
#if defined(ENABLE_SERVICE_DISCOVERY)
cloud_print_mdns_ui_enabled_ = true;
@@ -1134,10 +1135,6 @@ void BrowserOptionsHandler::SetDefaultWebClientUIState(
SetDefaultBrowserUIString(status_string_id);
}
-bool BrowserOptionsHandler::IsInteractiveSetDefaultPermitted() {
- return true; // This is UI so we can allow it.
-}
-
void BrowserOptionsHandler::SetDefaultBrowserUIString(int status_string_id) {
base::StringValue status_string(
l10n_util::GetStringFUTF16(status_string_id,

Powered by Google App Engine
This is Rietveld 408576698