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

Issue 2344333002: Expose remote associated interfaces through SyncMessageFilter (Closed)

Created:
4 years, 3 months ago by Ken Rockot(use gerrit already)
Modified:
4 years, 2 months ago
CC:
chromium-reviews, darin-cc_chromium.org, jam
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Expose remote associated interfaces through SyncMessageFilter This grants child process code safe and convenient access to remote Channel-associated interfaces on the IO thread. BUG=612500 Committed: https://crrev.com/336fc62ab5493e58dda7953cba587f382bde9c3b Cr-Commit-Position: refs/heads/master@{#423215}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -0 lines) Patch
M ipc/ipc_sync_message_filter.h View 4 chunks +27 lines, -0 lines 0 comments Download
M ipc/ipc_sync_message_filter.cc View 2 chunks +19 lines, -0 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 19 (12 generated)
Reilly Grant (use Gerrit)
I'm not sure when I can call GetRemoteAssociatedInterface(). If I post a task to the ...
4 years, 3 months ago (2016-09-21 03:10:20 UTC) #6
Reilly Grant (use Gerrit)
On 2016/09/21 at 03:10:20, Reilly Grant wrote: > I'm not sure when I can call ...
4 years, 3 months ago (2016-09-21 03:37:57 UTC) #7
Ken Rockot(use gerrit already)
Note that this is needed for https://codereview.chromium.org/2370643004. Created a separate CL to keep the change ...
4 years, 2 months ago (2016-10-05 14:11:33 UTC) #9
jam
lgtm
4 years, 2 months ago (2016-10-05 17:45:57 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2344333002/1
4 years, 2 months ago (2016-10-05 17:49:58 UTC) #16
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 2 months ago (2016-10-05 17:56:10 UTC) #17
commit-bot: I haz the power
4 years, 2 months ago (2016-10-05 17:59:17 UTC) #19
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/336fc62ab5493e58dda7953cba587f382bde9c3b
Cr-Commit-Position: refs/heads/master@{#423215}

Powered by Google App Engine
This is Rietveld 408576698