Index: chrome/browser/policy/policy_path_parser_unittest.cc |
diff --git a/chrome/browser/policy/policy_path_parser_unittest.cc b/chrome/browser/policy/policy_path_parser_unittest.cc |
index 9ae45973d94d6814aafcd783537f94fae4dfe171..b6437a4d3bed91f3fbe5803a756448f50e986ab0 100644 |
--- a/chrome/browser/policy/policy_path_parser_unittest.cc |
+++ b/chrome/browser/policy/policy_path_parser_unittest.cc |
@@ -23,7 +23,14 @@ class PolicyPathParserTests : public testing::Test { |
// TODO(pastarmovj): Modify test or change the logic after investigating why it |
// is flaky. Please only redisable after it has had a chance to run for a few |
// times. See bug http://crbug.com/327520 for context. |
-TEST_F(PolicyPathParserTests, AllPlatformVariables) { |
+// TODO(pastarmovj): Disabling on Mac OS X as the test is flaky. See |
+// crbug.com/458590 for details. |
+#if defined(OS_MACOSX) |
+#define MAYBE_AllPlatformVariables DISABLED_AllPlatformVariables |
+#else |
+#define MAYBE_AllPlatformVariables AllPlatformVariables |
+#endif |
+TEST_F(PolicyPathParserTests, MAYBE_AllPlatformVariables) { |
// No vars whatsoever no substitution should occur. |
base::FilePath::StringType no_vars(FILE_PATH_LITERAL("//$C/shares")); |
base::FilePath::StringType no_vars_result = |