Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9)

Unified Diff: remoting/webapp/crd/js/client_session.js

Issue 1016623002: [Webapp Refactor] Reparent the ConnectedView into the delegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebasing Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/webapp/browser_test/bump_scroll_browser_test.js ('k') | remoting/webapp/crd/js/connected_view.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/crd/js/client_session.js
diff --git a/remoting/webapp/crd/js/client_session.js b/remoting/webapp/crd/js/client_session.js
index a77be6934f6ad28c156d6c4f50d9aa5071c593c9..63500d1507a4a6875ac1da926813e684556d0c50 100644
--- a/remoting/webapp/crd/js/client_session.js
+++ b/remoting/webapp/crd/js/client_session.js
@@ -387,9 +387,7 @@ remoting.ClientSession.prototype.onIncomingMessage_ = function(message) {
*/
remoting.ClientSession.prototype.onConnectionStatusUpdate =
function(status, error) {
- if (status == remoting.ClientSession.State.CONNECTED) {
- remoting.desktopConnectedView.onConnected();
- } else if (status == remoting.ClientSession.State.FAILED) {
+ if (status == remoting.ClientSession.State.FAILED) {
switch (error) {
case remoting.ClientSession.ConnectionError.HOST_IS_OFFLINE:
this.error_ = new remoting.Error(
@@ -450,8 +448,6 @@ remoting.ClientSession.prototype.onConnectionReady = function(ready) {
return;
}
- remoting.desktopConnectedView.onConnectionReady(ready);
-
this.raiseEvent(remoting.ClientSession.Events.videoChannelStateChanged,
ready);
};
@@ -475,10 +471,6 @@ remoting.ClientSession.prototype.onSetCapabilities = function(capabilities) {
if (this.hasCapability(remoting.ClientSession.Capability.GOOGLE_DRIVE)) {
this.sendGoogleDriveAccessToken_();
}
- if (this.hasCapability(
- remoting.ClientSession.Capability.VIDEO_RECORDER)) {
- remoting.desktopConnectedView.initVideoFrameRecorder();
- }
};
/**
@@ -613,16 +605,3 @@ remoting.ClientSession.prototype.sendGoogleDriveAccessToken_ = function() {
remoting.ACCESS_TOKEN_RESEND_INTERVAL_MS);
};
-/**
- * Enables or disables rendering of dirty regions for debugging.
- * @param {boolean} enable True to enable rendering.
- */
-remoting.ClientSession.prototype.enableDebugRegion = function(enable) {
- if (enable) {
- this.plugin_.setDebugDirtyRegionHandler(
- remoting.desktopConnectedView.handleDebugRegion.bind(
- remoting.desktopConnectedView));
- } else {
- this.plugin_.setDebugDirtyRegionHandler(null);
- }
-}
« no previous file with comments | « remoting/webapp/browser_test/bump_scroll_browser_test.js ('k') | remoting/webapp/crd/js/connected_view.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698