clang/win: Let some chromium_code targets build with -Wextra.

To make warning flags between Windows and non-Windows clang consistent,
I'd like to build chromium_code targets with -Wextra enabled.  This fixes
-Wextra warnings in about 14k of 31k build steps.

Most of the warnings were about signed/unsigned mismatches; some were about
"const"s that had no effect.

(This CL does not contain the change to actually turn on -Wextra, only
cleanups.)

BUG=567877
TBR=content/common/sandbox

Review URL: https://codereview.chromium.org/1507413003

Cr-Commit-Position: refs/heads/master@{#364232}

1 files changed, 5 insertions, 5 deletions

diff --git a/win8/delegate_execute/delegate_execute_util_unittest.cc b/win8/delegate_execute/delegate_execute_util_unittest.cc
index 63a5075..47916fd 100644
--- a/win8/delegate_execute/delegate_execute_util_unittest.cc
+++ b/win8/delegate_execute/delegate_execute_util_unittest.cc
@@ -47,16 +47,16 @@ TEST(DelegateExecuteUtil, MakeChromeCommandLineTest) {
       this_exe,
       delegate_execute::CommandLineFromParameters(NULL),
       base::string16());
-  EXPECT_EQ(1, cl.argv().size());
+  EXPECT_EQ(1u, cl.argv().size());
   EXPECT_EQ(this_exe.value(), cl.GetProgram().value());
 
   // Empty params with arg contains the arg.
   cl = delegate_execute::MakeChromeCommandLine(
       this_exe, delegate_execute::CommandLineFromParameters(NULL),
       base::string16(kSomeArgument));
-  EXPECT_EQ(2, cl.argv().size());
+  EXPECT_EQ(2u, cl.argv().size());
   EXPECT_EQ(this_exe.value(), cl.GetProgram().value());
-  EXPECT_EQ(1, cl.GetArgs().size());
+  EXPECT_EQ(1u, cl.GetArgs().size());
   EXPECT_EQ(base::string16(kSomeArgument), cl.GetArgs()[0]);
 
   // Params with switchs and args plus arg contains the arg.
@@ -66,11 +66,11 @@ TEST(DelegateExecuteUtil, MakeChromeCommandLineTest) {
                              base::ASCIIToUTF16(kSomeSwitch).c_str(),
                              kOtherArgument).c_str()),
       base::string16(kSomeArgument));
-  EXPECT_EQ(5, cl.argv().size());
+  EXPECT_EQ(5u, cl.argv().size());
   EXPECT_EQ(this_exe.value(), cl.GetProgram().value());
   EXPECT_TRUE(cl.HasSwitch(kSomeSwitch));
   base::CommandLine::StringVector args(cl.GetArgs());
-  EXPECT_EQ(2, args.size());
+  EXPECT_EQ(2u, args.size());
   EXPECT_NE(
       args.end(),
       std::find(args.begin(), args.end(), base::string16(kOtherArgument)));