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

Issue 23851010: Modify ResourceMessageFilter so that it can support attaching to PluginProcessHost. (Closed)

Created:
7 years, 3 months ago by jam
Modified:
4 years, 8 months ago
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, miu+watch_chromium.org
Visibility:
Public.

Description

Modify ResourceMessageFilter so that it can support attaching to PluginProcessHost. Previously, a ResourceMessageFilter could assume that its client has one ResourceContext. For plugins this assumption won't hold anymore, as one plugin process serves many renderers. BUG=286074 R=ananta@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=222077

Patch Set 1 : #

Patch Set 2 : sync #

Unified diffs Side-by-side diffs Delta from patch set Stats (+200 lines, -196 lines) Patch
M content/browser/loader/async_resource_handler.h View 1 3 chunks +3 lines, -0 lines 0 comments Download
M content/browser/loader/async_resource_handler.cc View 1 3 chunks +6 lines, -6 lines 0 comments Download
M content/browser/loader/resource_dispatcher_host_impl.h View 1 1 chunk +0 lines, -7 lines 0 comments Download
M content/browser/loader/resource_dispatcher_host_impl.cc View 1 8 chunks +11 lines, -31 lines 0 comments Download
M content/browser/loader/resource_dispatcher_host_unittest.cc View 1 3 chunks +15 lines, -21 lines 0 comments Download
M content/browser/loader/resource_message_filter.h View 1 4 chunks +17 lines, -25 lines 0 comments Download
M content/browser/loader/resource_message_filter.cc View 1 2 chunks +7 lines, -10 lines 0 comments Download
M content/browser/loader/resource_scheduler_unittest.cc View 1 1 chunk +15 lines, -16 lines 0 comments Download
M content/browser/loader/sync_resource_handler.h View 1 3 chunks +3 lines, -0 lines 0 comments Download
M content/browser/loader/sync_resource_handler.cc View 1 3 chunks +5 lines, -4 lines 0 comments Download
M content/browser/renderer_host/render_process_host_impl.cc View 1 4 chunks +42 lines, -35 lines 0 comments Download
M content/browser/renderer_host/socket_stream_dispatcher_host.h View 1 3 chunks +5 lines, -4 lines 0 comments Download
M content/browser/renderer_host/socket_stream_dispatcher_host.cc View 1 2 chunks +3 lines, -5 lines 0 comments Download
M content/browser/web_contents/web_contents_impl.cc View 1 3 chunks +24 lines, -0 lines 0 comments Download
M content/browser/worker_host/worker_process_host.h View 1 2 chunks +13 lines, -0 lines 0 comments Download
M content/browser/worker_host/worker_process_host.cc View 1 5 chunks +25 lines, -32 lines 0 comments Download
M content/public/common/content_switches.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M content/public/common/content_switches.cc View 1 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
jam
This is the first step in getting plugin requests working. I have a work in ...
7 years, 3 months ago (2013-09-09 00:29:56 UTC) #1
ananta
lgtm
7 years, 3 months ago (2013-09-09 19:20:10 UTC) #2
courtneylaine072491
On 2013/09/09 00:29:56, jam wrote: > This is the first step in getting plugin requests ...
4 years, 9 months ago (2016-03-27 08:55:24 UTC) #3
courtneylaine072491
On 2013/09/09 00:29:56, jam wrote: > This is the first step in getting plugin requests ...
4 years, 9 months ago (2016-03-27 08:56:28 UTC) #4
courtneylaine072491
4 years, 8 months ago (2016-04-02 13:18:15 UTC) #5
Message was sent while issue was closed.
Why did this email come to me?
On Mar 27, 2016 3:56 AM, <courtneylaine072491@gmail.com> wrote:

> On 2013/09/09 00:29:56, jam wrote:
> > This is the first step in getting plugin requests working. I have a work
> in
> > progress at https://codereview.chromium.org/23503043 (which isn't ready
> for
> > review yet).
>
>
>
> https://codereview.chromium.org/23851010/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698