From b5a48189ca63951ffa855d57ee88da08615daeb8 Mon Sep 17 00:00:00 2001 From: jamiewalch Date: Mon, 24 Nov 2014 11:08:57 -0800 Subject: [PATCH] Disable grey-screen functionality. 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} --- remoting/webapp/crd/js/client_session.js | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/remoting/webapp/crd/js/client_session.js b/remoting/webapp/crd/js/client_session.js index 7def739a2ec1..81a6ae100184 100644 --- a/remoting/webapp/crd/js/client_session.js +++ b/remoting/webapp/crd/js/client_session.js @@ -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 {