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

chrome/browser/browser_about_handler.h

Index: chrome/browser/browser_about_handler.h
diff --git a/chrome/browser/browser_about_handler.h b/chrome/browser/browser_about_handler.h
index 93b1a205f010a3f7022b94b0953f456a4407abca..7cc3785dc8b1ae26260f64b405cbda9f3320fd6b 100644
--- a/chrome/browser/browser_about_handler.h
+++ b/chrome/browser/browser_about_handler.h
@@ -11,6 +11,10 @@ namespace content {
 class BrowserContext;
 }
 
+// A preliminary URLHandler that performs cleanup on the URL before it is
+// rewritten.  Changes that happen here will not lead to a virtual URL.
+bool FixupBrowserAboutURL(GURL* url, content::BrowserContext* browser_context);

[msw, 2015/02/18 22:01:23]

Looking through all the URLHandler code, I don't see any actual handlers that
use |browser_context|. If it's not actually used, we should just nix the arg.

[Charlie Reis, 2015/02/18 22:16:17]

Sadly, HandleNewTabURLRewrite and HandleNewTabURLReverseRewrite use it.  (Would
have been nice!)

+
 // Returns true if the given URL will be handled by the browser about handler.
 // Nowadays, these go through the webui, so the return is always false.
 // Either way, |url| will be processed by url_fixer::FixupURL, which