Fixes the interactive default browser UX for policy setting

chrome/browser/custom_handlers/protocol_handler_registry.cc

Index: chrome/browser/custom_handlers/protocol_handler_registry.cc
diff --git a/chrome/browser/custom_handlers/protocol_handler_registry.cc b/chrome/browser/custom_handlers/protocol_handler_registry.cc
index 1bce59dda7c2c5e8df28385a0fefaae1238cf010..fbd7de7b87f84e9449ead5e3a2ac0248f827be9c 100644
--- a/chrome/browser/custom_handlers/protocol_handler_registry.cc
+++ b/chrome/browser/custom_handlers/protocol_handler_registry.cc
@@ -261,20 +261,11 @@ void ProtocolHandlerRegistry::DefaultClientObserver::SetDefaultWebClientUIState(
   }
 }
 
-bool ProtocolHandlerRegistry::DefaultClientObserver::
-    IsInteractiveSetDefaultPermitted() {
-  return true;
-}
-
 void ProtocolHandlerRegistry::DefaultClientObserver::SetWorker(
     ShellIntegration::DefaultProtocolClientWorker* worker) {
   worker_ = worker;
 }
 
-bool ProtocolHandlerRegistry::DefaultClientObserver::IsOwnedByWorker() {
-  return true;
-}
-
 // Delegate --------------------------------------------------------------------
 
 ProtocolHandlerRegistry::Delegate::~Delegate() {}
@@ -303,7 +294,8 @@ ShellIntegration::DefaultProtocolClientWorker*
 ProtocolHandlerRegistry::Delegate::CreateShellWorker(
     ShellIntegration::DefaultWebClientObserver* observer,
     const std::string& protocol) {
-  return new ShellIntegration::DefaultProtocolClientWorker(observer, protocol);
+  return new ShellIntegration::DefaultProtocolClientWorker(observer, protocol,
+                                                           true);

[Nico, 2016/02/09 03:01:31]

I'd discourage boolean parameters as they make it impossible to know what "true"
means here. Consider making this an enum instead, then this line is much less
mysterious. (or do `bool named_thing = true;` then pass named_thing, or do
`/*named_thing=*/true`)

[Patrick Monette, 2016/02/09 17:15:05]

Done.

 }
 
 ProtocolHandlerRegistry::DefaultClientObserver*