Move hello message from It2MeService to It2MeChannel

remoting/webapp/unittests/it2me_helper_channel_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';
 
 var appLauncher = null;
 var hangoutPort = null;
@@ skipping 43 matching lines @@
 });
 
 function promiseResolveSynchronous(value) {
   return {
     then: function(callback) {
       callback('tabId');
     }
   };
 }
 
+test('onHangoutMessage_("hello") should return supportedFeatures', function() {
+  hangoutPort.onMessage.mock$fire(
+      { method: remoting.It2MeHelperChannel.HangoutMessageTypes.HELLO });
+
+  sinon.assert.calledWith(hangoutPort.postMessage, {
+    method: remoting.It2MeHelperChannel.HangoutMessageTypes.HELLO_RESPONSE,
+    supportedFeatures: base.values(remoting.It2MeHelperChannel.Features)

[Jamie, 2014/08/15 23:30:12]

This is testing something stricter than it needs to: namely that the order
matches. In practise, I can't see that being a problem, and if the test fails,
it will at least fail safe. Unless I've missed some more serious failure mode, I
think this is fine, but if it's easy to make the test order-agnostic, it's
probably worthwhile.

+  });
+});
+
 test('onHangoutMessage_(|connect|) should launch the webapp',
     function() {
   sinon.assert.called(appLauncher.launch);
   QUnit.equal(helperChannel.instanceId(), 'tabId');
 });
 
 test('onWebappMessage() should forward messages to hangout', function() {
   // Execute.
   helperChannel.onWebappConnect(webappPort);
   webappPort.onMessage.mock$fire({
@@ skipping 96 matching lines @@
   hangoutPort.onDisconnect.mock$fire();
 
   sinon.assert.calledOnce(appLauncher.close);
   sinon.assert.calledOnce(disconnectCallback);
 
   sinon.assert.called(hangoutPort.disconnect);
   sinon.assert.called(webappPort.disconnect);
 });
 
 })();