Move URL fixup to a preliminary phase that doesn't affect virtual URLs.

chrome/browser/browser_about_handler_unittest.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 "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "chrome/browser/browser_about_handler.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/testing_profile.h"
+#include "content/public/browser/navigation_controller.h"
+#include "content/public/browser/navigation_entry.h"
+#include "content/public/common/referrer.h"
 #include "content/public/test/test_browser_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 using content::BrowserThread;
+using content::NavigationController;
+using content::NavigationEntry;
+using content::Referrer;
 
 typedef testing::Test BrowserAboutHandlerTest;
 
 TEST_F(BrowserAboutHandlerTest, WillHandleBrowserAboutURL) {
   std::string chrome_prefix(content::kChromeUIScheme);
   chrome_prefix.append(url::kStandardSchemeSeparator);
   struct AboutURLTestData {
     GURL test_url;
     GURL result_url;
   } test_data[] = {
@@ skipping 41 matching lines @@
   base::MessageLoopForUI message_loop;
   content::TestBrowserThread ui_thread(BrowserThread::UI, &message_loop);
   TestingProfile profile;
 
   for (size_t i = 0; i < arraysize(test_data); ++i) {
     GURL url(test_data[i].test_url);
     WillHandleBrowserAboutURL(&url, &profile);
     EXPECT_EQ(test_data[i].result_url, url);
   }
 }
+
+// Ensure that minor BrowserAboutHandler fixup to a URL does not cause us to
+// keep a separate virtual URL, which would not be updated on redirects.
+// See https://crbug.com/449829.
+TEST_F(BrowserAboutHandlerTest, NoVirtualURLForFixup) {
+  GURL url("view-source:http://.foo");
+
+  // Fixup will remove the dot and add a slash.
+  GURL fixed_url("view-source:http://foo/");
+
+  // Rewriters will remove the view-source prefix and expect it to stay in the
+  // virtual URL.
+  GURL rewritten_url("http://foo/");
+
+  TestingProfile profile;
+  scoped_ptr<NavigationEntry> entry(NavigationController::CreateNavigationEntry(
+      url, Referrer(), ui::PAGE_TRANSITION_RELOAD, false, std::string(),
+      &profile));
+  EXPECT_EQ(fixed_url, entry->GetVirtualURL());
+  EXPECT_EQ(rewritten_url, entry->GetURL());
+}