Reworked the plumbing required to pass the client resolution to the desktop resizer.

remoting/host/resizing_host_observer.cc

Index: remoting/host/resizing_host_observer.cc
diff --git a/remoting/host/resizing_host_observer.cc b/remoting/host/resizing_host_observer.cc
index 7491eea6f759651f3b1565e048f98e5a0736b733..9c0316575b98ce1270aa318f345e4219c2f38d31 100644
--- a/remoting/host/resizing_host_observer.cc
+++ b/remoting/host/resizing_host_observer.cc
@@ -8,6 +8,7 @@
 
 #include "base/logging.h"
 #include "remoting/host/desktop_resizer.h"
+#include "remoting/host/screen_resolution.h"
 
 namespace {
 
@@ -108,24 +109,22 @@ ResizingHostObserver::~ResizingHostObserver() {
     desktop_resizer_->RestoreSize(original_size_);
 }
 
-void ResizingHostObserver::OnClientResolutionChanged(
-    const SkIPoint& client_dpi,
-    const SkISize& client_size) {
-  if (client_size.isEmpty()) {
+void ResizingHostObserver::SetScreenResolution(
+    const ScreenResolution& resolution) {
+  if (resolution.IsNull())

[Jamie, 2013/03/15 18:33:24]

Why not IsValid() here?

[alexeypa (please no reviews), 2013/03/15 20:30:41]

IsValid() checks for negative sizes only. I.e. it checks whether the structure
completely bogus.

InNull() additionally checks if the dimensions is an empty rectangle. 

This method expects the dimensions to be not empty.

     return;
-  }
 
   // If the implementation returns any sizes, pick the best one according to
   // the algorithm described in CandidateSize::IsBetterThen.
   std::list<SkISize> sizes =
-      desktop_resizer_->GetSupportedSizes(client_size);
+      desktop_resizer_->GetSupportedSizes(resolution.dimensions_);
   if (sizes.empty()) {
     return;
   }
-  CandidateSize best_size(sizes.front(), client_size);
+  CandidateSize best_size(sizes.front(), resolution.dimensions_);
   for (std::list<SkISize>::const_iterator i = ++sizes.begin();
        i != sizes.end(); ++i) {
-    CandidateSize candidate_size(*i, client_size);
+    CandidateSize candidate_size(*i, resolution.dimensions_);
     if (candidate_size.IsBetterThan(best_size)) {
       best_size = candidate_size;
     }