Replace RectVectors with SkRegions in Decoder.

remoting/client/frame_consumer.h

Index: remoting/client/frame_consumer.h
diff --git a/remoting/client/frame_consumer.h b/remoting/client/frame_consumer.h
index 65b0c641b99fd77d20c9aa55266107f740df052b..4f0c6e74f2e76a3a1bd7e016792066493ec45571 100644
--- a/remoting/client/frame_consumer.h
+++ b/remoting/client/frame_consumer.h
@@ -43,10 +43,10 @@ class FrameConsumer {
   // 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: