Profiles: Fixed crash when avatar bubble is launched using ctrl+shift+m with only default profile

chrome/browser/ui/gtk/avatar_menu_button_gtk.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/gtk/avatar_menu_button_gtk.h"
 
 #include "base/i18n/rtl.h"
 #include "chrome/browser/profiles/profile_metrics.h"
 #include "chrome/browser/ui/gtk/avatar_menu_bubble_gtk.h"
 #include "chrome/browser/ui/gtk/bubble/bubble_gtk.h"
@@ skipping 31 matching lines @@
                                               GdkEventButton* event) {
   if (event->button != 1)
     return FALSE;
 
   ShowAvatarBubble();
   ProfileMetrics::LogProfileOpenMethod(ProfileMetrics::ICON_AVATAR_BUBBLE);
   return TRUE;
 }
 
 void AvatarMenuButtonGtk::ShowAvatarBubble() {
-  new AvatarMenuBubbleGtk(browser_, widget_.get(), arrow_location_, NULL);
+  if (gtk_widget_get_parent_window(widget_.get()))

[James Hawkins, 2011/12/01 17:15:55]

This isn't the same logic as ShouldDisplayAvatar().

[sail, 2011/12/01 19:12:23]

I think this is ok. ShoudDisplayAvatar decides if the button gets placed in the
title bar.

At this point we just want to know if we're in the title bar or not. If we're
not then we shouldn't show the bubble view.

I think a comment should be added to this code. Maybe something like:
// Only show the avatar bubble if the avatar button is in the title bar.

+    new AvatarMenuBubbleGtk(browser_, widget_.get(), arrow_location_, NULL);
 }