Do not show permission bubble when browser is not active

chrome/browser/permissions/permission_request_manager_browsertest.cc

 // 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.
 
 #include "chrome/browser/permissions/permission_request_manager.h"
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "build/build_config.h"
 #include "chrome/browser/custom_handlers/protocol_handler_registry_factory.h"
@@ skipping 282 matching lines @@
       browser(),
       embedded_test_server()->GetURL(
           "/permissions/requests-before-after-load.html"),
       1);
   bubble_factory()->WaitForPermissionBubble();
 
   EXPECT_EQ(1, bubble_factory()->show_count());
   EXPECT_EQ(1, bubble_factory()->total_request_count());
 }
 
+// Tests that the permission bubble should only show on active browser
+// (crbug.com/704350).
+IN_PROC_BROWSER_TEST_F(PermissionRequestManagerBrowserTest,
+                       ShowPromptOnActiveBrowser) {
+  ASSERT_TRUE(embedded_test_server()->Start());
+
+  // Simulate a native widget deactivation for browser().
+  BrowserList::GetInstance()->NotifyBrowserNoLongerActive(browser());

[Qiang(Joe) Xu, 2017/03/27 21:09:52]

This is a work around for testing this, otherwise we need native widget
deactivation for sending this signal. WDYT?

+  ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
+      browser(),
+      embedded_test_server()->GetURL(
+          "/permissions/requests-before-after-load.html"),
+      1);
+  bubble_factory()->WaitForPermissionBubble();
+
+  EXPECT_EQ(0, bubble_factory()->show_count());
+  EXPECT_EQ(0, bubble_factory()->total_request_count());
+
+  BrowserList::GetInstance()->SetLastActive(browser());
+  bubble_factory()->WaitForPermissionBubble();
+  EXPECT_EQ(1, bubble_factory()->show_count());
+  EXPECT_EQ(1, bubble_factory()->total_request_count());
+}
+
 // Navigating twice to the same URL should be equivalent to refresh. This means
 // showing the bubbles twice.
 // http://crbug.com/512849 flaky
 #if defined(OS_WIN)
 #define MAYBE_NavTwice DISABLED_NavTwice
 #else
 #define MAYBE_NavTwice NavTwice
 #endif
 IN_PROC_BROWSER_TEST_F(PermissionRequestManagerBrowserTest, MAYBE_NavTwice) {
   ASSERT_TRUE(embedded_test_server()->Start());
@@ skipping 178 matching lines @@
 #define MAYBE_InvokeDialog_protected_media InvokeDialog_protected_media
 #else
 #define MAYBE_InvokeDialog_protected_media DISABLED_InvokeDialog_protected_media
 #endif
 IN_PROC_BROWSER_TEST_F(PermissionDialogTest,
                        MAYBE_InvokeDialog_protected_media) {
   RunDialog();
 }
 
 }  // anonymous namespace