Mojo: (Theoretically) implement the read side of platform handle passing.

mojo/system/channel.cc

 // Copyright 2013 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 "mojo/system/channel.h"
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 255 matching lines @@
 size_t Channel::GetSerializedPlatformHandleSize() const {
   return raw_channel_->GetSerializedPlatformHandleSize();
 }
 
 Channel::~Channel() {
   // The channel should have been shut down first.
   DCHECK(!is_running_no_lock());
 }
 
 void Channel::OnReadMessage(const MessageInTransit::View& message_view) {
-  // Note: |ValidateReadMessage()| will call |HandleRemoteError()| if necessary.
-  if (!ValidateReadMessage(message_view))
-    return;
-
   switch (message_view.type()) {
     case MessageInTransit::kTypeMessagePipeEndpoint:
     case MessageInTransit::kTypeMessagePipe:
       OnReadMessageForDownstream(message_view);
       break;
     case MessageInTransit::kTypeChannel:
       OnReadMessageForChannel(message_view);
       break;
     default:
       HandleRemoteError(base::StringPrintf(
           "Received message of invalid type %u",
           static_cast<unsigned>(message_view.type())));
       break;
   }
 }
 
 void Channel::OnFatalError(FatalError fatal_error) {
   LOG(ERROR) << "RawChannel fatal error (type " << fatal_error << ")";
   Shutdown();
 }
 
-bool Channel::ValidateReadMessage(const MessageInTransit::View& message_view) {
-  const char* error_message = NULL;
-  if (!message_view.IsValid(&error_message)) {
-    DCHECK(error_message);
-    HandleRemoteError(error_message);
-    return false;
-  }
-
-  return true;
-}
-
 void Channel::OnReadMessageForDownstream(
     const MessageInTransit::View& message_view) {
   DCHECK(message_view.type() == MessageInTransit::kTypeMessagePipeEndpoint ||
          message_view.type() == MessageInTransit::kTypeMessagePipe);
 
   MessageInTransit::EndpointId local_id = message_view.destination_id();
   if (local_id == MessageInTransit::kInvalidEndpointId) {
     HandleRemoteError("Received message with no destination ID");
     return;
   }
@@ skipping 164 matching lines @@
   // TODO(vtl): Is this how we really want to handle this?
   // Sometimes we'll want to propagate the error back to the message pipe
   // (endpoint), and notify it that the remote is (effectively) closed.
   // Sometimes we'll want to kill the channel (and notify all the endpoints that
   // their remotes are dead.
   LOG(WARNING) << error_message;
 }
 
 }  // namespace system
 }  // namespace mojo