Add flag to decoder class to flip the image vertically since that is required...

remoting/client/decoder_verbatim.cc

 // 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.
 
 #include "remoting/client/decoder_verbatim.h"
 
 #include "remoting/base/protocol_util.h"
 
 namespace remoting {
 
 DecoderVerbatim::DecoderVerbatim()
-    : updated_rects_(NULL) {
+    : updated_rects_(NULL),
+      reverse_rows_(true) {

[dmac, 2010/06/28 20:41:51]

how do we plan to set this per platform?

 }
 
 bool DecoderVerbatim::BeginDecode(scoped_refptr<media::VideoFrame> frame,
                                   UpdatedRects* updated_rects,
                                   Task* partial_decode_done,
                                   Task* decode_done) {
   DCHECK(!partial_decode_done_.get());
   DCHECK(!decode_done_.get());
   DCHECK(!updated_rects_);
 
@@ skipping 21 matching lines @@
     NOTREACHED() << "Pixel format of message doesn't match the video frame. "
                     "Expected vs received = "
                  << frame_->format() << " vs " << pixel_format
                  << " Color space conversion required.";
   }
 
   int bytes_per_pixel = GetBytesPerPixel(pixel_format);
   // Copy the data line by line.
   const int src_stride = bytes_per_pixel * width;
   const char* src = message->update_stream_packet().data().c_str();
+  int src_stride_dir = src_stride;
+  if (reverse_rows_) {
+    // Copy rows from bottom to top to flip the image vertically.
+    src += (height - 1) * src_stride;
+    // Change the direction of the stride to work bottom to top.
+    src_stride_dir *= -1;
+  }
   const int dest_stride = frame_->stride(media::VideoFrame::kRGBPlane);
   uint8* dest = frame_->data(media::VideoFrame::kRGBPlane) +
       dest_stride * y + bytes_per_pixel * x;
   for (int i = 0; i < height; ++i) {
     memcpy(dest, src, src_stride);
     dest += dest_stride;
-    src += src_stride;
+    src += src_stride_dir;
   }
 
   updated_rects_->clear();
   updated_rects_->push_back(gfx::Rect(x, y, width, height));
   partial_decode_done_->Run();
   return true;
 }
 
 void DecoderVerbatim::EndDecode() {
   decode_done_->Run();
 
   partial_decode_done_.reset();
   decode_done_.reset();
   frame_ = NULL;
   updated_rects_ = NULL;
 }
 
 }  // namespace remoting