Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(226)

Unified Diff: chrome/test/data/extensions/api_test/native_messaging/test.js

Issue 24409004: Disconnect native messaging port on read pipe EOF. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/extensions/api_test/native_messaging/test.js
diff --git a/chrome/test/data/extensions/api_test/native_messaging/test.js b/chrome/test/data/extensions/api_test/native_messaging/test.js
index e1a7bfc6bf3fc82cbc80f29f9ca3652b19a3795f..f5cc66c9b7f85ed3e2d5b785d4fa90a317c33069 100644
--- a/chrome/test/data/extensions/api_test/native_messaging/test.js
+++ b/chrome/test/data/extensions/api_test/native_messaging/test.js
@@ -76,10 +76,7 @@ chrome.test.getConfig(function(config) {
port.onMessage.addListener(function(message) {
port.onDisconnect.addListener(chrome.test.callback(
function() {},
- "Error when communicating with the native messaging host."));
- // Sending second message here should fail because the host has
- // already stopped.
- port.postMessage({ "secondMessage" : "foo" });
+ "Native host has exited."));
});
// Send first message that should stop the host.

Powered by Google App Engine
This is Rietveld 408576698