Hangout remote desktop part II - background.html and AppLauncher

remoting/webapp/unittests/base_unittest.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 (function() {
 
 'use strict';
 
 module('base');
 
@@ skipping 38 matching lines @@
     base.dispose(obj);
     sinon.assert.called(obj.dispose);
 });
 
 test('dispose(obj) should not crash if |obj| is null',
   function() {
     expect(0);
     base.dispose(null);
 });
 
+test('urljoin(url, opt_param) should return url if |opt_param| is missing',
+  function() {
+    QUnit.equal(
+        base.urlJoin('http://www.chromium.org'), 'http://www.chromium.org');
+});
+
+test('urljoin(url, opt_param) should urlencode |opt_param|',
+  function() {
+    var result = base.urlJoin('http://www.chromium.org', {
+      a: 'a',
+      foo: 'foo='

[Jamie, 2014/08/12 20:42:55]

Can you add a few more escape-requiring characters here, just to be on the safe
side?

[kelvinp, 2014/08/12 21:42:43]

Done.

+    });
+    QUnit.equal(result, 'http://www.chromium.org?a=a&foo=foo%3D');
+});
+
 QUnit.asyncTest('Promise.sleep(delay) should fulfill the promise after |delay|',
   function() {
     var isCalled = false;
     var clock = this.clock;
 
     base.Promise.sleep(100).then(function(){
       isCalled = true;
       ok(true, 'Promise.sleep() is fulfilled after delay.');
       QUnit.start();
     });
@@ skipping 106 matching lines @@
 
     source.addEventListener('foo', sink.listener);
     source.raiseEvent('foo');
     sinon.assert.calledOnce(sink.listener);
 
     source.raiseEvent('foo');
     sinon.assert.calledOnce(sink.listener);
 });
 
 })();