Attempt at fixing crash in menus shown from infobars. Here's what the

chrome/browser/ui/views/infobars/infobar_view.h

Index: chrome/browser/ui/views/infobars/infobar_view.h
diff --git a/chrome/browser/ui/views/infobars/infobar_view.h b/chrome/browser/ui/views/infobars/infobar_view.h
index 75d87605cd3c9759f3cd7eee27308905f8322ab4..72979d90fadab31b6e1a9a5c9d769a2d5f0c1e3f 100644
--- a/chrome/browser/ui/views/infobars/infobar_view.h
+++ b/chrome/browser/ui/views/infobars/infobar_view.h
@@ -83,6 +83,11 @@ class InfoBarView : public InfoBar,
   // Convenience getter.
   const InfoBarContainer::Delegate* container_delegate() const;
 
+  // Subclasses must override and cancel any menus. Because we are deleted after
+  // a delay and after our delegate is deleted we have to explicitly cancel the
+  // menu rather than relying on the destructor to cancel the menu.
+  virtual void CancelMenu() = 0;

[Peter Kasting, 2011/09/12 18:06:00]

Nit: I would prefer to not make this pure virtual.  One way would be to simply
provide a no-op implementation here.  Another way would be to move the menu
runner object into this base class and then provide subclasses sufficient
accessors and/or hoist more of the menu-running code too (if it's sufficiently
the same in the subclasses that use it).

I would probably also name this CloseMenus() and simply comment it "Closes any
open menus" or similar.

+
  private:
   static const int kHorizontalPadding;