New issue
Advanced search Search tips

Issue 608491 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: May 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 0
Type: Bug



Sign in to add a comment

Chromoting client crashes on connect

Project Member Reported by jamiewa...@chromium.org, May 2 2016

Issue description

CRD Version: 51.0.2704.7 (v2)

What steps will reproduce the problem?
(1) Connect to an IT2Me host.

What is the expected output?
Successful connection

What do you see instead?
White screen and the following error in the console log:

console_wrapper.js:115 TypeError: Cannot read property 'toFixed' of undefined(…) "client_plugin_impl.js:168"
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 2 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/37dc19f6f053ae326c3cd9049926fb7466f9e1bd

commit 37dc19f6f053ae326c3cd9049926fb7466f9e1bd
Author: jamiewalch <jamiewalch@chromium.org>
Date: Mon May 02 23:23:39 2016

Guard against undefined average in stats display.

BUG= 608491 

Review-Url: https://codereview.chromium.org/1941983002
Cr-Commit-Position: refs/heads/master@{#391083}

[modify] https://crrev.com/37dc19f6f053ae326c3cd9049926fb7466f9e1bd/remoting/webapp/base/js/connection_stats.js

Labels: Merge-Request-51
Status: Fixed (was: Started)
Requesting merge to M51. This change only affects the Chrome Remote Desktop app; there are no changes to Chrome.

Comment 3 by tin...@google.com, May 3 2016

Labels: -Merge-Request-51 Merge-Approved-51 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M51 (branch: 2704)
Project Member

Comment 4 by bugdroid1@chromium.org, May 3 2016

Labels: -merge-approved-51 merge-merged-2704
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/78c73d8cd327bec0bdd58f1bb9a40788746a5d25

commit 78c73d8cd327bec0bdd58f1bb9a40788746a5d25
Author: Jamie Walch <jamiewalch@chromium.org>
Date: Tue May 03 22:38:40 2016

Guard against undefined average in stats display.

BUG= 608491 

Review-Url: https://codereview.chromium.org/1941983002
Cr-Commit-Position: refs/heads/master@{#391083}
(cherry picked from commit 37dc19f6f053ae326c3cd9049926fb7466f9e1bd)

Review URL: https://codereview.chromium.org/1951633002 .

Cr-Commit-Position: refs/branch-heads/2704@{#365}
Cr-Branched-From: 6e53600def8f60d8c632fadc70d7c1939ccea347-refs/heads/master@{#386251}

[modify] https://crrev.com/78c73d8cd327bec0bdd58f1bb9a40788746a5d25/remoting/webapp/base/js/connection_stats.js

 Issue 608770  has been merged into this issue.
Summary: Chromoting client crashes on connect (was: IT2Me client crashes on connect)
Issue also affects Me2Me, abeit intermittently.
Status: Verified (was: Fixed)
Verified Fixed in 51.0.2704.53

Sign in to add a comment