Move hello message from It2MeService to It2MeChannel

remoting/webapp/unittests/it2me_service_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 79 matching lines @@
     hangoutPort2.onMessage.mock$fire({
       method: 'connect',
       accessCode: "123412341234"
     });
 
     it2meService.onWebappConnect_(webappPort);
 
     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
     });
 });
 
 test('should reject unknown connection', function() {
   it2meService.onWebappConnect_(webappPort);
   sinon.assert.called(webappPort.disconnect);
 
   var randomPort = createPort('unsupported.port.name');
   it2meService.onConnectExternal_(randomPort);
   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);
-});
-
 })();