Move hello message from It2MeService to It2MeChannel

remoting/webapp/unittests/it2me_helper_channel_unittest.js

Index: remoting/webapp/unittests/it2me_helper_channel_unittest.js
diff --git a/remoting/webapp/unittests/it2me_helper_channel_unittest.js b/remoting/webapp/unittests/it2me_helper_channel_unittest.js
index b5511c33058c9c9a0ef7d3ceb1aea7589e0561d1..f0a0fa230ea08838d45436cb292d28b4daa4812c 100644
--- a/remoting/webapp/unittests/it2me_helper_channel_unittest.js
+++ b/remoting/webapp/unittests/it2me_helper_channel_unittest.js
@@ -61,6 +61,16 @@ function promiseResolveSynchronous(value) {
   };
 }
 
+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);