Webapp share dialog is not closed when client end of the connection is closed

remoting/host/it2me/it2me_confirmation_dialog_proxy_unittest.cc

 // Copyright 2014 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/it2me/it2me_confirmation_dialog_proxy.h"
 
 #include <memory>
 
 #include "base/bind.h"
+#include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/run_loop.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gmock_mutant.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using ::testing::InvokeWithoutArgs;
 using ::testing::CreateFunctor;
@@ skipping 87 matching lines @@
 
   StubIt2MeConfirmationDialog* dialog() {
     return dialog_;
   }
 
  private:
   base::MessageLoop message_loop_;
   base::RunLoop run_loop_;
   base::Thread dialog_thread_;
 
-  // |dialog_| is owned by |dialog_proxy_| but we keep an alias for test
-  // purposes.
-  StubIt2MeConfirmationDialog* dialog_;
+  // |dialog_| is owned by |dialog_proxy_| but we keep an alias for testing.
+  StubIt2MeConfirmationDialog* dialog_ = nullptr;
   std::unique_ptr<It2MeConfirmationDialogProxy> dialog_proxy_;
 };
 
 It2MeConfirmationDialogProxyTest::It2MeConfirmationDialogProxyTest()
     : dialog_thread_("test dialog thread") {
   dialog_thread_.Start();
 
-  dialog_ = new StubIt2MeConfirmationDialog(dialog_task_runner());
-  dialog_proxy_.reset(new It2MeConfirmationDialogProxy(
-      dialog_task_runner(), std::unique_ptr<It2MeConfirmationDialog>(dialog_)));
+  auto dialog =
+      base::MakeUnique<StubIt2MeConfirmationDialog>(dialog_task_runner());
+  dialog_ = dialog.get();
+  dialog_proxy_.reset(new It2MeConfirmationDialogProxy(dialog_task_runner(),
+                                                       std::move(dialog)));
 }
 
 It2MeConfirmationDialogProxyTest::~It2MeConfirmationDialogProxyTest() {}
 
 TEST_F(It2MeConfirmationDialogProxyTest, Show) {
   ResultCallbackTarget callback_target(main_task_runner());
 
   EXPECT_CALL(*dialog(), OnShow())
       .WillOnce(
           InvokeWithoutArgs(
               CreateFunctor(
                   &StubIt2MeConfirmationDialog::ReportResult,
                   base::Unretained(dialog()),
                   It2MeConfirmationDialog::Result::CANCEL)));
 
   EXPECT_CALL(callback_target,
               OnDialogResult(It2MeConfirmationDialog::Result::CANCEL))
       .WillOnce(
           InvokeWithoutArgs(this, &It2MeConfirmationDialogProxyTest::Quit));
 
   dialog_proxy()->Show(kTestEmailAddress, callback_target.MakeCallback());
 
   Run();
 }
 
 }  // namespace remoting