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

Unified Diff: chrome/browser/renderer_host/chrome_extension_message_filter.cc

Issue 2300453002: [Extensions] Begin making Extension port initialization asynchronous (Closed)
Patch Set: Nasko's Created 4 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
« no previous file with comments | « chrome/browser/renderer_host/chrome_extension_message_filter.h ('k') | chrome/renderer/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_host/chrome_extension_message_filter.cc
diff --git a/chrome/browser/renderer_host/chrome_extension_message_filter.cc b/chrome/browser/renderer_host/chrome_extension_message_filter.cc
index a34a8ac157eb10b35ce6136272eb67af6f47071c..1533889e36885c2947d0d464e816d1d08e135233 100644
--- a/chrome/browser/renderer_host/chrome_extension_message_filter.cc
+++ b/chrome/browser/renderer_host/chrome_extension_message_filter.cc
@@ -129,9 +129,11 @@ void ChromeExtensionMessageFilter::OnOpenChannelToExtension(
const ExtensionMsg_ExternalConnectionInfo& info,
const std::string& channel_name,
bool include_tls_channel_id,
- int* port_id) {
- int port2_id;
- extensions::MessageService::AllocatePortIdPair(port_id, &port2_id);
+ int request_id) {
+ int port1_id = 0;
+ int port2_id = 0;
+ extensions::MessageService::AllocatePortIdPair(&port1_id, &port2_id);
+ Send(new ExtensionMsg_AssignPortId(routing_id, port1_id, request_id));
BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
« no previous file with comments | « chrome/browser/renderer_host/chrome_extension_message_filter.h ('k') | chrome/renderer/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698