Add DPI information to video packets.

remoting/base/capture_data.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_BASE_CAPTURE_DATA_H_
 #define REMOTING_BASE_CAPTURE_DATA_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 41 matching lines @@
   void set_capture_time_ms(int capture_time_ms) {
     capture_time_ms_ = capture_time_ms;
   }
 
   int64 client_sequence_number() const { return client_sequence_number_; }
 
   void set_client_sequence_number(int64 client_sequence_number) {
     client_sequence_number_ = client_sequence_number;
   }
 
+  SkIPoint dpi() const { return dpi_; }

[Wez, 2012/07/13 21:49:02]

nit: Should this be SkISize?

[Jamie, 2012/07/13 22:17:13]

I had that at first, but the code ends up referring to the width and height of
the DPI, which doesn't seem right. SkIPoint isn't a perfect fit, but it seems
better than SkISize.

[Wez, 2012/07/14 00:55:11]

OK, SGTM

+
+  void set_dpi(const SkIPoint& dpi) { dpi_ = dpi; }
+
  private:
   friend class base::RefCountedThreadSafe<CaptureData>;
   virtual ~CaptureData();
 
   const DataPlanes data_planes_;
   SkRegion dirty_region_;
   SkISize size_;
   media::VideoFrame::Format pixel_format_;
 
   // Time spent in capture. Unit is in milliseconds.
   int capture_time_ms_;
 
   // Sequence number supplied by client for performance tracking.
   int64 client_sequence_number_;
+
+  // DPI for this frame.
+  SkIPoint dpi_;
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_BASE_CAPTURE_DATA_H_