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

Unified Diff: chrome/browser/extensions/api/messaging/message_service.cc

Issue 10918255: The Windows portion of Native Messagaing (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years 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
« no previous file with comments | « base/process_util_win.cc ('k') | chrome/browser/extensions/api/messaging/native_message_port.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/messaging/message_service.cc
diff --git a/chrome/browser/extensions/api/messaging/message_service.cc b/chrome/browser/extensions/api/messaging/message_service.cc
index afc9407a64ebb14a26314acb76299963ec47f808..23fb7f3aaf8fe5a87d3b6ffb15dfef519ef32954 100644
--- a/chrome/browser/extensions/api/messaging/message_service.cc
+++ b/chrome/browser/extensions/api/messaging/message_service.cc
@@ -216,7 +216,7 @@ void MessageService::OpenChannelToNativeApp(
NativeMessageProcessHost::TYPE_CONNECT;
content::BrowserThread::PostTask(
- content::BrowserThread::FILE,
+ content::BrowserThread::IO,
FROM_HERE,
base::Bind(&NativeMessageProcessHost::Create,
base::WeakPtr<NativeMessageProcessHost::Client>(
@@ -242,6 +242,8 @@ void MessageService::FinalizeOpenChannelToNativeApp(
// Abandon the channel
if (!native_process.get()) {
LOG(ERROR) << "Failed to create native process.";
+ channel->opener->DispatchOnDisconnect(
+ GET_OPPOSITE_PORT_ID(receiver_port_id), true);
return;
}
channel->receiver.reset(new NativeMessagePort(native_process.release()));
« no previous file with comments | « base/process_util_win.cc ('k') | chrome/browser/extensions/api/messaging/native_message_port.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698