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 /content/content_child.gypi | |
| 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 'content/content_child.gypi')
| -rw-r--r-- | content/content_child.gypi | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/content/content_child.gypi b/content/content_child.gypi index 5ad5b67..42587bd 100644 --- a/content/content_child.gypi +++ b/content/content_child.gypi @@ -114,6 +114,8 @@ 'child/indexed_db/webidbdatabase_impl.h', 'child/indexed_db/webidbfactory_impl.cc', 'child/indexed_db/webidbfactory_impl.h', + 'child/memory/child_memory_message_filter.cc', + 'child/memory/child_memory_message_filter.h', 'child/mojo/mojo_application.cc', 'child/mojo/mojo_application.h', 'child/multipart_response_delegate.cc', |