Reland #2 "ipc: Add a new field num_brokered_attachments to the message header."
commita2e71be46dc4bdcbb544db479680f65a390ae8f3
authorerikchen <erikchen@chromium.org>
Wed, 16 Sep 2015 00:59:36 +0000 (15 17:59 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 16 Sep 2015 01:00:25 +0000 (16 01:00 +0000)
treefae1bab0f6557b8c34e149724af47c53787e8943
parent92aee1d33aa7fc8517daad0c14f14eb92ae2f835
Reland #2 "ipc: Add a new field num_brokered_attachments to the message header."

This original version of this CL is causing an unusual crash in Canary. This CL
adds a message verifier to the ipc code so that it can dynamically verify the
contents of this message that is being corrupted. This CL also verifies the
message at several different points in the dispatch process. This will help
narrow down the range of code that is corrupting the message.

I expect this CL to cause ~100 crashes in the next Chrome Canary. I intend to
revert this CL after a single Canary release.

BUG=527588

Review URL: https://codereview.chromium.org/1334593002

Cr-Commit-Position: refs/heads/master@{#349056}
14 files changed:
content/browser/browser_main_loop.cc
content/browser/loader/async_resource_handler.cc
content/child/child_thread_impl.cc
content/child/resource_dispatcher.cc
content/child/resource_scheduling_filter.cc
content/child/threaded_data_provider.cc
content/common/resource_messages.cc
content/common/resource_messages.h
ipc/ipc_channel.h
ipc/ipc_channel_common.cc
ipc/ipc_channel_reader.cc
ipc/ipc_channel_win.cc
ipc/ipc_message.cc
ipc/ipc_message.h