Change semicolon replace logic in url_fixer

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 284 matching lines @@
   // 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"},
   // Semicolon as scheme separator for standard schemes.
   {"http;//www.google.com/", "http://www.google.com/"},
   {"about;chrome", "chrome://chrome/"},
-  // Semicolon left as-is for non-standard schemes.
-  {"whatsup;//fool", "whatsup://fool"},
+  // 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/"},
   // Semicolon at start.
   {";http://www.google.com/", "http://%3Bhttp//www.google.com/"},
 };
 
 TEST(URLFixerTest, FixupURL) {
@@ skipping 213 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"));
 }