Propagate DPI information to web-app.

remoting/client/rectangle_update_decoder.cc

 // 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.
 
 #include "remoting/client/rectangle_update_decoder.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/location.h"
@@ skipping 12 matching lines @@
 using remoting::protocol::SessionConfig;
 
 namespace remoting {
 
 RectangleUpdateDecoder::RectangleUpdateDecoder(
     scoped_refptr<base::SingleThreadTaskRunner> task_runner,
     scoped_refptr<FrameConsumerProxy> consumer)
     : task_runner_(task_runner),
       consumer_(consumer),
       source_size_(SkISize::Make(0, 0)),
+      source_dpi_(SkIPoint::Make(0, 0)),
       view_size_(SkISize::Make(0, 0)),
       clip_area_(SkIRect::MakeEmpty()),
       paint_scheduled_(false) {
 }
 
 RectangleUpdateDecoder::~RectangleUpdateDecoder() {
 }
 
 void RectangleUpdateDecoder::Initialize(const SessionConfig& config) {
   // Initialize decoder based on the selected codec.
@@ skipping 14 matching lines @@
   if (!task_runner_->BelongsToCurrentThread()) {
     task_runner_->PostTask(
         FROM_HERE, base::Bind(&RectangleUpdateDecoder::DecodePacket,
                               this, base::Passed(&packet), done));
     return;
   }
 
   base::ScopedClosureRunner done_runner(done);
   bool decoder_needs_reset = false;
 
-  // If the packet includes a screen size, store it.
+  // If the packet includes screen size or DPI information, store them.
   if (packet->format().has_screen_width() &&
       packet->format().has_screen_height()) {
     SkISize source_size = SkISize::Make(packet->format().screen_width(),
                                         packet->format().screen_height());
     if (source_size_ != source_size) {
       source_size_ = source_size;
       decoder_needs_reset = true;
     }
   }
+  if (packet->format().has_x_dpi() && packet->format().has_y_dpi()) {
+    SkIPoint source_dpi(SkIPoint::Make(packet->format().x_dpi(),
+                                       packet->format().y_dpi()));
+    if (source_dpi != source_dpi_) {
+      source_dpi_ = source_dpi;
+      decoder_needs_reset = true;

[Wez, 2012/07/19 23:15:29]

nit: The Decoder doesn't actually need resetting, though; you just need to pass
the new DPI to the FrameConsumer. Restting the Decoder without a resize may
actually confuse it because it'll be in the middle of a stream.

[Jamie, 2012/07/20 00:42:14]

Done.

+    }
+  }
 
   // If we've never seen a screen size, ignore the packet.
   if (source_size_.isZero()) {
     return;
   }
 
   if (decoder_needs_reset) {
     decoder_->Initialize(source_size_);
-    consumer_->SetSourceSize(source_size_);
+    consumer_->SetSourceSize(source_size_, source_dpi_);
   }
 
   if (!decoder_->IsReadyForData()) {
     // TODO(ajwong): This whole thing should move into an invalid state.
     LOG(ERROR) << "Decoder is unable to process data. Dropping packet.";
     return;
   }
 
   if (decoder_->DecodePacket(packet.get()) == Decoder::DECODE_DONE)
     SchedulePaint();
@@ skipping 113 matching lines @@
       } else {
         ++i;
       }
     }
 
     SchedulePaint();
   }
 }
 
 }  // namespace remoting