Made MessagePump a non-thread safe class.

chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller_unittest.mm

 // Copyright (c) 2012 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.
 
 #import "chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.h"
 
 #include "base/mac/scoped_nsobject.h"
+#include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_pump_mac.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/prefs/pref_service_syncable.h"
 #include "chrome/browser/profiles/avatar_menu_model.h"
 #include "chrome/browser/profiles/avatar_menu_model_observer.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
 #import "chrome/browser/ui/cocoa/cocoa_test_helper.h"
 #include "chrome/test/base/testing_browser_process.h"
 #include "chrome/test/base/testing_profile_manager.h"
 #include "testing/gtest_mac.h"
@@ skipping 117 matching lines @@
   }
 
   [controller() close];
 }
 
 // This subclass is used to inject a delegate into the hide/show edit link
 // animation.
 @interface TestingAvatarMenuItemController : AvatarMenuItemController
                                                  <NSAnimationDelegate> {
  @private
-  scoped_refptr<base::MessagePumpNSRunLoop> pump_;
+  scoped_ptr<base::MessagePumpNSRunLoop> pump_;
 }
 // After calling |-highlightForEventType:| an animation will possibly be
 // started. Since the animation is non-blocking, the run loop will need to be
 // spun (via the MessagePump) until the animation has finished.
 - (void)runMessagePump;
 @end
 
 @implementation TestingAvatarMenuItemController
 - (void)runMessagePump {
-  if (!pump_.get())
-    pump_ = new base::MessagePumpNSRunLoop;
+  if (!pump_)
+    pump_.reset(new base::MessagePumpNSRunLoop);
   pump_->Run(NULL);
 }
 
 - (void)willStartAnimation:(NSAnimation*)anim {
   [anim setDelegate:self];
 }
 
 - (void)animationDidEnd:(NSAnimation*)anim {
   [super animationDidEnd:anim];
   pump_->Quit();
@@ skipping 86 matching lines @@
   [controller() keyDown:event];
   EXPECT_EQ([[controller() items] objectAtIndex:0], GetHighlightedItem());
 
   [controller() keyDown:event];
   EXPECT_EQ([[controller() items] objectAtIndex:1], GetHighlightedItem());
 
   // There are no more items now so going up should stay at the first item.
   [controller() keyDown:event];
   EXPECT_EQ([[controller() items] objectAtIndex:1], GetHighlightedItem());
 }