Create net::FormatOriginForDisplay helper function.

net/base/net_util_icu_unittest.cc

 // Copyright 2014 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 "net/base/net_util.h"
 
 #include <string.h>
 
 #include <vector>
 
@@ skipping 346 matching lines @@
 struct UrlTestData {
   const char* const description;
   const char* const input;
   const char* const languages;
   FormatUrlTypes format_types;
   UnescapeRule::Type escape_rules;
   const wchar_t* output;  // Use |wchar_t| to handle Unicode constants easily.
   size_t prefix_len;
 };
 
+struct OriginTestData {
+  const char* description;
+  const char* input;
+  const char* languages;
+  const bool omit_scheme;
+  const wchar_t* output;
+};
+
 // A helper for IDN*{Fast,Slow}.
 // Append "::<language list>" to |expected| and |actual| to make it
 // easy to tell which sub-case fails without debugging.
 void AppendLanguagesToOutputs(const char* languages,
                               base::string16* expected,
                               base::string16* actual) {
   base::string16 to_append = ASCIIToUTF16("::") + ASCIIToUTF16(languages);
   expected->append(to_append);
   actual->append(to_append);
 }
@@ skipping 697 matching lines @@
                        UnescapeRule::NORMAL, omit_http_start_with_ftp_offsets);
 
   const size_t omit_all_offsets[] = {
     0, kNpos, kNpos, kNpos, kNpos, kNpos, kNpos, 0, kNpos, kNpos, kNpos, kNpos,
     0, 1, 2, 3, 4, 5, 6, 7
   };
   CheckAdjustedOffsets("http://user@foo.com/", "en", kFormatUrlOmitAll,
                        UnescapeRule::NORMAL, omit_all_offsets);
 }
 
+TEST(NetUtilTest, FormatOriginForDisplay) {
+  const OriginTestData tests[] = {
+      {"Empty URL", "", "", false, L""},
+      {"HTTP URL, no omit scheme",
+       "http://www.google.com/",
+       "",
+       false,
+       L"http://www.google.com"},
+      {"HTTP URL, omit scheme",
+       "http://www.google.com/",
+       "",
+       true,
+       L"www.google.com"},
+      {"HTTPS URL, no omit scheme",
+       "https://www.google.com/",
+       "",
+       false,
+       L"https://www.google.com"},
+      {"HTTPS URL, omit scheme",
+       "https://www.google.com/",
+       "",
+       true,
+       L"www.google.com"},
+      {"Standard HTTP port",
+       "http://www.google.com:80/",
+       "",
+       false,
+       L"http://www.google.com"},
+      {"Standard HTTPS port",
+       "https://www.google.com:443/",
+       "",
+       false,
+       L"https://www.google.com"},
+      {"Non-standard HTTP port",
+       "http://www.google.com:9000/",
+       "",
+       false,
+       L"http://www.google.com:9000"},
+      {"Non-standard HTTPS port",
+       "https://www.google.com:9000/",
+       "",
+       false,
+       L"https://www.google.com:9000"},
+      {"File URI, omit scheme",
+       "file:///usr/example/file.html",
+       "",
+       true,
+       L"/usr/example/file.html"},
+      {"File URI, no omit scheme",
+       "file:///usr/example/file.html",
+       "",
+       false,
+       L"file:///usr/example/file.html"},
+      {"HTTP URL with path",
+       "http://www.google.com/test.html",
+       "",
+       false,
+       L"http://www.google.com"},
+      {"HTTPS URL with path",
+       "https://www.google.com/test.html",
+       "",
+       false,
+       L"https://www.google.com"},
+      {"Unusual secure scheme (wss)",
+       "wss://www.google.com/",
+       "",
+       false,
+       L"wss://www.google.com"},
+      {"Unusual non-secure scheme (gopher)",
+       "gopher://www.google.com/",
+       "",
+       false,
+       L"gopher://www.google.com"},
+      {"Unlisted scheme (chrome)",
+       "chrome://version",
+       "",
+       false,
+       L"chrome://version"},
+      {"HTTP IP address",
+       "http://173.194.65.103",
+       "",
+       false,
+       L"http://173.194.65.103"},
+      {"HTTPS IP address",
+       "https://173.194.65.103",
+       "",
+       false,
+       L"https://173.194.65.103"},
+      {"HTTPS IP address, non-default port",
+       "https://173.194.65.103:8443",
+       "",
+       false,
+       L"https://173.194.65.103:8443"},
+      {"HTTPS IP address, omit scheme",
+       "https://173.194.65.103",
+       "",
+       true,
+       L"173.194.65.103"},
+      {"HTTP filesystem: URL with path",
+       "filesystem:http://www.google.com/test.html",
+       "",
+       false,
+       L"filesystem:http://www.google.com"},

[asanka, 2015/05/11 23:39:44]

Nit: filesystem URIs are of the form filesystem:<origin>/<filessytem
type>/<path>

[palmer, 2015/05/12 22:20:53]

Done.

+      {"HTTP filesystem: URL with path, omit scheme",
+       "filesystem:http://www.google.com/test.html",
+       "",
+       true,
+       L"filesystem:www.google.com"},
+  };

[asanka, 2015/05/11 23:39:44]

Do you want to test how the code behaves with invalid URLs?

[palmer, 2015/05/12 22:20:53]

I added some, but I am not sure if it really helps. In most/all of the contexts
this function is likely to get called in, a page has loaded (implying the URL
was valid).

+  for (size_t i = 0; i < arraysize(tests); ++i) {
+    base::string16 formatted = FormatOriginForDisplay(
+        GURL(tests[i].input), tests[i].languages, tests[i].omit_scheme);
+    EXPECT_EQ(WideToUTF16(tests[i].output), formatted) << tests[i].description;
+  }
+}
+
 }  // namespace net