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

Unified Diff: android_webview/common/aw_message_port_messages.h

Issue 956763002: Implement the close() API for Message ports (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address hush comments Created 5 years, 10 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: android_webview/common/aw_message_port_messages.h
diff --git a/android_webview/common/aw_message_port_messages.h b/android_webview/common/aw_message_port_messages.h
index 00ec171a7d7f38edb5b7e2c149ee1ed0258f051b..57fabf64454b33c405d6c3e317aa2185270b59f5 100644
--- a/android_webview/common/aw_message_port_messages.h
+++ b/android_webview/common/aw_message_port_messages.h
@@ -58,6 +58,12 @@ IPC_MESSAGE_ROUTED3(AwMessagePortMsg_AppToWebMessage,
base::string16 /* message */,
std::vector<int> /* sent message port_ids */)
+// Used to defer message port closing until after all in-flight messages
+// are flushed from renderer to browser. Renderer piggy-backs the message
+// to browser.
+IPC_MESSAGE_ROUTED1(AwMessagePortMsg_ClosePort,
+ int /* message port id */)
+
//-----------------------------------------------------------------------------
// These are messages sent from the renderer to the browser process.
@@ -72,3 +78,7 @@ IPC_MESSAGE_ROUTED3(AwMessagePortHostMsg_ConvertedAppToWebMessage,
int /* recipient message port id */,
base::string16 /* converted message */,
std::vector<int> /* sent message port_ids */)
+
+// Response to AwMessagePortMsg_ClosePort
+IPC_MESSAGE_ROUTED1(AwMessagePortHostMsg_ClosePort,
nasko 2015/02/26 17:45:20 Since this is a response to a message sent from th
sgurun-gerrit only 2015/02/26 18:00:51 good point. done.
+ int /* message port id */)

Powered by Google App Engine
This is Rietveld 408576698