Sanitize enclosing quotes around paths in policies.

chrome/browser/policy/policy_path_parser_unittest.cc

-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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 <base/file_path.h>
 #include <chrome/browser/policy/policy_path_parser.h>
 
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace policy {
 
@@ skipping 14 matching lines @@
   FilePath::StringType no_vars_result =
       path_parser::ExpandPathVariables(no_vars);
   ASSERT_EQ(no_vars_result, no_vars);
 
   // This is unknown variable and shouldn't be substituted.
   FilePath::StringType unknown_vars(FILE_PATH_LITERAL("//$C/${buggy}"));
   FilePath::StringType unknown_vars_result =
       path_parser::ExpandPathVariables(unknown_vars);
   ASSERT_EQ(unknown_vars_result, unknown_vars);
 
+  // Trim quotes around, but not inside paths. Test against bug 80211.
+  FilePath::StringType no_quotes(FILE_PATH_LITERAL("//$C/\"a\"/$path"));
+  FilePath::StringType single_quotes(FILE_PATH_LITERAL("'//$C/\"a\"/$path'"));
+  FilePath::StringType double_quotes(FILE_PATH_LITERAL("\"//$C/\"a\"/$path\""));
+  FilePath::StringType quotes_result =
+      path_parser::ExpandPathVariables(single_quotes);
+  ASSERT_EQ(quotes_result, no_quotes);
+  quotes_result = path_parser::ExpandPathVariables(double_quotes);
+  ASSERT_EQ(quotes_result, no_quotes);
+
   // Both should have been substituted.
   FilePath::StringType vars(FILE_PATH_LITERAL("${user_name}${machine_name}"));
   FilePath::StringType vars_result = path_parser::ExpandPathVariables(vars);
   ASSERT_EQ(vars_result.find(FILE_PATH_LITERAL("${user_name}")),
             FilePath::StringType::npos);
   ASSERT_EQ(vars_result.find(FILE_PATH_LITERAL("${machine_name}")),
             FilePath::StringType::npos);
 
   // Should substitute only one instance.
   vars = FILE_PATH_LITERAL("${machine_name}${machine_name}");
@@ skipping 40 matching lines @@
                        FILE_PATH_LITERAL("${global_app_data}"));
   CheckForSubstitution(FILE_PATH_LITERAL("//$C/${program_files}"),
                        FILE_PATH_LITERAL("${program_files}"));
   CheckForSubstitution(FILE_PATH_LITERAL("//$C/${windows}"),
                        FILE_PATH_LITERAL("${windows}"));
 }
 
 #endif  // OS_WIN
 
 }  // namespace policy