Changing UpdateStreamPacket protobuf definition for chromoting

remoting/host/client_connection.h

 // Copyright (c) 2010 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.
 
 #ifndef REMOTING_HOST_CLIENT_CONNECTION_H_
 #define REMOTING_HOST_CLIENT_CONNECTION_H_
 
 #include <deque>
 #include <vector>
 
@@ skipping 44 matching lines @@
   // Constructs a ClientConnection object. |message_loop| is the message loop
   // that this object runs on. A viewer object receives events and messages from
   // a libjingle channel, these events are delegated to |handler|.
   // It is guranteed that |handler| is called only on the |message_loop|.
   ClientConnection(MessageLoop* message_loop,
          ProtocolDecoder* decoder,
          EventHandler* handler);
 
   virtual ~ClientConnection();
 
+  // Creates a DataBuffer object that wraps around HostMessage. The DataBuffer
+  // object will be responsible for serializing and framing the message.
+  // DataBuffer will also own |msg| after this call.
+  static scoped_refptr<media::DataBuffer> CreateWireFormatDataBuffer(
+      const HostMessage* msg);
+
   virtual void set_jingle_channel(JingleChannel* channel) {
     channel_ = channel;
   }
 
   // Returns the channel in use.
   virtual JingleChannel* jingle_channel() { return channel_; }
 
   // Send information to the client for initialization.
   virtual void SendInitClientMessage(int width, int height);
 
   // Notifies the viewer the start of an update stream.
   virtual void SendBeginUpdateStreamMessage();
 
-  // Send encoded update stream data to the viewer. The viewer
-  // should not take ownership of the data.
+  // Send encoded update stream data to the viewer.
+  //
+  // |data| is the actual bytes in wire format. That means it is fully framed
+  // and serialized from a HostMessage. This is a special case only for
+  // UpdateStreamPacket to reduce the amount of memory copies.
+  //
+  // |data| should be created by calling to
+  // CreateWireFormatDataBuffer(HostMessage).
   virtual void SendUpdateStreamPacketMessage(
-      const UpdateStreamPacketHeader* header,
       scoped_refptr<media::DataBuffer> data);
 
   // Notifies the viewer the update stream has ended.
   virtual void SendEndUpdateStreamMessage();
 
   // Gets the number of update stream messages not yet transmitted.
   // Note that the value returned is an estimate using average size of the
   // most recent update streams.
   // TODO(hclam): Report this number accurately.
   virtual int GetPendingUpdateStreamMessages();
@@ skipping 43 matching lines @@
 
   // Event handler for handling events sent from this object.
   EventHandler* handler_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientConnection);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_CLIENT_CONNECTION_H_