Use DCHECK_GE() where possible in base/. Patch by Kushal Pisavadia (see http://codereview.chromium.org/6688056/ ), r=me.

BUG=58409
TEST=none

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

1 files changed, 2 insertions, 11 deletions

diff --git a/base/mach_ipc_mac.h b/base/mach_ipc_mac.h
index d506a00..4e80f62 100644
--- a/base/mach_ipc_mac.h
+++ b/base/mach_ipc_mac.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
 
@@ -140,6 +140,7 @@ class MachMsgPortDescriptor : public mach_msg_port_descriptor_t {
 //
 class MachMessage {
  public:
+  static const size_t kEmptyMessageSize;
 
   virtual ~MachMessage();
 
@@ -209,7 +210,6 @@ class MachMessage {
   // of the Mach header.
   size_t MaxSize() const { return storage_length_bytes_; }
 
- protected:
   mach_msg_header_t *Head() { return &(storage_->head); }
 
  private:
@@ -220,15 +220,6 @@ class MachMessage {
     u_int8_t           padding[1024];
   };
 
- // kEmptyMessageSize needs to have the definition of MachMessageData before
- // it.
- public:
-   // The size of an empty message with no data.
-  static const size_t kEmptyMessageSize = sizeof(mach_msg_header_t) +
-                                          sizeof(mach_msg_body_t) +
-                                          sizeof(MessageDataPacket);
-
- private:
   MachMessageData *storage_;
   size_t storage_length_bytes_;
   bool own_storage_;  // Is storage owned by this object?