Native Messaging revert-revert

chrome/renderer/resources/extensions/miscellaneous_bindings.js

Index: chrome/renderer/resources/extensions/miscellaneous_bindings.js
diff --git a/chrome/renderer/resources/extensions/miscellaneous_bindings.js b/chrome/renderer/resources/extensions/miscellaneous_bindings.js
index d860bab0322f613b8c0455faed2ac4f351367de4..223964a9d43164946a6718687e8f4b8a43c0c0e3 100644
--- a/chrome/renderer/resources/extensions/miscellaneous_bindings.js
+++ b/chrome/renderer/resources/extensions/miscellaneous_bindings.js
@@ -27,6 +27,7 @@
   // Note: sendRequest is deprecated.
   chromeHidden.kRequestChannel = "chrome.extension.sendRequest";
   chromeHidden.kMessageChannel = "chrome.extension.sendMessage";
+  chromeHidden.kNativeMessageChannel = "chrome.extension.sendNativeMessage";
 
   // Map of port IDs to port object.
   var ports = {};
@@ -257,7 +258,8 @@
   // Shared implementation used by tabs.sendMessage and extension.sendMessage.
   chromeHidden.Port.sendMessageImpl = function(port, request,
                                                responseCallback) {
-    port.postMessage(request);
+    if (port.name != chromeHidden.kNativeMessageChannel)
+      port.postMessage(request);
 
     if (port.name == chromeHidden.kMessageChannel && !responseCallback) {

[eaugusti, 2012/10/16 19:52:05]

This line was also an unnecessary change in the original CL.
It was changed to an logical equivalent, but still quite unnecessary. 

       // TODO(mpcomplete): Do this for the old sendRequest API too, after