Adding "static_assert" in lieu of "COMPILE_ASSERT" in ipc module

ipc/ipc_channel_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ipc/ipc_channel_win.h"
 
 #include <windows.h>
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
@@ skipping 11 matching lines @@
 #include "ipc/ipc_message_utils.h"
 
 namespace IPC {
 
 ChannelWin::State::State(ChannelWin* channel) : is_pending(false) {
   memset(&context.overlapped, 0, sizeof(context.overlapped));
   context.handler = channel;
 }
 
 ChannelWin::State::~State() {
-  COMPILE_ASSERT(!offsetof(ChannelWin::State, context),
-                 starts_with_io_context);
+  static_assert(offsetof(ChannelWin::State, context) == 0,
+                "should starts with io context");

[dmichael (off chromium), 2015/01/20 16:44:10]

"should starts" is not proper grammar, and it's not all that readable. How
about...
"ChannelWin::State should have context as its first data member."

[anujsharma, 2015/01/21 03:11:51]

Done.

 }
 
 ChannelWin::ChannelWin(const IPC::ChannelHandle &channel_handle,
                        Mode mode, Listener* listener)
     : ChannelReader(listener),
       input_state_(this),
       output_state_(this),
       peer_pid_(base::kNullProcessId),
       waiting_connect_(mode & MODE_SERVER_FLAG),
       processing_incoming_(false),
@@ skipping 455 matching lines @@
   int secret;
   do {  // Guarantee we get a non-zero value.
     secret = base::RandInt(0, std::numeric_limits<int>::max());
   } while (secret == 0);
 
   id.append(GenerateUniqueRandomChannelID());
   return id.append(base::StringPrintf("\\%d", secret));
 }
 
 }  // namespace IPC