Do not give instant focus if a view's toplevelwidget is not active

chrome/browser/extensions/api/downloads/downloads_api_browsertest.cc

Index: chrome/browser/extensions/api/downloads/downloads_api_browsertest.cc
diff --git a/chrome/browser/extensions/api/downloads/downloads_api_browsertest.cc b/chrome/browser/extensions/api/downloads/downloads_api_browsertest.cc
index 5d1cfcf59e682b0dfc9a61f5f1c6783a9388397b..5b23a659170007c855d8231eaaa0834a81970b73 100644
--- a/chrome/browser/extensions/api/downloads/downloads_api_browsertest.cc
+++ b/chrome/browser/extensions/api/downloads/downloads_api_browsertest.cc
@@ -35,6 +35,7 @@
 #include "chrome/common/extensions/api/downloads.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/in_process_browser_test.h"
+#include "chrome/test/base/interactive_test_utils.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "components/prefs/pref_service.h"
 #include "content/public/browser/browser_context.h"
@@ -472,6 +473,7 @@ class DownloadExtensionTest : public ExtensionApiTest {
     if (manager->InProgressCount() != 0)
       return NULL;
 
+    EXPECT_TRUE(ui_test_utils::BringBrowserWindowToFront(current_browser()));

[sky, 2016/09/28 21:56:53]

Why is this necessary? In process browser tests already do this. If it is
necessary, presumably it should be an ASSERT as if it fails the rest of the test
won't work, right?

[Qiang(Joe) Xu, 2016/09/29 00:07:25]

These two browser_tests are both related to more than one browser window
(because of incognito windows). I feel I could put it on a better place for this
one. Let me check it.

     ui_test_utils::NavigateToURL(current_browser(), slow_download_url);
 
     observer->WaitForFinished();