Now command line flag does not beat Policy

chrome/browser/chrome_browser_main.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chrome_browser_main.h"
 
 #include <set>
 #include <string>
 #include <vector>
 
@@ skipping 1125 matching lines @@
       parsed_command_line().HasSwitch(switches::kShowIcons)) {
     return ChromeBrowserMainPartsWin::HandleIconsCommands(
         parsed_command_line_);
   }
 
   ui::SelectFileDialog::SetFactory(new ChromeSelectFileDialogFactory(
       BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO)));
 #endif
 
   if (parsed_command_line().HasSwitch(switches::kMakeDefaultBrowser)) {
-    return ShellIntegration::SetAsDefaultBrowser() ?
+    bool is_managed = g_browser_process->local_state()->IsManagedPreference(
+      prefs::kDefaultBrowserSettingEnabled);
+    if (!is_managed ||
+      g_browser_process->local_state()->GetBoolean(
+      prefs::kDefaultBrowserSettingEnabled)) {
+      return ShellIntegration::SetAsDefaultBrowser() ?
         static_cast<int>(content::RESULT_CODE_NORMAL_EXIT) :
         static_cast<int>(chrome::RESULT_CODE_SHELL_INTEGRATION_FAILED);
+    }
+    return static_cast<int>(chrome::RESULT_CODE_SHELL_INTEGRATION_FAILED);

[pastarmovj, 2015/03/18 16:02:17]

Any chance you can add a new error result code here to distinguish between
failure reasons. Especially if this error is then propagated to the user in one
way or another even if only through logging.

   }
 
 #if defined(USE_AURA)
   // Make sure aura::Env has been initialized.
   CHECK(aura::Env::GetInstance());
 #endif
 
   // Android doesn't support extensions and doesn't implement ProcessSingleton.
 #if !defined(OS_ANDROID)
   // If the command line specifies --pack-extension, attempt the pack extension
@@ skipping 549 matching lines @@
   chromeos::CrosSettings::Shutdown();
 #endif
 #endif
 }
 
 // Public members:
 
 void ChromeBrowserMainParts::AddParts(ChromeBrowserMainExtraParts* parts) {
   chrome_extra_parts_.push_back(parts);
 }