Fixed memory leak. Untangled confusing code with menu initialization and ownership.

chrome/browser/ui/views/button_dropdown_test.cc

Index: chrome/browser/ui/views/button_dropdown_test.cc
===================================================================
--- chrome/browser/ui/views/button_dropdown_test.cc	(revision 150365)
+++ chrome/browser/ui/views/button_dropdown_test.cc	(working copy)
@@ -15,7 +15,6 @@
  public:
   ButtonDropDownDragTest()
       : button_(NULL),
-        ALLOW_THIS_IN_INITIALIZER_LIST(menu_model_(this)),
         menu_shown_(false),
         menu_closed_(false) {
   }
@@ -25,7 +24,7 @@
 
   // ViewEventTestBase implementation.
   virtual void SetUp() OVERRIDE {
-    button_ = new views::ButtonDropDown(NULL, &menu_model_);
+    button_ = new views::ButtonDropDown(NULL, new ui::SimpleMenuModel(this));
 
     ViewEventTestBase::SetUp();
   }
@@ -125,7 +124,6 @@
 
  private:
   views::ButtonDropDown* button_;
-  ui::SimpleMenuModel menu_model_;
   bool menu_shown_;
   bool menu_closed_;
 };