GTK: Add CHECKs() to try to find the cause of a GlobalBookmarkMenu crash.

chrome/browser/ui/gtk/global_bookmark_menu.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/global_bookmark_menu.h"
 
 #include <dlfcn.h>
 #include <gtk/gtk.h>
 
 #include "base/logging.h"
@@ skipping 92 matching lines @@
   // Only display the other bookmarks folder in the menu if it has items in it.
   const BookmarkNode* other_node = model->other_node();
   if (other_node->child_count()) {
     GtkWidget* submenu = gtk_menu_new();
     AddNodeToMenu(other_node, submenu);
 
     AddBookmarkMenuItem(bookmark_menu_.get(), gtk_separator_menu_item_new());
 
     GtkWidget* menu_item = gtk_image_menu_item_new_with_label(
         l10n_util::GetStringUTF8(IDS_BOOMARK_BAR_OTHER_FOLDER_NAME).c_str());
-    gtk_util::SetAlwaysShowImage(menu_item);
     gtk_menu_item_set_submenu(GTK_MENU_ITEM(menu_item), submenu);
     gtk_image_menu_item_set_image(
         GTK_IMAGE_MENU_ITEM(menu_item),
         gtk_image_new_from_pixbuf(default_folder_));
+    gtk_util::SetAlwaysShowImage(menu_item);
 
     AddBookmarkMenuItem(bookmark_menu_.get(), menu_item);
   }
 }
 
 void GlobalBookmarkMenu::AddBookmarkMenuItem(GtkWidget* menu,
                                              GtkWidget* menu_item) {
   g_object_set_data(G_OBJECT(menu_item), "type-tag",
                     GINT_TO_POINTER(GlobalMenuBar::TAG_BOOKMARK_CLEARABLE));
   gtk_menu_shell_append(GTK_MENU_SHELL(menu), menu_item);
   gtk_widget_show(menu_item);
 }
 
 void GlobalBookmarkMenu::AddNodeToMenu(const BookmarkNode* node,
                                        GtkWidget* menu) {
   int child_count = node->child_count();
   if (!child_count) {
     GtkWidget* item = gtk_menu_item_new_with_label(
         l10n_util::GetStringUTF8(IDS_MENU_EMPTY_SUBMENU).c_str());
     gtk_widget_set_sensitive(item, FALSE);
     AddBookmarkMenuItem(menu, item);
   } else {
     for (int i = 0; i < child_count; i++) {
       const BookmarkNode* child = node->GetChild(i);
       GtkWidget* item = gtk_image_menu_item_new();
-      gtk_util::SetAlwaysShowImage(item);
       ConfigureMenuItem(child, item);
       bookmark_nodes_[child] = item;
 
       if (child->is_folder()) {
         GtkWidget* submenu = gtk_menu_new();
         AddNodeToMenu(child, submenu);
         gtk_menu_item_set_submenu(GTK_MENU_ITEM(item), submenu);
       } else {
         g_object_set_data(G_OBJECT(item), "bookmark-node",
                           const_cast<BookmarkNode*>(child));
         g_signal_connect(item, "activate",
                          G_CALLBACK(OnBookmarkItemActivatedThunk), this);
       }
 
       AddBookmarkMenuItem(menu, item);
     }
   }
 }
 
 void GlobalBookmarkMenu::ConfigureMenuItem(const BookmarkNode* node,
                                            GtkWidget* menu_item) {
+  CHECK(node);
+  CHECK(menu_item);
+
   // This check is only to make things compile on Hardy; this code won't
   // display any visible widgets in older systems that don't have a global menu
   // bar.
   if (gtk_menu_item_set_label_sym) {
     gtk_menu_item_set_label_sym(
         GTK_MENU_ITEM(menu_item),
         bookmark_utils::BuildMenuLabelFor(node).c_str());
   }
 
   if (node->is_url()) {
     gtk_widget_set_tooltip_markup(
         menu_item,
         bookmark_utils::BuildTooltipFor(node).c_str());
   }
 
   const SkBitmap& bitmap = profile_->GetBookmarkModel()->GetFavicon(node);
   if (!bitmap.isNull()) {
     GdkPixbuf* pixbuf = gfx::GdkPixbufFromSkBitmap(&bitmap);
     gtk_image_menu_item_set_image(GTK_IMAGE_MENU_ITEM(menu_item),
                                   gtk_image_new_from_pixbuf(pixbuf));
     g_object_unref(pixbuf);
   } else {
-    gtk_image_menu_item_set_image(GTK_IMAGE_MENU_ITEM(menu_item),
-                                  gtk_image_new_from_pixbuf(
-                                      node->is_url() ? default_favicon_ :
-                                      default_folder_));
+    GdkPixbuf* pixbuf = node->is_url() ? default_favicon_ : default_folder_;
+    CHECK(pixbuf);
+    GtkWidget* image = gtk_image_new_from_pixbuf(pixbuf);
+    CHECK(image);
+
+    gtk_image_menu_item_set_image(GTK_IMAGE_MENU_ITEM(menu_item), image);
   }
+
+  gtk_util::SetAlwaysShowImage(menu_item);
 }
 
 GtkWidget* GlobalBookmarkMenu::MenuItemForNode(const BookmarkNode* node) {
   if (!node)
     return NULL;
   std::map<const BookmarkNode*, GtkWidget*>::iterator it =
       bookmark_nodes_.find(node);
   if (it == bookmark_nodes_.end())
     return NULL;
   return it->second;
@@ skipping 84 matching lines @@
 void GlobalBookmarkMenu::OnBookmarkItemActivated(GtkWidget* menu_item) {
   // The actual mouse event that generated this activated event was in a
   // different process. Go with something default.
   const BookmarkNode* node = static_cast<const BookmarkNode*>(
       g_object_get_data(G_OBJECT(menu_item), "bookmark-node"));
 
   browser_->OpenURL(node->GetURL(), GURL(), NEW_FOREGROUND_TAB,
                     PageTransition::AUTO_BOOKMARK);
 }