Skip to content

Commit

Permalink
Disable grey-screen functionality.
Browse files Browse the repository at this point in the history
This is a temporary solution until we can fix the logic that determines when the network connection is lost.

BUG=435315

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

Cr-Commit-Position: refs/heads/master@{#305461}
  • Loading branch information
jamiewalch authored and Commit bot committed Nov 24, 2014
1 parent d138f04 commit b5a4818
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions remoting/webapp/crd/js/client_session.js
Original file line number Diff line number Diff line change
Expand Up @@ -1014,6 +1014,16 @@ remoting.ClientSession.prototype.onRouteChanged_ =
* @param {boolean} ready True if the connection is ready.
*/
remoting.ClientSession.prototype.onConnectionReady_ = function(ready) {
// TODO(jamiewalch): Currently, the logic for determining whether or not the
// connection is available is based solely on whether or not any video frames
// have been received recently. which leads to poor UX on slow connections.
// Re-enable this once crbug.com/435315 has been fixed.
var ignoreVideoChannelState = true;
if (ignoreVideoChannelState) {
console.log('Video channel ' + (ready ? '' : 'not ') + 'ready.');
return;
}

if (!ready) {
this.container_.classList.add('session-client-inactive');
} else {
Expand Down

0 comments on commit b5a4818

Please sign in to comment.