Replace RectVectors with SkRegions in Decoder.

remoting/client/frame_consumer.h

 // 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.
 
 #ifndef REMOTING_CLIENT_FRAME_CONSUMER_H_
 #define REMOTING_CLIENT_FRAME_CONSUMER_H_
 
 #include "remoting/base/decoder.h"  // For UpdatedRects
 
 namespace remoting {
@@ skipping 23 matching lines @@
   // corresponding call ReleaseFrame(scoped_refptr<VideoFrame>* frame_out.
   virtual void AllocateFrame(media::VideoFrame::Format format,
                              const SkISize& size,
                              scoped_refptr<media::VideoFrame>* frame_out,
                              const base::Closure& done) = 0;
 
   virtual void ReleaseFrame(media::VideoFrame* frame) = 0;
 
   // OnPartialFrameOutput() is called every time at least one rectangle of
   // output is produced.  The |frame| is guaranteed to have valid data for every
   // region included in the |rects| list.

[Sergey Ulanov, 2012/01/23 19:59:54]

s/rects/region/

[Wez, 2012/01/23 21:37:23]

Done.

   //
-  // Both |frame| and |rects| are guaranteed to be valid until the |done|
+  // Both |frame| and |region| are guaranteed to be valid until the |done|
   // callback is invoked.
   virtual void OnPartialFrameOutput(media::VideoFrame* frame,
-                                    RectVector* rects,
+                                    SkRegion* region,

[Sergey Ulanov, 2012/01/23 19:59:54]

should this be a const ref?

[Wez, 2012/01/23 21:37:23]

Probably.  However, I think the FrameConsumer API is due some other changes, so
I'll put those in a follow-up CL.

                                     const base::Closure& done) = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(FrameConsumer);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_FRAME_CONSUMER_H_