win: Remove a few instances of the "goto error" pattern.

remoting/host/win/rdp_client_window.h

 // Copyright (c) 2013 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.
 
 #ifndef REMOTING_HOST_WIN_RDP_HOST_WINDOW_H_
 #define REMOTING_HOST_WIN_RDP_HOST_WINDOW_H_
 
 #include <atlbase.h>
 #include <atlcom.h>
 #include <atlcrack.h>
@@ skipping 108 matching lines @@
   END_SINK_MAP()
 
   // mstsc::IMsTscAxEvents notifications.
   STDMETHOD(OnAuthenticationWarningDisplayed)();
   STDMETHOD(OnAuthenticationWarningDismissed)();
   STDMETHOD(OnConnected)();
   STDMETHOD(OnDisconnected)(long reason);
   STDMETHOD(OnFatalError)(long error_code);
   STDMETHOD(OnConfirmClose)(VARIANT_BOOL* allow_close);
 
+  int LogOnCreateError(HRESULT error);

[Sergey Ulanov, 2015/07/14 16:09:50]

Please add a comment here, e.g.: Helper used to handle errors in OnCreate().

+
   // Wrappers for the event handler's methods that make sure that
   // OnDisconnected() is the last notification delivered and is delevered
   // only once.
   void NotifyConnected();
   void NotifyDisconnected();
 
   // Invoked to report connect/disconnect events.
   EventHandler* event_handler_;
 
   // Contains the requested dimensions of the screen.
@@ skipping 10 matching lines @@
   base::win::ScopedComPtr<mstsc::IMsRdpClientAdvancedSettings> client_settings_;
 
   // Used to cancel modal dialog boxes shown by the RDP control.
   class WindowHook;
   scoped_refptr<WindowHook> window_activate_hook_;
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_WIN_RDP_HOST_WINDOW_H_