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

Unified Diff: extensions/renderer/extension_frame_helper.cc

Issue 1966283002: Remove port lifetime management from renderer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Simplify: Assume that opener port is open, only send reply if closed Created 4 years, 7 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: extensions/renderer/extension_frame_helper.cc
diff --git a/extensions/renderer/extension_frame_helper.cc b/extensions/renderer/extension_frame_helper.cc
index ba47a7c74887068b031a3ef9ef62e316f5bc303f..fc62fdf8f5585b5cdbcb6ac0618db77508108d57 100644
--- a/extensions/renderer/extension_frame_helper.cc
+++ b/extensions/renderer/extension_frame_helper.cc
@@ -189,6 +189,8 @@ void ExtensionFrameHelper::WillReleaseScriptContext(
bool ExtensionFrameHelper::OnMessageReceived(const IPC::Message& message) {
bool handled = true;
IPC_BEGIN_MESSAGE_MAP(ExtensionFrameHelper, message)
+ IPC_MESSAGE_HANDLER(ExtensionMsg_CheckHasMessagePort,
+ OnExtensionCheckHasMessagePort)
IPC_MESSAGE_HANDLER(ExtensionMsg_DispatchOnConnect,
OnExtensionDispatchOnConnect)
IPC_MESSAGE_HANDLER(ExtensionMsg_DeliverMessage, OnExtensionDeliverMessage)
@@ -206,6 +208,11 @@ bool ExtensionFrameHelper::OnMessageReceived(const IPC::Message& message) {
return handled;
}
+void ExtensionFrameHelper::OnExtensionCheckHasMessagePort(int port_id) {
+ MessagingBindings::CheckHasMessagePort(
+ extension_dispatcher_->script_context_set(), port_id, render_frame());
+}
+
void ExtensionFrameHelper::OnExtensionDispatchOnConnect(
int target_port_id,
const std::string& channel_name,
@@ -223,6 +230,7 @@ void ExtensionFrameHelper::OnExtensionDispatchOnConnect(
}
void ExtensionFrameHelper::OnExtensionDeliverMessage(int target_id,
+ int source_tab_id,
const Message& message) {
MessagingBindings::DeliverMessage(
extension_dispatcher_->script_context_set(), target_id, message,

Powered by Google App Engine
This is Rietveld 408576698