memory-infra: Disable two MemoryDumpManager tests on iOS

memory-infra: Disable two MemoryDumpManager tests on iOS

The tests seem to be flaky and we are about to restructure the
MemoryDumpManager to not know about tracing so it doesn't seem
worth hunting down the cause.

R=primiano@chromium.org
BUG=706961
Review-Url: https://codereview.chromium.org/2787823004
Cr-Commit-Position: refs/heads/master@{#461093}
Committed: https://chromium.googlesource.com/chromium/src/+/9770fad0b3ebe389c34672dc8797bf8fbffeda9b

[hjd, 2017-03-31 09:41:39]

The CQ bit was checked by hjd@chromium.org to run a CQ dry run

[hjd, 2017-03-31 09:41:49]

ptal, thanks!

[commit-bot: I haz the power, 2017-03-31 09:41:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-31 09:53:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-31 09:53:41]

Dry run: This issue passed the CQ dry run.

[Primiano Tucci (use gerrit), 2017-03-31 09:59:50]

LGTM with one comment, thanks

https://codereview.chromium.org/2787823004/diff/20001/base/trace_event/memory...
File base/trace_event/memory_dump_manager_unittest.cc (right):

https://codereview.chromium.org/2787823004/diff/20001/base/trace_event/memory...
base/trace_event/memory_dump_manager_unittest.cc:35: #include
"testing/gmock/include/gmock/gmock.h"
plz ad an explicit include to build/build_config.h or th fact that  OS_* is
defined will depend  on whether any of the include above includes build_config

[hjd, 2017-03-31 10:24:21]

The CQ bit was checked by hjd@chromium.org

[hjd, 2017-03-31 10:24:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
primiano@chromium.org
Link to the patchset: https://codereview.chromium.org/2787823004/#ps40001
(title: "add a missing include")

[commit-bot: I haz the power, 2017-03-31 10:24:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-31 10:36:31]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490955861605460,
"parent_rev": "511fde94942ca59c7ac26c0a40e237251be45d8b", "commit_rev":
"9770fad0b3ebe389c34672dc8797bf8fbffeda9b"}

[commit-bot: I haz the power, 2017-03-31 10:37:22]

Description was changed from

==========
memory-infra: Disable two MemoryDumpManager tests on iOS

The tests seem to be flaky and we are about to restructure the
MemoryDumpManager to not know about tracing so it doesn't seem
worth hunting down the cause.

R=primiano@chromium.org
BUG=706961
==========

to

==========
memory-infra: Disable two MemoryDumpManager tests on iOS

The tests seem to be flaky and we are about to restructure the
MemoryDumpManager to not know about tracing so it doesn't seem
worth hunting down the cause.

R=primiano@chromium.org
BUG=706961

Review-Url: https://codereview.chromium.org/2787823004
Cr-Commit-Position: refs/heads/master@{#461093}
Committed:
https://chromium.googlesource.com/chromium/src/+/9770fad0b3ebe389c34672dc8797...
==========

[commit-bot: I haz the power, 2017-03-31 10:37:23]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/9770fad0b3ebe389c34672dc8797...