Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1605)

Unified Diff: chrome/browser/net/url_fixer_upper.h

Issue 7068007: Revise about: and chrome: url handling. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments. Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/net/url_fixer_upper.h
diff --git a/chrome/browser/net/url_fixer_upper.h b/chrome/browser/net/url_fixer_upper.h
index e6096bb4cf9569445e3e20060b5df92a83e7e2e5..a0970fb47f205a50c05e6f1034630e4ead661597 100644
--- a/chrome/browser/net/url_fixer_upper.h
+++ b/chrome/browser/net/url_fixer_upper.h
@@ -49,6 +49,11 @@ namespace URLFixerUpper {
// instead of ".com").
GURL FixupURL(const std::string& text, const std::string& desired_tld);
+ // Normalizes about:foo, about://foo, and chrome:foo to chrome://foo.
+ // The host "version" is added if the url does not contain a host ("about:");
+ // otherwise GURL considers the result invalid.
+ GURL FixupChromeURL(const GURL& url);
abarth-chromium 2011/05/31 19:27:54 This description doesn't seem quite right. This f
msw 2011/05/31 20:58:25 Done.
+
// Converts |text| to a fixed-up URL, allowing it to be a relative path on
// the local filesystem. Begin searching in |base_dir|; if empty, use the
// current working directory. If this resolves to a file on disk, convert it

Powered by Google App Engine
This is Rietveld 408576698