Fixed crash on double profile delete operation.

chrome/browser/browsing_data/browsing_data_remover.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/browsing_data/browsing_data_remover.h"
 
 #include <map>
 #include <set>
 #include <string>
 
@@ skipping 336 matching lines @@
 }
 
 void BrowsingDataRemover::SetRemoving(bool is_removing) {
   DCHECK_NE(is_removing_, is_removing);
   is_removing_ = is_removing;
 }
 
 void BrowsingDataRemover::Remove(const TimeRange& time_range,
                                  int remove_mask,
                                  int origin_type_mask) {
+  if (is_removing_)

[Bernhard Bauer, 2016/05/05 10:06:47]

I'm not sure if silently returning is the right thing to do here. We should
disable UI that allows clearing browsing data while that is in progress, and
then DCHECK that we are not trying to clear data twice (which already happens in
SetRemoving()).

The same probably goes for profile deletion --
ProfileManager::ScheduleProfileForDeletion() and/or
ScheduleQueueProfileDirectoryForDeletion should DCHECK that the profile isn't
already scheduled for deletion, and deleting a profile should disable the UI
while that is in progress.

+    return;
   // Any instance of BrowsingDataFilterBuilder that |IsEmptyBlacklist()|
   // is OK to pass here.
   RegistrableDomainFilterBuilder builder(
       RegistrableDomainFilterBuilder::BLACKLIST);
   DCHECK(builder.IsEmptyBlacklist());
   RemoveImpl(time_range, remove_mask, builder, origin_type_mask);
 }
 
 void BrowsingDataRemover::RemoveWithFilter(
     const TimeRange& time_range,
@@ skipping 949 matching lines @@
   waiting_for_clear_domain_reliability_monitor_ = false;
   NotifyIfDone();
 }
 
 // static
 BrowsingDataRemover::CallbackSubscription
     BrowsingDataRemover::RegisterOnBrowsingDataRemovedCallback(
         const BrowsingDataRemover::Callback& callback) {
   return GetOnBrowsingDataRemovedCallbacks()->Add(callback);
 }