Delete GlobarError and Protector only after the error was removed from global list.

chrome/browser/protector/settings_change_global_error.cc

 // Copyright (c) 2011 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/protector/settings_change_global_error.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/stl_util.h"
@@ skipping 135 matching lines @@
         kBubbleKeepSettingIDs[change->type()],
         old_setting);
   }
 }
 
 bool SettingsChangeGlobalError::IsAcceptButtonDefault() {
   return false;
 }
 
 void SettingsChangeGlobalError::BubbleViewAcceptButtonPressed() {
+  closed_by_button_ = true;
   DCHECK(delegate_);
   VLOG(1) << "Apply changes";
   delegate_->OnApplyChanges();
-  closed_by_button_ = true;
 }
 
 void SettingsChangeGlobalError::BubbleViewCancelButtonPressed() {
+  closed_by_button_ = true;
   DCHECK(delegate_);
   VLOG(1) << "Discard changes";
   delegate_->OnDiscardChanges();
-  closed_by_button_ = true;
 }
 
 void SettingsChangeGlobalError::RemoveFromProfile() {
   DCHECK(delegate_);
   if (profile_)
     GlobalErrorServiceFactory::GetForProfile(profile_)->RemoveGlobalError(this);
   if (!closed_by_button_)
     delegate_->OnDecisionTimeout();
+  delegate_->OnRemovedFromProfile();
 }
 
 void SettingsChangeGlobalError::BubbleViewDidClose() {
   browser_ = NULL;
   if (!closed_by_button_) {
     BrowserThread::PostDelayedTask(
         BrowserThread::UI, FROM_HERE,
         base::Bind(&SettingsChangeGlobalError::RemoveFromProfile,
                    weak_factory_.GetWeakPtr()),
         kMenuItemDisplayPeriodMs);
@@ skipping 23 matching lines @@
 
 void SettingsChangeGlobalError::Show() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   DCHECK(profile_);
   browser_ = BrowserList::GetLastActiveWithProfile(profile_);
   if (browser_)
     ShowBubbleView(browser_);
 }
 
 }  // namespace protector