Make sure ShellIntegration::CommandLineArgsForLauncher respects the UserDataDir policy.

chrome/browser/policy/policy_path_parser.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_POLICY_POLICY_PATH_PARSER_H_
 #define CHROME_BROWSER_POLICY_POLICY_PATH_PARSER_H_
 
 #include <string>
 
 #include "base/file_path.h"
@@ skipping 31 matching lines @@
 //   ${users}           - The folder where users profiles are stored
 //                        (example : "/Users")
 //   ${documents}       - The "Documents" folder of the current user.
 //                        (example : "/Users/johndoe/Documents")
 // Any non recognized variable is not being translated at all. Variables are
 // translated only once in every string because for most of these there is no
 // sense in concatenating them more than once in a single path.
 base::FilePath::StringType ExpandPathVariables(
     const base::FilePath::StringType& untranslated_string);
 
+// A helper function used to read the UserDataDir path policy without relying on
+// any policy infrastructure. This is required because this policy is needed
+// much earlier before the PrefService is initialized.
+// The function will fill |user_data_dir| if the corresponding policy is set and

[brettw, 2013/02/19 18:23:45]

Can you rephrase this a bit? I don't know what the "corresponding policy" is. I
gather there is a policy for where user data dirs should be, can you say that
explicitly? (This isn't clear to random people not working on policy stuff).

[pastarmovj, 2013/02/20 10:15:05]

Done.

+// leave it intact if the policy is missing.
+void CheckUserDataDirPolicy(base::FilePath* user_data_dir);
+
 }  // namespace path_parser
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_POLICY_PATH_PARSER_H_