New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 681159 link

Starred by 0 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Please select Share dialog remains up if Client cancels and Host hits Share

Project Member Reported by ajnolley@chromium.org, Jan 13 2017

Issue description

CRD 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


 
What do you mean by the "Please select to Share dialog"? Can you attach a screen shot?
Screenshot from 2017-01-13 18:26:50.png
66.2 KB View Download

Comment 3 Deleted

Description: Show this description
Summary: Please select Share dialog remains up if Client cancels and Host hits Share (was: Please select to Share dialog remains up if Client cancels and Host hits Share)
Labels: M-57
Status: Assigned (was: Untriaged)

Comment 7 by joedow@chromium.org, Jan 20 2017

Status: Started (was: Assigned)
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.

Comment 8 by joedow@chromium.org, Jan 24 2017

Labels: -M-57 M-58

Comment 9 by joedow@chromium.org, Mar 15 2017

Labels: -M-58 M-59
Project Member

Comment 10 by bugdroid1@chromium.org, 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

Owner: ajnolley@chromium.org
Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified Fixed with Host version 59.0.3064.0. 

Sign in to add a comment