Rework of ResourcePrefetchPredictor browser tests infrastructure.

chrome/test/base/ui_test_utils.cc

Index: chrome/test/base/ui_test_utils.cc
diff --git a/chrome/test/base/ui_test_utils.cc b/chrome/test/base/ui_test_utils.cc
index 316f08b2b97ef2874b64f90c25eee90d7cd3ea78..d180631130ed0aa9e0e40ea082f27187c80cb33e 100644
--- a/chrome/test/base/ui_test_utils.cc
+++ b/chrome/test/base/ui_test_utils.cc
@@ -213,7 +213,8 @@ void NavigateToURLWithDispositionBlockUntilNavigationsComplete(
       return;
   } else if ((disposition == WindowOpenDisposition::CURRENT_TAB) ||
              (disposition == WindowOpenDisposition::NEW_FOREGROUND_TAB) ||
-             (disposition == WindowOpenDisposition::SINGLETON_TAB)) {
+             (disposition == WindowOpenDisposition::SINGLETON_TAB) ||
+             (disposition == WindowOpenDisposition::NEW_POPUP)) {

[ahemery, 2016/12/07 14:13:02]

I believe this one was just forgotten. Behavior was buggy with
NEW_POPUP+NAVIGATION.

[alexilin, 2016/12/07 16:57:11]

We don't need it for this CL anymore.

     // The currently selected tab is the right one.
     web_contents = browser->tab_strip_model()->GetActiveWebContents();
   }