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

Issue 7791005: Stop using the default profile's proxy service for plugin proxy requests, and instead use the ass... (Closed)

Created:
9 years, 3 months ago by jam
Modified:
9 years, 3 months ago
Reviewers:
ananta
CC:
chromium-reviews, joi+watch-content_chromium.org, pam+watch_chromium.org, darin-cc_chromium.org
Visibility:
Public.

Description

Stop using the default profile's proxy service for plugin proxy requests, and instead use the associated profile's proxy service. I proxy the IPC through the renderer first, as that makes it easy to get to the associated profile. BUG=92361, 64339 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98728

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : fix test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+90 lines, -137 lines) Patch
M chrome_frame/renderer_glue.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M content/browser/plugin_process_host.cc View 2 chunks +0 lines, -2 lines 0 comments Download
M content/browser/renderer_host/browser_render_process_host.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/browser/resolve_proxy_msg_helper.h View 5 chunks +8 lines, -10 lines 0 comments Download
M content/browser/resolve_proxy_msg_helper.cc View 5 chunks +19 lines, -48 lines 0 comments Download
M content/browser/resolve_proxy_msg_helper_unittest.cc View 1 2 3 10 chunks +14 lines, -16 lines 0 comments Download
M content/common/child_process_messages.h View 1 chunk +0 lines, -11 lines 0 comments Download
M content/common/plugin_messages.h View 1 1 chunk +5 lines, -0 lines 0 comments Download
M content/common/view_messages.h View 1 chunk +7 lines, -0 lines 0 comments Download
M content/plugin/plugin_thread.cc View 2 chunks +0 lines, -18 lines 0 comments Download
M content/plugin/webplugin_proxy.h View 1 chunk +1 line, -0 lines 0 comments Download
M content/plugin/webplugin_proxy.cc View 1 1 chunk +6 lines, -0 lines 0 comments Download
M content/renderer/pepper_plugin_delegate_impl.cc View 2 chunks +2 lines, -3 lines 0 comments Download
M content/renderer/webplugin_delegate_proxy.h View 1 chunk +1 line, -0 lines 0 comments Download
M content/renderer/webplugin_delegate_proxy.cc View 2 chunks +9 lines, -0 lines 0 comments Download
M webkit/glue/webkit_glue.h View 1 chunk +0 lines, -3 lines 0 comments Download
M webkit/plugins/npapi/plugin_host.cc View 1 chunk +7 lines, -1 line 0 comments Download
M webkit/plugins/npapi/webplugin.h View 1 chunk +3 lines, -0 lines 0 comments Download
M webkit/plugins/npapi/webplugin_impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M webkit/plugins/npapi/webplugin_impl.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M webkit/tools/test_shell/simple_resource_loader_bridge.cc View 2 chunks +0 lines, -20 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jam
9 years, 3 months ago (2011-08-29 18:17:15 UTC) #1
ananta
9 years, 3 months ago (2011-08-29 18:41:30 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698