[Extensions UI] Handle multiple warning bubbles racing to show

chrome/browser/ui/extensions/extension_message_bubble_browsertest.h

 // Copyright 2015 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.
 
 #ifndef CHROME_BROWSER_UI_EXTENSIONS_EXTENSION_MESSAGE_BUBBLE_BROWSERTEST_H_
 #define CHROME_BROWSER_UI_EXTENSIONS_EXTENSION_MESSAGE_BUBBLE_BROWSERTEST_H_
 
 #include <memory>
 
 #include "base/macros.h"
@@ skipping 12 matching lines @@
   };
 
   ExtensionMessageBubbleBrowserTest();
   ~ExtensionMessageBubbleBrowserTest() override;
 
   // BrowserActionsBarBrowserTest:
   void SetUpCommandLine(base::CommandLine* command_line) override;
   void TearDownOnMainThread() override;
 
   // Checks the position of the bubble present in the given |browser|, when the
-  // bubble should be anchored at the given |anchor|.
-  virtual void CheckBubble(Browser* browser, AnchorPosition anchor) = 0;
+  // bubble should be anchored at the given |anchor| and that the toolbar model
+  // is correctly highlighting.
+  void CheckBubble(Browser* browser,
+                   AnchorPosition anchor,
+                   bool should_be_highlighting);
+  // Performs the platform-specific checks.
+  virtual void CheckBubbleNative(Browser* browser, AnchorPosition anchor) = 0;
 
   // Closes the bubble present in the given |browser|.
-  virtual void CloseBubble(Browser* browser) = 0;
+  void CloseBubble(Browser* browser);
+  // Performs the platform-specific close.
+  virtual void CloseBubbleNative(Browser* browser) = 0;
 
   // Checks that there is no active bubble for the given |browser|.
-  virtual void CheckBubbleIsNotPresent(Browser* browser) = 0;
+  // We specify whether or not the toolbar model should be highlighting or there
+  // is a bubble active since another browser window may have an active bubble.
+  void CheckBubbleIsNotPresent(Browser* browser,
+                               bool should_profile_have_bubble,
+                               bool should_be_highlighting);
+  // Performs the platform-specific checks.
+  virtual void CheckBubbleIsNotPresentNative(Browser* browser) = 0;
 
   // Adds a new extension that uses the chrome_settings_overrides api to
   // override a setting specified in |settings_override_value|.
   void AddSettingsOverrideExtension(const std::string& settings_override_value);
 
   // The following are essentially the different tests, but we can't define the
   // tests in this file, since it relies on platform-specific implementation
   // (the above virtual methods).
 
   // Tests that an extension bubble will be anchored to an extension action when
@@ skipping 30 matching lines @@
   void TestControlledNewTabPageBubbleShown();
 
   // Tests that the bubble indicating an extension is controlling a user's
   // home page is shown.
   void TestControlledHomeBubbleShown();
 
   // Tests that the bubble indicating an extension is controlling a user's
   // search engine is shown.
   void TestControlledSearchBubbleShown();
 
+  // Tests that having multiple windows, all of which could be vying to show a
+  // warning bubble, behaves properly.
+  void TestBubbleWithMultipleWindows();
+
  private:
   std::unique_ptr<extensions::FeatureSwitch::ScopedOverride>
       dev_mode_bubble_override_;
 
   // The backing directory for a custom extension loaded during a test. Null if
   // no custom extension is loaded.
   std::unique_ptr<extensions::TestExtensionDir> custom_extension_dir_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionMessageBubbleBrowserTest);
 };
 
 #endif  // CHROME_BROWSER_UI_EXTENSIONS_EXTENSION_MESSAGE_BUBBLE_BROWSERTEST_H_