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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 6628035: Move resource related IPCs to their own file in content. (Closed)

Created:
9 years, 9 months ago by jam
Modified:
9 years, 7 months ago
Reviewers:
Tom Sepez
CC:
chromium-reviews, cbentzel+watch_chromium.org, tburkard+watch_chromium.org, Paweł Hajdan Jr., darin-cc_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

Move resource related IPCs to their own file in content. I added a IPC_STRUCT_TRAITS_PARENT macro to allow the automatic serializing macros to add a parent struct. TBR=tsepez Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=77048

Patch Set 1 : '' #

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+852 lines, -1097 lines) Patch
M chrome/browser/debugger/devtools_netlog_observer.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/net/load_timing_observer.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/prerender/prerender_resource_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/prerender/prerender_resource_handler_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/renderer_host/browser_render_process_host.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/renderer_host/download_resource_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/renderer_host/download_throttling_resource_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/renderer_host/safe_browsing_resource_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_common.gypi View 1 chunk +1 line, -3 lines 0 comments Download
M chrome/common/common_param_traits.h View 1 2 4 chunks +1 line, -35 lines 0 comments Download
M chrome/common/common_param_traits.cc View 2 chunks +0 lines, -221 lines 0 comments Download
M chrome/common/render_messages.h View 3 chunks +21 lines, -82 lines 0 comments Download
M chrome/common/render_messages.cc View 2 chunks +1 line, -268 lines 0 comments Download
M chrome/common/render_messages_internal.h View 3 chunks +0 lines, -85 lines 0 comments Download
M chrome/common/render_messages_params.h View 3 chunks +0 lines, -69 lines 0 comments Download
M chrome/common/render_messages_params.cc View 3 chunks +0 lines, -124 lines 0 comments Download
M chrome/common/resource_dispatcher.cc View 1 2 13 chunks +30 lines, -31 lines 0 comments Download
M chrome/common/resource_dispatcher_unittest.cc View 5 chunks +7 lines, -6 lines 0 comments Download
D chrome/common/resource_response.h View 1 chunk +0 lines, -50 lines 0 comments Download
D chrome/common/resource_response.cc View 1 chunk +0 lines, -18 lines 0 comments Download
M chrome/renderer/render_thread.cc View 4 chunks +10 lines, -9 lines 0 comments Download
M chrome/renderer/safe_browsing/render_view_fake_resources_test.h View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/renderer/safe_browsing/render_view_fake_resources_test.cc View 1 5 chunks +7 lines, -6 lines 0 comments Download
M content/browser/plugin_process_host.cc View 2 chunks +2 lines, -1 line 0 comments Download
M content/browser/renderer_host/async_resource_handler.cc View 7 chunks +14 lines, -14 lines 0 comments Download
M content/browser/renderer_host/buffered_resource_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/redirect_to_file_resource_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/resource_dispatcher_host.h View 2 chunks +4 lines, -4 lines 0 comments Download
M content/browser/renderer_host/resource_dispatcher_host.cc View 7 chunks +15 lines, -14 lines 0 comments Download
M content/browser/renderer_host/resource_dispatcher_host_unittest.cc View 10 chunks +21 lines, -20 lines 0 comments Download
M content/browser/renderer_host/resource_message_filter.h View 3 chunks +3 lines, -3 lines 0 comments Download
M content/browser/renderer_host/resource_message_filter.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/socket_stream_dispatcher_host.cc View 2 chunks +2 lines, -1 line 0 comments Download
M content/browser/renderer_host/sync_resource_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/sync_resource_handler.cc View 2 chunks +2 lines, -1 line 0 comments Download
M content/browser/renderer_host/x509_user_cert_resource_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/worker_host/worker_process_host.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/common/common_param_traits.h View 1 2 2 chunks +73 lines, -0 lines 0 comments Download
M content/common/common_param_traits.cc View 2 chunks +400 lines, -0 lines 0 comments Download
M content/common/content_message_generator.h View 1 chunk +1 line, -0 lines 0 comments Download
A content/common/resource_messages.h View 1 chunk +196 lines, -0 lines 0 comments Download
A + content/common/resource_response.h View 3 chunks +3 lines, -12 lines 0 comments Download
M content/content_common.gypi View 1 chunk +2 lines, -0 lines 0 comments Download
M ipc/ipc_message_macros.h View 1 chunk +5 lines, -3 lines 0 comments Download
M ipc/ipc_message_null_macros.h View 2 chunks +2 lines, -0 lines 0 comments Download
M ipc/ipc_message_utils.h View 1 chunk +1 line, -0 lines 0 comments Download
M ipc/param_traits_log_macros.h View 2 chunks +6 lines, -0 lines 0 comments Download
M ipc/param_traits_macros.h View 1 chunk +1 line, -0 lines 0 comments Download
M ipc/param_traits_read_macros.h View 1 chunk +2 lines, -0 lines 0 comments Download
M ipc/param_traits_write_macros.h View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jam
hey Tom, I'm checking this in TBR as with all the file moves. If you ...
9 years, 9 months ago (2011-03-05 18:56:48 UTC) #1
Tom Sepez
9 years, 9 months ago (2011-03-05 19:14:04 UTC) #2 
The ipc/ changes LGTM.  Thanks.

Powered by Google App Engine
This is Rietveld 408576698