Fixes Issue 7377: Regression: Omnibox trims URL ending with 0x85...

chrome/browser/net/url_fixer_upper_unittest.cc

 // Copyright (c) 2006-2008 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 <stdlib.h>
 
 #include "base/basictypes.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 159 matching lines @@
   // It would be better if this next case got treated as http, but I don't see
   // a clean way to guess this isn't the new-and-exciting "user" scheme.
   {"user:passwd@www.google.com:8080/", "", "user:passwd@www.google.com:8080/"},
   //{"file:///c:/foo/bar%20baz.txt", "", "file:///C:/foo/bar%20baz.txt"},
   {"ftp.google.com", "", "ftp://ftp.google.com/"},
   {"    ftp.google.com", "", "ftp://ftp.google.com/"},
   {"FTP.GooGle.com", "", "ftp://FTP.GooGle.com/"},
   {"ftpblah.google.com", "", "http://ftpblah.google.com/"},
   {"ftp", "", "http://ftp/"},
   {"google.ftp.com", "", "http://google.ftp.com/"},
+  // URLs which end with an ISO-8859 next-line (0x85).
+  { "http://google.com/search?q=\xd0\x85", "",
+    "http://google.com/search?q=\xd0\x85"
+  },
+  { "http://google.com/search?q=\xec\x97\x85", "",
+    "http://google.com/search?q=\xec\x97\x85"
+  },
+  { "http://google.com/search?q=\xf0\x90\x80\x85", "",
+    "http://google.com/search?q=\xf0\x90\x80\x85"
+  },
+  // URLs which end with a non-break space (0xA0).
+  { "http://google.com/search?q=\xd0\xa0", "",
+    "http://google.com/search?q=\xd0\xa0"
+  },
+  { "http://google.com/search?q=\xec\x97\xa0", "",
+    "http://google.com/search?q=\xec\x97\xa0"
+  },
+  { "http://google.com/search?q=\xf0\x90\x80\xa0", "",
+    "http://google.com/search?q=\xf0\x90\x80\xa0"
+  },
 };
 
 TEST(URLFixerUpperTest, FixupURL) {
   std::string output;
 
   for (size_t i = 0; i < arraysize(fixup_cases); ++i) {
     fixup_case value = fixup_cases[i];
     output = URLFixerUpper::FixupURL(value.input, value.desired_tld);
     EXPECT_EQ(value.output, output);
   }
@@ skipping 147 matching lines @@
   // (should resolve to the same file as above)
   relative_file_str = sub_dir.value() + FILE_PATH_LITERAL("/../") +
       sub_dir.value() + FILE_PATH_LITERAL("///./") + sub_file.value();
   fixedup = URLFixerUpper::FixupRelativeFile(dir, FilePath(relative_file_str));
   EXPECT_TRUE(IsMatchingFileURL(fixedup, full_path));
 
   // done with the subdir
   EXPECT_TRUE(file_util::Delete(full_path, false));
   EXPECT_TRUE(file_util::Delete(new_dir, true));
 }