Fix did_replace_entry reported as false for meta refresh tags

chrome/browser/history/history_browsertest.cc

 // Copyright (c) 2012 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 <utility>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
@@ skipping 456 matching lines @@
   std::string script = "location.replace('form.html')";
   content::TestFrameNavigationObserver observer(frame);
   EXPECT_TRUE(ExecuteScript(frame, script));
   observer.Wait();
   GURL auto_subframe =
       ui_test_utils::GetTestUrl(base::FilePath().AppendASCII("History"),
                                 base::FilePath().AppendASCII("form.html"));
   ASSERT_FALSE(HistoryContainsURL(auto_subframe));
 }
 
-// HTTP meta-refresh redirects should have separate history entries.
-IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, RedirectHistory) {
+// HTTP meta-refresh redirects should have separate history entries iff the
+// timeout is > 1 seconds.
+IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, RedirectHistoryFast) {
   GURL redirector = ui_test_utils::GetTestUrl(
       base::FilePath().AppendASCII("History"),
-      base::FilePath().AppendASCII("redirector.html"));
+      base::FilePath().AppendASCII("redirector_fast.html"));
+  GURL landing_url =
+      ui_test_utils::GetTestUrl(base::FilePath().AppendASCII("History"),
+                                base::FilePath().AppendASCII("landing.html"));
+  ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(browser(),
+                                                            redirector, 2);
+  ASSERT_EQ(landing_url,
+            browser()->tab_strip_model()->GetActiveWebContents()->GetURL());
+  std::vector<GURL> urls(GetHistoryContents());
+  ASSERT_EQ(1u, urls.size());
+  ASSERT_EQ(landing_url, urls[0]);
+}
+
+// HTTP meta-refresh redirects should have separate history entries iff the
+// timeout is > 1 seconds.
+IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, RedirectHistorySlow) {
+  GURL redirector = ui_test_utils::GetTestUrl(
+      base::FilePath().AppendASCII("History"),
+      base::FilePath().AppendASCII("redirector_slow.html"));
   GURL landing_url = ui_test_utils::GetTestUrl(
       base::FilePath().AppendASCII("History"),
       base::FilePath().AppendASCII("landing.html"));
   ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
       browser(), redirector, 2);
   ASSERT_EQ(landing_url,
             browser()->tab_strip_model()->GetActiveWebContents()->GetURL());
   std::vector<GURL> urls(GetHistoryContents());
   ASSERT_EQ(2u, urls.size());
   ASSERT_EQ(landing_url, urls[0]);
@@ skipping 116 matching lines @@
 
   content::WebContents* active_web_contents =
       browser()->tab_strip_model()->GetActiveWebContents();
   ASSERT_EQ(web_contents, active_web_contents);
   ASSERT_EQ(history_url, active_web_contents->GetURL());
 
   content::WebContents* second_tab =
       browser()->tab_strip_model()->GetWebContentsAt(1);
   ASSERT_NE(history_url, second_tab->GetURL());
 }