From bce3e02f453dfbc7fcdf101e282582c794f52ccd Mon Sep 17 00:00:00 2001 From: Sergey Ulanov Date: Mon, 7 May 2018 18:13:06 +0000 Subject: [PATCH] Enable ProcessUtilTest.LaunchWithHandleTransfer on Fuchsia VM performance was improved significantly with TSC clock, so the test is not expected to timeout anymore. Bug: 793412 Change-Id: I8a4cd1ddd35a364e490252d02ccd59946a7600fa Reviewed-on: https://chromium-review.googlesource.com/1045612 Reviewed-by: Scott Graham Reviewed-by: Daniel Cheng Commit-Queue: Sergey Ulanov Cr-Commit-Position: refs/heads/master@{#556499} --- base/process/process_util_unittest.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/base/process/process_util_unittest.cc b/base/process/process_util_unittest.cc index 88890c8b71b6c9..c9b3e98bacd7e8 100644 --- a/base/process/process_util_unittest.cc +++ b/base/process/process_util_unittest.cc @@ -875,9 +875,8 @@ TEST_F(ProcessUtilTest, FDRemappingIncludesStdio) { EXPECT_EQ(0, exit_code); } -// TODO(https://crbug.com/793412): Disable on Debug/component builds due to -// process launch taking too long and triggering timeouts. -#if defined(OS_FUCHSIA) && defined(NDEBUG) +#if defined(OS_FUCHSIA) + const uint16_t kStartupHandleId = 43; MULTIPROCESS_TEST_MAIN(ProcessUtilsVerifyHandle) { zx_handle_t handle = @@ -932,7 +931,8 @@ TEST_F(ProcessUtilTest, LaunchWithHandleTransfer) { &exit_code)); EXPECT_EQ(0, exit_code); } -#endif // defined(OS_FUCHSIA) && defined(NDEBUG) + +#endif // defined(OS_FUCHSIA) namespace {