From 035459168ac247f089c25d5f5c3c1f36814bb1f5 Mon Sep 17 00:00:00 2001 From: Hans Wennborg Date: Fri, 25 Mar 2016 09:23:26 -0700 Subject: process_policy_test.cc: Fix mixed-sign comparison warning. Clang builds were failing like this: In file included from ..\..\sandbox\win\src\process_policy_test.cc:19: ..\..\testing/gtest/include/gtest/gtest.h(1392,16): error: comparison of integers of different signs: 'const unsigned long' and 'const int' [-Werror,-Wsign-compare] if (expected == actual) { ~~~~~~~~ ^ ~~~~~~ ..\..\testing/gtest/include/gtest/gtest.h(1422,12): note: in instantiation of function template specialization 'testing::internal::CmpHelperEQ' requested here return CmpHelperEQ(expected_expression, actual_expression, expected, ^ ..\..\sandbox\win\src\process_policy_test.cc(419,3): note: in instantiation of function template specialization 'testing::internal::EqHelper::Compare' requested here EXPECT_EQ(STATUS_BREAKPOINT, runner.RunTest(L"Process_Crash")); ^ ..\..\testing/gtest/include/gtest/gtest.h(1928,67): note: expanded from macro 'EXPECT_EQ' EqHelper::Compare, \ ^ BUG=82385 TBR=wfh Review URL: https://codereview.chromium.org/1833863004 . Cr-Commit-Position: refs/heads/master@{#383286} --- sandbox/win/src/process_policy_test.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'sandbox') diff --git a/sandbox/win/src/process_policy_test.cc b/sandbox/win/src/process_policy_test.cc index 28d5ac2..779cd15 100644 --- a/sandbox/win/src/process_policy_test.cc +++ b/sandbox/win/src/process_policy_test.cc @@ -416,7 +416,8 @@ TEST(ProcessPolicyTest, CreateProcessAW) { // Tests that the broker correctly handles a process crashing within the job. TEST(ProcessPolicyTest, CreateProcessCrashy) { TestRunner runner; - EXPECT_EQ(STATUS_BREAKPOINT, runner.RunTest(L"Process_Crash")); + EXPECT_EQ(static_cast(STATUS_BREAKPOINT), + runner.RunTest(L"Process_Crash")); } TEST(ProcessPolicyTest, OpenToken) { -- cgit v1.1