Added the desktop shape fields to VideoPacket.

remoting/codec/video_decoder.h

 // 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.
 
 #ifndef REMOTING_CODEC_VIDEO_DECODER_H_
 #define REMOTING_CODEC_VIDEO_DECODER_H_
 
 #include "base/basictypes.h"
 #include "remoting/proto/video.pb.h"
 #include "third_party/skia/include/core/SkRect.h"
@@ skipping 50 matching lines @@
   // and the buffer must be large enough to hold |clip_area| RGBA32 pixels.
   // |image_stride| gives the output buffer's stride in pixels.
   //
   // On return, |output_region| contains the updated area, in |view_size|
   // coordinates.
   virtual void RenderFrame(const SkISize& view_size,
                            const SkIRect& clip_area,
                            uint8* image_buffer,
                            int image_stride,
                            SkRegion* output_region) = 0;
+
+  // Returns the "shape", if any, of the most recently rendered frame.
+  // The shape is returned in source dimensions.
+  virtual const SkRegion* GetImageShape() { return NULL; }

[Jamie, 2013/06/20 21:15:57]

Everything else here is pure virtual. Does this need a default implementation?
If it's convenient to leave it like this for now, can you add a TODO to make it
pure virtual once all subclasses inplemented it?

[Sergey Ulanov, 2013/06/20 21:27:21]

Can you use webrtc::DesktopRegion here instead of SkRegion? mainly for
consistency between client and host code. I'm planning to switch all client code
to using webrtc::DesktopRegion.

[alexeypa (please no reviews), 2013/06/20 22:05:11]

Done.

[alexeypa (please no reviews), 2013/06/20 22:05:11]

I will not be able to do it today. We would really like to submit it before the
branch point.

 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CODEC_VIDEO_DECODER_H_