Rename ClientDimensions to ClientResolution and add pixel-size and DPI fields.

remoting/host/resizing_host_observer_unittest.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/host/resizing_host_observer.h"
 #include "remoting/host/desktop_resizer.h"
 
 #include <list>
 
 #include "base/compiler_specific.h"
@@ skipping 56 matching lines @@
  public:
   void SetDesktopResizer(FakeDesktopResizer* desktop_resizer) {
     CHECK(!desktop_resizer_.get()) << "Call SetDeskopResizer once per test";
     resizing_host_observer_.reset(new ResizingHostObserver(desktop_resizer,
                                                            NULL));
     desktop_resizer_.reset(desktop_resizer);
     resizing_host_observer_->OnClientAuthenticated("");
   }
 
   SkISize GetBestSize(const SkISize& client_size) {
-    resizing_host_observer_->OnClientDimensionsChanged("", client_size);
+    resizing_host_observer_->OnClientResolutionChanged(
+        "", client_size, SkIPoint());

[Jamie, 2013/02/11 23:25:58]

Can you add a unit test to make sure that the DPI information is propagated
correctly?

[Wez, 2013/02/12 01:55:28]

You mean when hooking up the host-side DPI-handling?  SGTM.

     return desktop_resizer_->GetCurrentSize();
   }
 
   void VerifySizes(const SkISize* client_sizes, const SkISize* expected_sizes,
                    int number_of_sizes) {
     for (int i = 0; i < number_of_sizes; ++i) {
       SkISize best_size = GetBestSize(client_sizes[i]);
       EXPECT_EQ(expected_sizes[i], best_size)
           << "Input size = " << client_sizes[i];
     }
@@ skipping 114 matching lines @@
       new FakeDesktopResizer(SkISize::Make(640, 480), false,
                              supported_sizes, arraysize(supported_sizes));
   SetDesktopResizer(desktop_resizer);
   SkISize client_sizes[] = { { 640, 640 }, { 1024, 768 }, { 640, 480 } };
   SkISize expected_sizes[] = { { 640, 480 }, { 640, 480 }, { 640, 480 } };
   VerifySizes(client_sizes, expected_sizes, arraysize(client_sizes));
   EXPECT_EQ(desktop_resizer->set_size_call_count(), 0);
 }
 
 }  // namespace remoting