Suppress the exception thrown when responding to message with no response callback.

chrome/browser/resources/hotword_helper/manager.js

 // Copyright (c) 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.
 
 'use strict';
 
 /**
  * @fileoverview This extension manages communications between Chrome,
  * Google.com pages and the Chrome Hotword extension.
  *
@@ skipping 40 matching lines @@
 
 /**
  * @param {Tab} tab Tab to inject.
  * @param {function(HotwordStatus)} sendResponse Callback function to respond
  *     to sender.
  * @param {HotwordStatus} hotwordStatus Status of the hotword extension.
  * @private
  */
 OptInManager.prototype.injectTab_ = function(
     tab, sendResponse, hotwordStatus) {
-  if (tab.incognito || !hotwordStatus.available) {
+  try {
+    if (tab.incognito || !hotwordStatus.available) {
+      sendResponse({'doNotShowOptinMessage': true});
+      return;
+    }
+
+    if (!hotwordStatus.enabledSet) {
+      sendResponse(hotwordStatus);
+      return;
+    }
+
+    if (hotwordStatus.enabled)
+      chrome.tabs.executeScript(tab.id, {'file': 'audio_client.js'});
     sendResponse({'doNotShowOptinMessage': true});
-    return;
+  } catch (err) {
+    // Suppress the exception thrown when the page doesn't specify a response
+    // callback. Unfortunately, there doesn't appear to be a way to detect
+    // one-way messages without explicitly saying in the message itself.
+    if (err.message == 'Attempting to use a disconnected port object')

[rpetterson, 2014/09/22 23:04:41]

Weird question, but is this always going to be in english or the same phrasing?
It would seem that an error code would be better if possible.

[Anand Mistry (off Chromium), 2014/09/22 23:50:43]

It's an untranslated constant in
https://code.google.com/p/chromium/codesearch#chromium/src/extensions/rendere...

I realise this is brittle, but I'm afraid to change this exception to be
something other than a string since it might break existing apps which have
discovered this (undocumented) behaviour.

[rpetterson, 2014/09/22 23:53:03]

Understood. Please add a note to the comment listing where this string is
defined.

[Anand Mistry (off Chromium), 2014/09/23 00:31:30]

Done.

+      return;
+    throw err;
   }
-
-  if (!hotwordStatus.enabledSet) {
-    sendResponse(hotwordStatus);
-    return;
-  }
-
-  if (hotwordStatus.enabled)
-    chrome.tabs.executeScript(tab.id, {'file': 'audio_client.js'});
-  sendResponse({'doNotShowOptinMessage': true});
 };
 
 
 /**
  * Handles messages from the helper content script.
  * @param {*} request Message from the sender.
  * @param {MessageSender} sender Information about the sender.
  * @param {function(HotwordStatus)} sendResponse Callback function to respond
  *     to sender.
  * @return {boolean} Whether to maintain the port open to call sendResponse.
@@ skipping 120 matching lines @@
   // TODO(rlp): Possibly remove the next line. It's proably not used, but
   // leaving for now to be safe. We should remove it once all messsage
   // relaying is removed form the content scripts.
   chrome.runtime.onMessage.addListener(this.handleMessage_.bind(this));
   chrome.runtime.onMessageExternal.addListener(
       this.handleMessage_.bind(this));
 };
 
 
 new OptInManager().initialize();