Revise about: and chrome: url handling.

chrome/browser/ui/webui/web_ui_util.cc

 // Copyright (c) 2011 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 "chrome/browser/ui/webui/web_ui_util.h"
 
 #include <vector>
 
 #include "base/base64.h"
 #include "base/string_util.h"
@@ skipping 24 matching lines @@
   scoped_refptr<RefCountedMemory> raw_icon(icon_data);
   std::string str_url;
   str_url.insert(str_url.end(),
     raw_icon->front(),
     raw_icon->front() + raw_icon->size());
   base::Base64Encode(str_url, &str_url);
   str_url.insert(0, "data:image/png;base64,");
   return str_url;
 }
 
+bool IsChromeURL(const GURL& url) {
+  return url.SchemeIs(chrome::kChromeUIScheme) ||
+         url.SchemeIs(chrome::kAboutScheme);
+}
+
+std::string GetChromeURLHost(const GURL& url) {
+  if (url.has_host())
+    return url.host();
+  std::string path = url.path();
+  // "about://host" has path "//host"; skip "//" in this special case.
+  size_t begin = StartsWithASCII(path, "//", true) ? 2 : 0;
+  size_t end = path.find_first_of("/?#", begin);
+  size_t length = (end == std::string::npos) ? end : end - begin;
+  return path.length() > begin ? path.substr(begin, length) : std::string();

[abarth-chromium, 2011/05/27 17:19:25]

These functions are way too low-level for chrome/browser/ui.

I'm sympathetic if you're rushing to meet M13, but we should at least have TODO
comments and bugs filed about doing this the right way.  (Of course, doing it
the right way first is preferable.)

[msw, 2011/05/31 13:34:29]

I rewrote this as URLFixerUpper::FixupChromeURL.

+}
+
+std::string GetChromeURLPath(const GURL& url) {
+  if (url.has_host())
+    return url.path();
+  std::string path = url.path();
+  // "about://host" has path "//host"; skip "//" in this special case.
+  size_t host_begin = StartsWithASCII(path, "//", true) ? 2 : 0;
+  size_t path_begin = path.find_first_of("/?#", host_begin);
+  return path.length() > path_begin ? path.substr(path_begin) : std::string();
+}
+
+bool HostEquals(const GURL& url, const char* host) {
+  return LowerCaseEqualsASCII(GetChromeURLHost(url), host);
+}
+
 bool ChromeURLHostEquals(const GURL& url, const char* host) {
-  return (url.SchemeIs(chrome::kChromeUIScheme) ||
-          url.SchemeIs(chrome::kAboutScheme)) &&
-         LowerCaseEqualsASCII(url.host(), host);
+  return IsChromeURL(url) && HostEquals(url, host);
 }
 
 }  // namespace web_ui_util