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

Unified Diff: remoting/webapp/app_remoting/js/app_remoting.js

Issue 1015553003: Added more typechecking functions and unit tests for existing code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/remoting_webapp_files.gypi ('k') | remoting/webapp/base/js/base.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/app_remoting/js/app_remoting.js
diff --git a/remoting/webapp/app_remoting/js/app_remoting.js b/remoting/webapp/app_remoting/js/app_remoting.js
index 83493ebf5e370678727c054bb3d800840b2ba9c3..f6b46a6dbcd476e8a034e3d59456633b4742f83f 100644
--- a/remoting/webapp/app_remoting/js/app_remoting.js
+++ b/remoting/webapp/app_remoting/js/app_remoting.js
@@ -315,7 +315,7 @@ remoting.AppRemoting.prototype.onExtensionMessage = function(type, message) {
case 'openURL':
// URL requests from the hosted app are untrusted, so disallow anything
// other than HTTP or HTTPS.
- var url = getStringAttr(message, 'url');
+ var url = base.getStringAttr(message, 'url');
if (url.indexOf('http:') != 0 && url.indexOf('https:') != 0) {
console.error('Bad URL: ' + url);
} else {
@@ -324,13 +324,13 @@ remoting.AppRemoting.prototype.onExtensionMessage = function(type, message) {
return true;
case 'onWindowRemoved':
- var id = getNumberAttr(message, 'id');
+ var id = base.getNumberAttr(message, 'id');
this.windowActivationMenu_.remove(id);
return true;
case 'onWindowAdded':
- var id = getNumberAttr(message, 'id');
- var title = getStringAttr(message, 'title');
+ var id = base.getNumberAttr(message, 'id');
+ var title = base.getStringAttr(message, 'title');
this.windowActivationMenu_.add(id, title);
return true;
@@ -339,15 +339,15 @@ remoting.AppRemoting.prototype.onExtensionMessage = function(type, message) {
return true;
case 'setKeyboardLayouts':
- var supportedLayouts = getArrayAttr(message, 'supportedLayouts');
- var currentLayout = getStringAttr(message, 'currentLayout');
+ var supportedLayouts = base.getArrayAttr(message, 'supportedLayouts');
+ var currentLayout = base.getStringAttr(message, 'currentLayout');
console.log('Current host keyboard layout: ' + currentLayout);
console.log('Supported host keyboard layouts: ' + supportedLayouts);
this.keyboardLayoutsMenu_.setLayouts(supportedLayouts, currentLayout);
return true;
case 'pingResponse':
- var then = getNumberAttr(message, 'timestamp');
+ var then = base.getNumberAttr(message, 'timestamp');
var now = new Date().getTime();
this.contextMenu_.updateConnectionRTT(now - then);
return true;
« no previous file with comments | « remoting/remoting_webapp_files.gypi ('k') | remoting/webapp/base/js/base.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698