Revert 51520 - Up the warnings in ipc

 add chromium_code:1 to the GYP file
 Fix some unittest compares of literal 0 to apis that return size_t
 initializer order match declared order
 type_id is a uint32, so fix up comparison warnings by using the right type in the test code.

BUG=none
TEST=everything still builds/works
Review URL: http://codereview.chromium.org/2863034

TBR=thomasvl@google.com
Review URL: http://codereview.chromium.org/2852042

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51523 0039d316-1c4b-4281-b951-d872f2087c98

5 files changed, 13 insertions, 16 deletions

diff --git a/ipc/ipc.gyp b/ipc/ipc.gyp
index 69da83b..5560abe 100644
--- a/ipc/ipc.gyp
+++ b/ipc/ipc.gyp
@@ -3,9 +3,6 @@
 # found in the LICENSE file.
 
 {
-  'variables': {
-    'chromium_code': 1,
-  },
   'target_defaults': {
     'sources/': [
       ['exclude', '/win/'],
diff --git a/ipc/ipc_channel_posix.cc b/ipc/ipc_channel_posix.cc
index 3f910cf..6e8e2642 100644
--- a/ipc/ipc_channel_posix.cc
+++ b/ipc/ipc_channel_posix.cc
@@ -699,7 +699,7 @@ bool Channel::ChannelImpl::ProcessIncomingMessages() {
             // On non-Mac, the Hello message from the client to the server
             // also contains the fd_pipe_, which  will be used for all
             // subsequent file descriptor passing.
-            DCHECK_EQ(m.file_descriptor_set()->size(), 1U);
+            DCHECK_EQ(m.file_descriptor_set()->size(), 1);
             base::FileDescriptor descriptor;
             if (!m.ReadFileDescriptor(&iter, &descriptor)) {
               NOTREACHED();
@@ -777,7 +777,7 @@ bool Channel::ChannelImpl::ProcessOutgoingMessages() {
         NOTREACHED();
       }
       msg = hello.get();
-      DCHECK_EQ(msg->file_descriptor_set()->size(), 1U);
+      DCHECK_EQ(msg->file_descriptor_set()->size(), 1);
     }
 #endif
 
@@ -853,7 +853,7 @@ bool Channel::ChannelImpl::ProcessOutgoingMessages() {
       if (mode_ != MODE_SERVER && !uses_fifo_ &&
           msg->routing_id() == MSG_ROUTING_NONE &&
           msg->type() == HELLO_MESSAGE_TYPE) {
-        DCHECK_EQ(msg->file_descriptor_set()->size(), 1U);
+        DCHECK_EQ(msg->file_descriptor_set()->size(), 1);
       }
       if (!uses_fifo_ && !msgh.msg_controllen) {
         bytes_written = HANDLE_EINTR(write(pipe_, out_bytes, amt_to_write));
diff --git a/ipc/ipc_fuzzing_tests.cc b/ipc/ipc_fuzzing_tests.cc
index aaccd70..951ccbb 100644
--- a/ipc/ipc_fuzzing_tests.cc
+++ b/ipc/ipc_fuzzing_tests.cc
@@ -186,7 +186,7 @@ class FuzzerServerListener : public SimpleListener {
     Cleanup();
   }
 
-  bool RoundtripAckReply(int routing, uint32 type_id, int reply) {
+  bool RoundtripAckReply(int routing, int type_id, int reply) {
     IPC::Message* message = new IPC::Message(routing, type_id,
                                              IPC::Message::PRIORITY_NORMAL);
     message->WriteInt(reply + 1);
@@ -201,7 +201,7 @@ class FuzzerServerListener : public SimpleListener {
       MessageLoop::current()->Quit();
   }
 
-  void ReplyMsgNotHandled(uint32 type_id) {
+  void ReplyMsgNotHandled(int type_id) {
     RoundtripAckReply(FUZZER_ROUTING_ID, CLIENT_UNHANDLED_IPC, type_id);
     Cleanup();
   }
@@ -227,7 +227,7 @@ class FuzzerClientListener : public SimpleListener {
     MessageLoop::current()->Quit();
   }
 
-  bool ExpectMessage(int value, uint32 type_id) {
+  bool ExpectMessage(int value, int type_id) {
     if (!MsgHandlerInternal(type_id))
       return false;
     int msg_value1 = 0;
@@ -247,12 +247,12 @@ class FuzzerClientListener : public SimpleListener {
     return true;
   }
 
-  bool ExpectMsgNotHandled(uint32 type_id) {
+  bool ExpectMsgNotHandled(int type_id) {
     return ExpectMessage(type_id, CLIENT_UNHANDLED_IPC);
   }
 
  private:
-  bool MsgHandlerInternal(uint32 type_id) {
+  bool MsgHandlerInternal(int type_id) {
     MessageLoop::current()->Run();
     if (NULL == last_msg_)
       return false;
diff --git a/ipc/ipc_sync_channel_unittest.cc b/ipc/ipc_sync_channel_unittest.cc
index ae9ae66..2566f9b 100644
--- a/ipc/ipc_sync_channel_unittest.cc
+++ b/ipc/ipc_sync_channel_unittest.cc
@@ -727,8 +727,8 @@ class QueuedReplyClient : public Worker {
                     const std::string& expected_text,
                     bool pump_during_send)
       : Worker(channel_name, Channel::MODE_CLIENT),
-        pump_during_send_(pump_during_send),
-        expected_text_(expected_text) {
+        expected_text_(expected_text),
+        pump_during_send_(pump_during_send) {
     Worker::OverrideThread(listener_thread);
   }
 
diff --git a/ipc/sync_socket_unittest.cc b/ipc/sync_socket_unittest.cc
index a17de68..e07fdf9 100644
--- a/ipc/sync_socket_unittest.cc
+++ b/ipc/sync_socket_unittest.cc
@@ -195,7 +195,7 @@ class SyncSocketClientListener : public IPC::Channel::Listener {
     socket_->Receive(static_cast<void*>(buf), kHelloStringLength);
     EXPECT_EQ(strcmp(str.c_str(), buf), 0);
     // After receiving from the socket there should be no bytes left.
-    EXPECT_EQ(0U, socket_->Peek());
+    EXPECT_EQ(0, socket_->Peek());
     IPC::Message* msg = new MsgClassShutdown();
     EXPECT_TRUE(chan_->Send(msg));
     MessageLoop::current()->Quit();
@@ -220,8 +220,8 @@ TEST_F(SyncSocketTest, SanityTest) {
   base::SyncSocket* pair[2];
   base::SyncSocket::CreatePair(pair);
   // Immediately after creation there should be no pending bytes.
-  EXPECT_EQ(0U, pair[0]->Peek());
-  EXPECT_EQ(0U, pair[1]->Peek());
+  EXPECT_EQ(0, pair[0]->Peek());
+  EXPECT_EQ(0, pair[1]->Peek());
   base::SyncSocket::Handle target_handle;
   // Connect the channel and listener.
   ASSERT_TRUE(chan.Connect());