Convert OneClickSigninBubbleViewTest to browser_tests.

chrome/browser/ui/views/sync/one_click_signin_bubble_view.h

Index: chrome/browser/ui/views/sync/one_click_signin_bubble_view.h
diff --git a/chrome/browser/ui/views/sync/one_click_signin_bubble_view.h b/chrome/browser/ui/views/sync/one_click_signin_bubble_view.h
index 78b58f8fd98c5b5b2cca7cd9b7ba7e5ede9f0917..28cc22c824d8ec2decb5d924c05b420c8e9b3dc8 100644
--- a/chrome/browser/ui/views/sync/one_click_signin_bubble_view.h
+++ b/chrome/browser/ui/views/sync/one_click_signin_bubble_view.h
@@ -53,6 +53,14 @@ class OneClickSigninBubbleView : public views::BubbleDelegateView,
   // method is meant to be called only from tests.
   static OneClickSigninBubbleView* view_for_testing() { return bubble_view_; }
 
+  // We normally close the bubble any time it becomes inactive but this can lead
+  // to flaky tests where unexpected UI events are triggering this behavior.
+  // Tests should call this with "false" for more consistent operation.
+  // (crbug.com/266972)
+  static void set_close_on_deactivate_for_testing(bool close) {

[fdoray, 2013/08/09 18:22:42]

The method is not called set_close_on_deactivate() because I don't want to
override BubbleDelegateView::set_close_on_deactivate.

This is an issue in AvatarMenuBubbleView and ProfileChooserView
crbug.com/270888.

+    close_on_deactivate_ = close;
+  }
+
  protected:
   // Creates a OneClickSigninBubbleView.
   OneClickSigninBubbleView(
@@ -148,6 +156,8 @@ class OneClickSigninBubbleView : public views::BubbleDelegateView,
   // The bubble, if we're showing one.
   static OneClickSigninBubbleView* bubble_view_;
 
+  static bool close_on_deactivate_;

[Roger Tawa OOO till Jul 10th, 2013/08/09 18:55:10]

Better to rename this variable so its not the same as the base class member. 
Maybe |close_on_deactivate_for_testing_| ?

+
   DISALLOW_COPY_AND_ASSIGN(OneClickSigninBubbleView);
 };