Track all extension frames in ProcessManager, inspect extensionoptions

chrome/test/data/extensions/api_test/messaging/connect_nobackground/test.js

+// Copyright (c) 2015 The Chromium Authors. All rights reserved.

[Devlin, 2015/10/30 01:21:44]

ditto

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+chrome.runtime.onConnect.addListener(function(port) {
+  var succeed1 = chrome.test.callbackAdded();
+  var succeed2 = chrome.test.callbackAdded();
+  var succeed3 = chrome.test.callbackAdded();
+
+  port.onMessage.addListener(function(msg) {
+    console.log('port.onMessage was triggered.');

[Devlin, 2015/10/30 01:21:43]

nit: try to avoid noisy logs in tests.  My general advice is to do something
like:
var enableLogging = false;
function log(message) {
  if (enableLogging)
    console.log(message);
}

So that for debugging, we can turn it on, but otherwise, we avoid the noise.

(We should probably add a test API method or something, but haven't yet.)

[robwu, 2015/10/30 10:15:49]

We have chrome.test.log, which causes messages to be printed in case of failure.
Sounds like what you want, doesn't it?
https://cs.chromium.org/%22TestNatives::Log%22

[Devlin, 2015/10/30 16:03:51]

Huh - thought that always logged.  Apparently not.  Yes, that's pretty much what
we want.

+    chrome.test.assertEq('Hello from content script', msg);
+    succeed1();
+  });

[Devlin, 2015/10/30 01:21:43]

nit: newline

+  port.onDisconnect.addListener(function() {
+    console.log('port.onDisconnect was triggered.');
+    succeed2();
+  });
+
+  chrome.tabs.sendMessage(port.sender.tab.id, '', function(reply) {
+    console.log('tab.sendMessage\'s response callback was invoked');
+    chrome.test.assertEq('Reply here', reply);
+    succeed3();
+  });
+});
+
+chrome.test.getConfig(function(config) {
+  var url = 'http://localhost:' + config.testServer.port +
+    '/extensions/test_file.html?will_test_connect_and_sendMessage';

[Devlin, 2015/10/30 01:21:44]

nit: 4-space indent

+  // Content script will try to connect and trigger onConnect.
+  chrome.tabs.create({
+    url: url
+  });
+});