Be a little more careful whether something is an URL or a file path.

chrome/browser/net/url_fixer_upper_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 <stdlib.h>
 
 #include "base/basictypes.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
@@ skipping 498 matching lines @@
   // test relative directories and duplicate slashes
   // (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();
   EXPECT_TRUE(IsMatchingFileURL(URLFixerUpper::FixupRelativeFile(dir,
       FilePath(relative_file_str)).possibly_invalid_spec(), full_path));
 
   // done with the subdir
   EXPECT_TRUE(file_util::Delete(full_path, false));
   EXPECT_TRUE(file_util::Delete(new_dir, true));
+
+  // test than an obvious HTTP URL isn't accidentally treated as an absolute

[jar (doing other things), 2011/08/06 02:33:28]

nit: Comments should be sentences, starting with upper case, and ending with
periods (even if the other comments had a strange style.... and should be
fixed... by some other CL.)

+  // file path (on account of system-specific craziness)
+  FilePath empty_path;
+  FilePath http_url_path(FILE_PATH_LITERAL("http://../"));
+  EXPECT_TRUE(URLFixerUpper::FixupRelativeFile(
+      empty_path, http_url_path).SchemeIs("http"));
 }