Make the histograms for setting the default browser consistent

chrome/browser/ui/browser_command_controller.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/ui/browser_command_controller.h"
 
+#include <string>
+
 #include "base/command_line.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/defaults.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_util.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/prefs/incognito_mode_prefs.h"
@@ skipping 141 matching lines @@
     }
     delete this;
   }
 
   void OnSetAsDefaultConcluded(bool success) override {
     if (!success) {
       delete this;
       return;
     }
     first_check_ = false;
-    default_browser_worker_->StartCheckIsDefault();

[grt (UTC plus 2), 2015/11/03 21:12:22]

huh. was this never needed?

[Patrick Monette, 2015/11/10 19:22:47]

StartCheckIsDefault() is already called in OnSetAsDefaultAttemptComplete. So I
guess it was being called twice everytime here.

   }
 
   bool IsInteractiveSetDefaultPermitted() override {
     return true;
   }
 
   scoped_refptr<ShellIntegration::DefaultBrowserWorker> default_browser_worker_;
   bool first_check_;
 
   DISALLOW_COPY_AND_ASSIGN(SwitchToMetroUIHandler);
@@ skipping 1194 matching lines @@
 
 BrowserWindow* BrowserCommandController::window() {
   return browser_->window();
 }
 
 Profile* BrowserCommandController::profile() {
   return browser_->profile();
 }
 
 }  // namespace chrome