[CRD iOS] In session connection failure not handled properly |
|||||||
Issue descriptionApp Version: ToT CRD build (64.0.3261.0) OS: iOS 11.2 What steps will reproduce the problem? (1) Connect to a host (2) Minimize it to the background (3) Wait for 1-2 minutes (4) Reopen the app What is the expected result? App shows the P2P error message. User can reconnect or go back. What happens instead? App gets stuck in the host view. It seems you can still operate but the app will crash afterwards.
,
Dec 13 2017
This CL only affects Chrome Remote Desktop for iOS and does not affect Chrome.
,
Dec 14 2017
This bug requires manual review: M64 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), abdulsyed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 18 2017
Ping?
,
Dec 21 2017
Sorry yuweih@. Please can you provide the rational as to why this should be merged in M64? What is the risk associated with this CL?
,
Dec 21 2017
This CL fixes a bug that the Chrome Remote Desktop iOS app occasionally crashes when bringing the app back to the foreground. The risk is generally low: * Changes are all within the Chrome Remote Desktop iOS project. It doesn't affect Chrome or anything else. * The change has been well tested on ToT and we have not found any issue so far. * The only possible impact could be build failure or other issues on the beta buildbots. If this happens we will be happy to revert the change and target it to M65.
,
Dec 21 2017
Merge approved! Please make sure to verify the fix after merging it.
,
Dec 22 2017
Thanks!
,
Dec 22 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2cfe78cf6e17e03a29131a64a26a1c2e3431a048 commit 2cfe78cf6e17e03a29131a64a26a1c2e3431a048 Author: Yuwei Huang <yuweih@chromium.org> Date: Fri Dec 22 00:13:07 2017 [CRD iOS] Properly handle in-session connection failures * Makes ChromotingSession automatically release its resources when the connection is closed or dropped remotely, so that scheduled tasks don't crash the app. * Makes ConnectionViewController pop out the host view and reveal the error message when the connection is dropped. Bug: 794268 Change-Id: I739dc6efd8dc716f16c8832d34f0f9a9b349af99 Reviewed-on: https://chromium-review.googlesource.com/822169 Commit-Queue: Yuwei Huang <yuweih@chromium.org> Reviewed-by: Jamie Walch <jamiewalch@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#523854}(cherry picked from commit 71b95d6fcc9a85902707560584d880ab4558ceed) Reviewed-on: https://chromium-review.googlesource.com/841662 Reviewed-by: Yuwei Huang <yuweih@chromium.org> Cr-Commit-Position: refs/branch-heads/3282@{#332} Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840} [modify] https://crrev.com/2cfe78cf6e17e03a29131a64a26a1c2e3431a048/remoting/client/chromoting_session.cc [modify] https://crrev.com/2cfe78cf6e17e03a29131a64a26a1c2e3431a048/remoting/ios/app/client_connection_view_controller.mm
,
Dec 27 2017
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by bugdroid1@chromium.org
, Dec 13 2017