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

Unified Diff: remoting/webapp/client_session.js

Issue 339613003: Remove the blue tool-bar for apps v2. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use classes for help and feedback elements. Created 6 years, 5 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
Index: remoting/webapp/client_session.js
diff --git a/remoting/webapp/client_session.js b/remoting/webapp/client_session.js
index fec67ff77eaa818412ea1af50250c2c51c91784c..3e0a5d8196d9d69cab622800f951748b0505af22 100644
--- a/remoting/webapp/client_session.js
+++ b/remoting/webapp/client_session.js
@@ -609,9 +609,10 @@ remoting.ClientSession.prototype.removePlugin = function() {
remoting.fullscreen.removeListener(listener);
});
if (remoting.windowFrame) {
- remoting.windowFrame.setConnected(false);
+ remoting.windowFrame.setClientSession(null);
+ } else {
+ remoting.toolbar.setClientSession(null);
}
- remoting.toolbar.setClientSession(null);
// Remove mediasource-rendering class from the container - this will also
// hide the <video> element.
@@ -995,9 +996,10 @@ remoting.ClientSession.prototype.onConnectionStatusUpdate_ =
remoting.fullscreen.addListener(this.callOnFullScreenChanged_);
remoting.fullscreen.syncWithMaximize(true);
if (remoting.windowFrame) {
- remoting.windowFrame.setConnected(true);
+ remoting.windowFrame.setClientSession(this);
+ } else {
+ remoting.toolbar.setClientSession(this);
}
- remoting.toolbar.setClientSession(this);
this.container_.addEventListener('mousemove',
this.updateMouseCursorPosition_,

Powered by Google App Engine
This is Rietveld 408576698