diff options
author | petrcermak <petrcermak@chromium.org> | 2015-09-24 01:28:48 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-09-24 08:29:37 +0000 |
commit | 29bd40509cd5f705d686a97f2c91e3a75501cf2a (patch) | |
tree | 104583329605d7f84f852872fa055045a0ae5e56 /ipc | |
parent | 2fbfee5256df9abdbdccfef120fef9b43e4f85bb (diff) | |
download | chromium_src-29bd40509cd5f705d686a97f2c91e3a75501cf2a.zip chromium_src-29bd40509cd5f705d686a97f2c91e3a75501cf2a.tar.gz chromium_src-29bd40509cd5f705d686a97f2c91e3a75501cf2a.tar.bz2 |
Re-land architecture for cross-process memory notification suppressing
This patch re-lands https://codereview.chromium.org/1332583002/ after
it was reverted in https://codereview.chromium.org/1359873002/ because
it was causing failures on the Cast Linux buildbot.
The failures were caused by std::set operations being carried out
inside DCHECK macros, which are not evaluated on release builds:
DCHECK(memory_message_filters_.insert(filter).second);
This patch moves the std::set operations outside the macros so that
they would always be evaluated:
const bool success = memory_message_filters_.insert(filter).second;
DCHECK(success);
BUG=516776
Review URL: https://codereview.chromium.org/1361963002
Cr-Commit-Position: refs/heads/master@{#350503}
Diffstat (limited to 'ipc')
-rw-r--r-- | ipc/ipc_message_start.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/ipc/ipc_message_start.h b/ipc/ipc_message_start.h index 718bfa7..bf93b94 100644 --- a/ipc/ipc_message_start.h +++ b/ipc/ipc_message_start.h @@ -126,6 +126,7 @@ enum IPCMessageStart { AttachmentBrokerMsgStart, RenderProcessMsgStart, PageLoadMetricsMsgStart, + MemoryMsgStart, LastIPCMsgStart // Must come last. }; |