Move hello message from It2MeService to It2MeChannel

remoting/webapp/unittests/it2me_service_unittest.js

Index: remoting/webapp/unittests/it2me_service_unittest.js
diff --git a/remoting/webapp/unittests/it2me_service_unittest.js b/remoting/webapp/unittests/it2me_service_unittest.js
index a5924262d1a282a653241047627953a7044d6d84..7981152ce71e525448812f2116bafbbdd4d41624 100644
--- a/remoting/webapp/unittests/it2me_service_unittest.js
+++ b/remoting/webapp/unittests/it2me_service_unittest.js
@@ -97,23 +97,23 @@ test('should handle multiple helper connections', function() {
     var webappPort2 = createPort('it2me.helper.webapp', 'tabId2');
     it2meService.onWebappConnect_(webappPort2);
 
+    // verify that Hangouts can receive messages from webapp 1

[Jamie, 2014/08/15 21:57:32]

I remember asking for this change in another CL. Has there been some sort of
rebase confusion?

[kelvinp, 2014/08/15 23:25:01]

No, I am just trying address your feedback since I am modifying the same files. 
I have reverted the changes and will send out a separate CL to address them. 

     webappPort.onMessage.mock$fire({
       method: 'sessionStateChanged',
       state: remoting.ClientSession.State.CONNECTED
     });
 
-    // verify that hangout can receive message events from webapp 1
     sinon.assert.calledWith(hangoutPort.postMessage, {
       method: 'sessionStateChanged',
       state: remoting.ClientSession.State.CONNECTED
     });
 
+    // verify that Hangouts can receive messages from webapp 2.
     webappPort2.onMessage.mock$fire({
       method: 'sessionStateChanged',
       state: remoting.ClientSession.State.CLOSED
     });
 
-    // verify that hangout can receive message events from webapp 2.
     sinon.assert.calledWith(hangoutPort2.postMessage, {
       method: 'sessionStateChanged',
       state: remoting.ClientSession.State.CLOSED
@@ -129,17 +129,4 @@ test('should reject unknown connection', function() {
   sinon.assert.called(randomPort.disconnect);
 });
 
-test('messageExternal("hello") should return supportedFeatures', function() {
-  var response = null;
-  function callback(msg) {
-    response = msg;
-  }
-
-  it2meService.onMessageExternal_({
-    method: 'hello'
-  }, null, callback);
-
-  QUnit.ok(response.supportedFeatures instanceof Array);
-});
-
 })();