Disable close-on-deactivate during tests so suprious UI events don't cause flakes

chrome/browser/ui/views/avatar_menu_button_browsertest.cc

 // 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.
 
 #include "chrome/browser/ui/views/avatar_menu_button.h"
 
 #include "base/command_line.h"
 #include "base/path_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 78 matching lines @@
 AvatarMenuButton* AvatarMenuButtonTest::GetAvatarMenuButton() {
   BrowserView* browser_view = reinterpret_cast<BrowserView*>(
       browser()->window());
   return browser_view->frame()->GetAvatarMenuButton();
 }
 
 void AvatarMenuButtonTest::StartAvatarMenu() {
   AvatarMenuButton* button = GetAvatarMenuButton();
   ASSERT_TRUE(button);
 
+  AvatarMenuBubbleView::set_close_on_deactiavte(false);
+  ProfileChooserView::set_close_on_deactiavte(false);
   static_cast<views::MenuButtonListener*>(button)->OnMenuButtonClicked(
       NULL, gfx::Point());
   base::MessageLoop::current()->RunUntilIdle();
   EXPECT_NE(AvatarMenuBubbleView::IsShowing(),
             ProfileChooserView::IsShowing());
 }
 
 IN_PROC_BROWSER_TEST_P(AvatarMenuButtonTest, HideOnSecondClick) {
   if (!profiles::IsMultipleProfilesEnabled() ||
       UsingNewProfileChooser()) {
@@ skipping 74 matching lines @@
   int final_tab_count = browser()->tab_strip_model()->count();
   EXPECT_EQ(1U, browser_list->size());
   EXPECT_EQ(starting_tab_count + 1, final_tab_count);
 
   GURL tab_url = browser()->tab_strip_model()->GetActiveWebContents()->GetURL();
   EXPECT_EQ(std::string(chrome::kChromeUIUserChooserURL), tab_url.spec());
 }
 
 INSTANTIATE_TEST_CASE_P(Old, AvatarMenuButtonTest, testing::Values(false));
 INSTANTIATE_TEST_CASE_P(New, AvatarMenuButtonTest, testing::Values(true));