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

remoting/host/desktop_session_proxy.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/desktop_session_proxy.h"
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/platform_file.h"
 #include "base/process_util.h"
 #include "base/single_thread_task_runner.h"
 #include "ipc/ipc_channel_proxy.h"
 #include "ipc/ipc_message_macros.h"
 #include "media/video/capture/screen/screen_capture_data.h"
 #include "remoting/host/chromoting_messages.h"
 #include "remoting/host/client_session.h"
 #include "remoting/host/desktop_session_connector.h"
 #include "remoting/host/ipc_audio_capturer.h"
 #include "remoting/host/ipc_event_executor.h"
 #include "remoting/host/ipc_session_controller.h"
 #include "remoting/host/ipc_video_frame_capturer.h"
+#include "remoting/host/screen_resolution.h"
 #include "remoting/host/session_controller.h"
 #include "remoting/proto/audio.pb.h"
 #include "remoting/proto/control.pb.h"
 #include "remoting/proto/event.pb.h"
 
 #if defined(OS_WIN)
 #include "base/win/scoped_handle.h"
 #endif  // defined(OS_WIN)
 
 namespace remoting {
@@ skipping 240 matching lines @@
       new ChromotingNetworkDesktopMsg_InjectMouseEvent(serialized_event));
 }
 
 void DesktopSessionProxy::StartEventExecutor(
     scoped_ptr<protocol::ClipboardStub> client_clipboard) {
   DCHECK(caller_task_runner_->BelongsToCurrentThread());
 
   client_clipboard_ = client_clipboard.Pass();
 }
 
+void DesktopSessionProxy::SetScreenResolution(

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

For a CL that's primarily a global rename, I was surprised not to see a
corresponding OnClientResolutionChanged method removed from the LHS. Was this
message not being proxied previously?

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

This leaked from the follow up CL. I moved this code to that CL.

+    const ScreenResolution& resolution) {
+  DCHECK(caller_task_runner_->BelongsToCurrentThread());
+
+  // Pass the client's resolution to the daemon.
+  if (desktop_session_connector_)
+    desktop_session_connector_->SetScreenResolution(this, resolution);
+
+  // Pass the client's resolution to the DesktopSessionAgent instance.
+  SendToDesktop(
+      new ChromotingNetworkDesktopMsg_SetScreenResolution(resolution));
+}
+
 void DesktopSessionProxy::ConnectToDesktopSession(
     base::WeakPtr<DesktopSessionConnector> desktop_session_connector,
     bool virtual_terminal) {
   DCHECK(caller_task_runner_->BelongsToCurrentThread());
   DCHECK(!desktop_session_connector_);
   DCHECK(desktop_session_connector);
 
   desktop_session_connector_ = desktop_session_connector;
   desktop_session_connector_->ConnectTerminal(
-      this, DesktopSessionParams(), virtual_terminal);
+      this, ScreenResolution(), virtual_terminal);
 }
 
 DesktopSessionProxy::~DesktopSessionProxy() {
   DCHECK(caller_task_runner_->BelongsToCurrentThread());
 
   if (desktop_session_connector_)
     desktop_session_connector_->DisconnectTerminal(this);
 
   if (desktop_process_ != base::kNullProcessHandle) {
     base::CloseProcessHandle(desktop_process_);
@@ skipping 162 matching lines @@
 }
 
 // static
 void DesktopSessionProxyTraits::Destruct(
     const DesktopSessionProxy* desktop_session_proxy) {
   desktop_session_proxy->caller_task_runner_->DeleteSoon(FROM_HERE,
                                                          desktop_session_proxy);
 }
 
 }  // namespace remoting