Fixed crash on double profile delete operation.

chrome/browser/ui/webui/profile_helper.cc

Index: chrome/browser/ui/webui/profile_helper.cc
diff --git a/chrome/browser/ui/webui/profile_helper.cc b/chrome/browser/ui/webui/profile_helper.cc
index f350b962f1c9a8c56524d20225f3292814038455..37bce0fa2206d0872a973dad7130e7c1ecea1d03 100644
--- a/chrome/browser/ui/webui/profile_helper.cc
+++ b/chrome/browser/ui/webui/profile_helper.cc
@@ -26,16 +26,17 @@ void OpenNewWindowForProfile(Profile* profile, Profile::CreateStatus status) {
       chrome::startup::IS_FIRST_RUN, false);
 }
 
-void DeleteProfileAtPath(base::FilePath file_path, content::WebUI* web_ui) {
+void DeleteProfileAtPath(base::FilePath file_path,
+                         content::WebUI* web_ui,
+                         ProfileMetrics::ProfileDelete user) {
   DCHECK(web_ui);
 
   if (!profiles::IsMultipleProfilesEnabled())
     return;
-
-  ProfileMetrics::LogProfileDeleteUser(ProfileMetrics::DELETE_PROFILE_SETTINGS);
-
-  g_browser_process->profile_manager()->ScheduleProfileForDeletion(
-      file_path, base::Bind(&OpenNewWindowForProfile));
+  if (g_browser_process->profile_manager()->MaybeScheduleProfileForDeletion(

[achuithb, 2016/05/12 22:33:59]

This seems a bit awkward. Can you pass user into MaybeScheduleProfileForDeletion
and call LogProfileDeleteUser in that function instead?

+          file_path, base::Bind(&OpenNewWindowForProfile))) {
+    ProfileMetrics::LogProfileDeleteUser(user);
+  }
 }
 
 }  // namespace webui