Stop prerendering of a site if it uses the Credential Manager API.

chrome/browser/password_manager/credential_manager_browsertest.cc

 // Copyright 2016 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 "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/password_manager/password_manager_test_base.h"
 #include "chrome/browser/password_manager/password_store_factory.h"
+#include "chrome/browser/prerender/prerender_test_utils.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/passwords/passwords_model_delegate.h"
+#include "chrome/test/base/ui_test_utils.h"
 #include "components/password_manager/core/browser/password_bubble_experiment.h"
 #include "components/password_manager/core/browser/password_store_consumer.h"
 #include "components/password_manager/core/browser/test_password_store.h"
 #include "content/public/test/browser_test.h"
 #include "content/public/test/browser_test_utils.h"
 
 namespace {
 
+using prerender::test_utils::TestPrerender;
 // A helper class that synchronously waits until the password store handles a
 // GetLogins() request.
 class PasswordStoreResultsObserver
     : public password_manager::PasswordStoreConsumer {
  public:
   PasswordStoreResultsObserver() = default;
 
   void OnGetPasswordStoreResults(
       std::vector<std::unique_ptr<autofill::PasswordForm>> results) override {
     run_loop_.Quit();
@@ skipping 218 matching lines @@
   EXPECT_FALSE(prompt_observer->IsShowingUpdatePrompt());
   signin_form.skip_zero_click = false;
   signin_form.times_used = 1;
   signin_form.password_value = base::ASCIIToUTF16("API");
   password_manager::TestPasswordStore::PasswordMap stored =
       password_store->stored_passwords();
   ASSERT_EQ(1u, stored.size());
   EXPECT_EQ(signin_form, stored[signin_form.signon_realm][0]);
 }
 
+class CredentialManagerPrerenderTest
+    : public prerender::test_utils::PrerenderInProcessBrowserTest {
+ private:
+  std::vector<std::unique_ptr<TestPrerender>> PrerenderTestURLImpl(

[pasko, 2016/10/26 12:37:00]

in a concurrent change I am removing PrerenderTestURLImpl from
PrerenderInProcessBrowserTest.

Can you put this test inside prerender_browsertest and use PrerenderTestURL()
machinery instead? This would then be more logical to move autosignin.html to
chrome/test/data/prerender, but the good part is that prerender_autosignin.html
can be eliminated.

[vasilii, 2016/10/27 09:02:09]

Done.

+      const GURL& prerender_url,
+      const std::vector<prerender::FinalStatus>& expected_final_status_queue,
+      int expected_number_of_loads) override {
+    return NavigateWithPrerenders(prerender_url, expected_final_status_queue,
+                                  expected_number_of_loads);
+  }
+};
+
+IN_PROC_BROWSER_TEST_F(CredentialManagerPrerenderTest,
+                       AutosigninInPrerenderer) {
+  std::unique_ptr<TestPrerender> prerender = prerender_contents_factory()->
+      ExpectPrerenderContents(prerender::FINAL_STATUS_CREDENTIAL_MANAGER_API);
+  GURL test_page = embedded_test_server()->GetURL(
+      "/password/prerender_autosignin.html");
+  ui_test_utils::NavigateToURL(current_browser(), test_page);
+  prerender->WaitForCreate();

[pasko, 2016/10/26 12:37:00]

can just WaitForStop() without WaitForCreate()

[vasilii, 2016/10/27 09:02:09]

Done.

+  prerender->WaitForStop();

[pasko, 2016/10/26 12:37:00]

Do we also want to make sure that a request to password/done.html is not made?
RequestCounter in prerender_browsertest may come handy.

[vasilii, 2016/10/27 09:02:09]

Done.

+}
+
 }  // namespace