Made DesktopEnvironment responsible for creation of the disconnect window.

remoting/host/desktop_process_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/desktop_process.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "base/memory/ref_counted.h"
@@ skipping 269 matching lines @@
   RunDesktopProcess();
 }
 
 void DesktopProcessTest::SendCrashRequest() {
   tracked_objects::Location location = FROM_HERE;
   daemon_channel_->Send(new ChromotingDaemonMsg_Crash(
       location.function_name(), location.file_name(), location.line_number()));
 }
 
 void DesktopProcessTest::SendStartSessionAgent() {
-  // TODO(alexeypa): Fix DesktopProcess to use the desktop environment
-  // to create the disconnect window instead of directly calling
-  // DisconnectWindow::Create(). This will take care of "Uninteresting mock
-  // function call" warnings printed when DisconnectWindow::Show() and
-  // DisconnectWindow::Hide() are called.
   network_channel_->Send(new ChromotingNetworkDesktopMsg_StartSessionAgent(
       "user@domain/rest-of-jid", ScreenResolution()));
 }
 
 // Launches the desktop process and waits when it connects back.
 TEST_F(DesktopProcessTest, Basic) {
   InSequence s;
   EXPECT_CALL(daemon_listener_, OnChannelConnected(_));
   EXPECT_CALL(daemon_listener_, OnDesktopAttached(_))
       .WillOnce(DoAll(
@@ skipping 37 matching lines @@
 }
 
 // Run the desktop process and ask it to crash.
 TEST_F(DesktopProcessTest, DeathTest) {
   testing::GTEST_FLAG(death_test_style) = "threadsafe";
 
   EXPECT_DEATH(RunDeathTest(), "");
 }
 
 }  // namespace remoting