Fix top crasher on the AvatarMenuButton destructor

chrome/browser/ui/views/avatar_menu_button.cc

Index: chrome/browser/ui/views/avatar_menu_button.cc
===================================================================
--- chrome/browser/ui/views/avatar_menu_button.cc	(revision 99454)
+++ chrome/browser/ui/views/avatar_menu_button.cc	(working copy)
@@ -34,8 +34,6 @@
 #if defined(OS_WIN)
   if (base::win::GetVersion() < base::win::VERSION_WIN7)
     return;
-  // During destruction of the browser frame, we might not have a window
-  // so the taksbar button will be removed by windows anyway.
   BrowserWindow* bw = browser->window();
   if (!bw)

[viettrungluu, 2011/09/04 04:54:01]

Are these checks (against |bw| and |window|, below) still necessary in this
function? Maybe they should be DCHECK()s?

     return;
@@ -75,6 +73,10 @@
 }
 
 AvatarMenuButton::~AvatarMenuButton() {
+  // During destruction of the browser frame, we might not have a window
+  // so the taksbar button will be removed by windows anyway.

[viettrungluu, 2011/09/04 04:54:01]

s/taksbar/taskbar

and s/windows/Windows

+  if (browser_->IsAttemptingToCloseBrowser())
+    return;
   DrawTaskBarDecoration(browser_, NULL);
 }