Fix URLCanonTest.CanonicalizeFileSystemURL

src/url_canon_unittest.cc

 // Copyright 2007, Google Inc.
 // All rights reserved.
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 // notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
@@ skipping 1660 matching lines @@
     EXPECT_EQ(4, out_parsed.scheme.len);
 
     EXPECT_EQ(cases[i].expected_host.begin, out_parsed.host.begin);
     EXPECT_EQ(cases[i].expected_host.len, out_parsed.host.len);
 
     EXPECT_EQ(cases[i].expected_path.begin, out_parsed.path.begin);
     EXPECT_EQ(cases[i].expected_path.len, out_parsed.path.len);
   }
 }
 
-/*
-TODO(ericu) bug 121671 this test fails.
 #ifdef FULL_FILESYSTEM_URL_SUPPORT
 TEST(URLCanonTest, CanonicalizeFileSystemURL) {
   struct URLCase {
     const char* input;
     const char* expected;
     bool expected_success;
   } cases[] = {
     {"Filesystem:htTp://www.Foo.com:80/tempoRary", "filesystem:http://www.foo.com/tempoRary/", true},
     {"filesystem:httpS://www.foo.com/temporary/", "filesystem:https://www.foo.com/temporary/", true},
     {"filesystem:http://www.foo.com//", "filesystem:http://www.foo.com//", false},
     {"filesystem:http://www.foo.com/persistent/bob?query#ref", "filesystem:http://www.foo.com/persistent/bob?query#ref", true},
     {"filesystem:fIle://\\temporary/", "filesystem:file:///temporary/", true},
     {"filesystem:fiLe:///temporary", "filesystem:file:///temporary/", true},
-    {"filesystem:file://///temporary", "filesystem:file:///temporary/", true},
+    {"filesystem:file:///////temporary", "filesystem:file:////", false},
     {"filesystem:File:///temporary/Bob?qUery#reF", "filesystem:file:///temporary/Bob?qUery#reF", true},
   };
 
   for (size_t i = 0; i < ARRAYSIZE(cases); i++) {
     int url_len = static_cast<int>(strlen(cases[i].input));
     url_parse::Parsed parsed;
     url_parse::ParseFileSystemURL(cases[i].input, url_len, &parsed);
 
     url_parse::Parsed out_parsed;
     std::string out_str;
     url_canon::StdStringCanonOutput output(&out_str);
     bool success = url_canon::CanonicalizeFileSystemURL(cases[i].input, url_len,
                                                         parsed, NULL, &output,
                                                         &out_parsed);
     output.Complete();
 
     EXPECT_EQ(cases[i].expected_success, success);
     EXPECT_EQ(cases[i].expected, out_str);
 
     // Make sure the spec was properly identified, the filesystem canonicalizer
     // has different code for writing the spec.
     EXPECT_EQ(0, out_parsed.scheme.begin);
     EXPECT_EQ(10, out_parsed.scheme.len);
   }
 }
 #endif
-*/
 
 TEST(URLCanonTest, CanonicalizePathURL) {
   // Path URLs should get canonicalized schemes but nothing else.
   struct PathCase {
     const char* input;
     const char* expected;
   } path_cases[] = {
     {"javascript:", "javascript:"},
     {"JavaScript:Foo", "javascript:Foo"},
     {":\":This /is interesting;?#", ":\":This /is interesting;?#"},
@@ skipping 395 matching lines @@
   url_canon::StdStringCanonOutput repl_output(&repl_str);
   url_canon::ReplaceFileURL(src, parsed, repl, NULL, &repl_output, &repl_parsed);
   repl_output.Complete();
 
   // Generate the expected string and check.
   std::string expected("file:///foo?");
   for (size_t i = 0; i < new_query.length(); i++)
     expected.push_back('a');
   EXPECT_TRUE(expected == repl_str);
 }