Fix 3 panel related bugs.

chrome/browser/ui/panels/panel_settings_menu_model.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/ui/panels/panel_settings_menu_model.h"
 
 #include "base/logging.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 71 matching lines @@
     case COMMAND_CONFIGURE:
       DCHECK(!extension->options_url().is_empty());
       browser->GetProfile()->GetExtensionProcessManager()->OpenOptionsPage(
           extension, browser);
       break;
     case COMMAND_DISABLE:
       browser->GetProfile()->GetExtensionService()->DisableExtension(
           extension->id());
       break;
     case COMMAND_UNINSTALL:
-      // TODO(jianli): Need to handle the case that the extension API requests
-      // the panel to be closed when the uninstall dialog is still showing.
-      // http://crbug.com/102742
+      // When the owning panel is being closed by the extension API, the
+      // currently showing uninstall dialog will also be dismissed.
       extension_uninstall_dialog_.reset(
           ExtensionUninstallDialog::Create(browser->GetProfile(), this));
       extension_uninstall_dialog_->ConfirmUninstall(extension);
       break;
     case COMMAND_MANAGE:
       browser->ShowOptionsTab(chrome::kExtensionsSubPage);
       break;
     default:
       NOTREACHED();
       break;
   }
 }
 
 void PanelSettingsMenuModel::ExtensionUninstallAccepted() {
   const Extension* extension = panel_->GetExtension();
   DCHECK(extension);
 
   panel_->browser()->GetProfile()->GetExtensionService()->
       UninstallExtension(extension->id(), false, NULL);
 }
 
 void PanelSettingsMenuModel::ExtensionUninstallCanceled() {
 }