Mark URLs with empty schemes as invalid.

components/url_formatter/url_fixer_unittest.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 <stddef.h>
 #include <stdlib.h>
 
 #include "base/base_paths.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
@@ skipping 164 matching lines @@
   { "http://:b005::68]", "http",
     url::Component(0, 4), // scheme
     url::Component(), // username
     url::Component(), // password
     url::Component(7, 10), // host
     url::Component(), // port
     url::Component(), // path
     url::Component(), // query
     url::Component(), // ref
   },
-  // Can't do anything useful with this.
-  { ":b005::68]", "",
-    url::Component(0, 0), // scheme
+  { ":b005::68]", "http",
+    url::Component(), // scheme
     url::Component(), // username
     url::Component(), // password
-    url::Component(), // host
+    url::Component(1, 9), // host
     url::Component(), // port
     url::Component(), // path
     url::Component(), // query
     url::Component(), // ref
   },
 };
 
 typedef testing::Test URLFixerTest;
 
 TEST(URLFixerTest, SegmentURL) {
@@ skipping 89 matching lines @@
   {"http://foo.com/s?q=\xf0\x90\x80\x85", "http://foo.com/s?q=%F0%90%80%85"},
   // URLs which end with 0xA0 (non-break space in ISO-8859).
   {"http://foo.com/s?q=\xd0\xa0", "http://foo.com/s?q=%D0%A0"},
   {"http://foo.com/s?q=\xec\x97\xa0", "http://foo.com/s?q=%EC%97%A0"},
   {"http://foo.com/s?q=\xf0\x90\x80\xa0", "http://foo.com/s?q=%F0%90%80%A0"},
   // URLs containing IPv6 literals.
   {"[2001:db8::2]", "http://[2001:db8::2]/"},
   {"[::]:80", "http://[::]/"},
   {"[::]:80/path", "http://[::]/path"},
   {"[::]:180/path", "http://[::]:180/path"},
-  // TODO(pmarks): Maybe we should parse bare IPv6 literals someday.
-  {"::1", "::1"},
+  // TODO(pmarks): Maybe we should parse bare IPv6 literals someday. Currently
+  // the first colon is treated as a scheme separator, and we default
+  // unspecified schemes to "http".
+  {"::1", "http://:1/"},
   // Semicolon as scheme separator for standard schemes.
   {"http;//www.google.com/", "http://www.google.com/"},
   {"about;chrome", "chrome://chrome/"},
   // Semicolon in non-standard schemes is not replaced by colon.
   {"whatsup;//fool", "http://whatsup%3B//fool"},
   // Semicolon left as-is in URL itself.
   {"http://host/port?query;moar", "http://host/port?query;moar"},
   // Fewer slashes than expected.
   {"http;www.google.com/", "http://www.google.com/"},
   {"http;/www.google.com/", "http://www.google.com/"},
@@ skipping 223 matching lines @@
   EXPECT_TRUE(base::DeleteFile(full_path, false));
   EXPECT_TRUE(base::DeleteFile(new_dir, true));
 
   // Test that an obvious HTTP URL isn't accidentally treated as an absolute
   // file path (on account of system-specific craziness).
   base::FilePath empty_path;
   base::FilePath http_url_path(FILE_PATH_LITERAL("http://../"));
   EXPECT_TRUE(url_formatter::FixupRelativeFile(empty_path, http_url_path)
                   .SchemeIs("http"));
 }