Please select Share dialog remains up if Client cancels and Host hits Share |
|||||||||
Issue descriptionCRD web app M56 (after fix for bug 658459 was added) OS: any What steps will reproduce the problem? (1)In IT2Me, hit Share on host (2)On client, enter the code and hit Connect (3)on the "Connecting..." dialog on Client, hit cancel (4)On the host, hit Share (5)Note that the "Please select Share..." dialog remains up until Host web app is restarted
,
Jan 14 2017
,
Jan 14 2017
,
Jan 14 2017
,
Jan 17 2017
,
Jan 20 2017
This problem is occurring because the session has been disconnected, but since we are in the middle of authenticating (the host is waiting for the user response via the dialog) the host is not notified of the disconnect. I am planning on adding a new method on the dialog to cancel it if the session is disconnected. This will prevent the hang. One note for Windows, the share dialog will remain visible until the timeout timer fires or the user makes a selection. THis is because the Windows dialog blocks the UI thread so it cannot receive the message. The right behavior will still happen though once one of those events occurs.
,
Jan 24 2017
,
Mar 15 2017
,
Mar 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d15dc6a9d267c09c67d36838cba240d69bc68ab2 commit d15dc6a9d267c09c67d36838cba240d69bc68ab2 Author: joedow <joedow@chromium.org> Date: Mon Mar 20 16:52:26 2017 Webapp share dialog is not closed when client end of the connection is closed This problem is occurring because the share request dialog must be accepted before the connection is considered authenticated and the host is only notified of a disconnect if the connection is authenticated. The host does receive an access denied callback in this case as OnSessionAuthenticationFailed is called. The initial state (session is not authenticated until dialog is accepted) is correct so I am adding code to dismiss the confirmation dialog if we receive OnAccessDenied (via OnSessionAuthenticationFailed) for the JID represented by the confirmation dialog. I also reset the host state (from 'connecting' to 'received access code' (i.e. ready for client connection) when this occurs so the webapp will update its UI appropriately. This new behavior allows the remote user to cancel while the share dialog is present and attempt to reconnect using the same access code. Since it is possible to display multiple confirmation dialogs in one session, I removed the static Create method from the confirmation dialog class and replaced it with a factory class. On platforms where the confirmation dialog is displayed with a blocking UI thread (i.e. Windows), the webapp UI will be updated once the timeout timer fires or the user makes a selection. On other platforms the dialog will be dismissed automatically. BUG= 681159 Review-Url: https://codereview.chromium.org/2650443002 Cr-Commit-Position: refs/heads/master@{#458096} [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_confirmation_dialog.h [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_confirmation_dialog_chromeos.cc [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_confirmation_dialog_linux.cc [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_confirmation_dialog_mac.mm [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_confirmation_dialog_proxy_unittest.cc [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_confirmation_dialog_win.cc [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_host.cc [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_host.h [modify] https://crrev.com/d15dc6a9d267c09c67d36838cba240d69bc68ab2/remoting/host/it2me/it2me_host_unittest.cc
,
Mar 20 2017
,
Apr 6 2017
Verified Fixed with Host version 59.0.3064.0. |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by jamiewa...@chromium.org
, Jan 14 2017