Change memory dumps to not use periodic dumps by default.

Change memory dumps to not use periodic dumps by default.

All consumers that require periodic dumps now explicitly specify a dump config.
Remove by default the periodic dump config.

BUG=726393
Review-Url: https://codereview.chromium.org/2952083004
Cr-Commit-Position: refs/heads/master@{#481951}
Committed: https://chromium.googlesource.com/chromium/src/+/c72902d0a30ee1fc71e7890d14c9ca79c3d387b3

[erikchen, 2017-06-23 01:14:14]

Description was changed from

==========
Change memory dumps to not use periodic dumps by default.

All consumers that require periodic dumps now explicitly specify a dump config.
Remove by default the periodic dump config.

BUG=726393
==========

to

==========
Change memory dumps to not use periodic dumps by default.

All consumers that require periodic dumps now explicitly specify a dump config.
Remove by default the periodic dump config.

BUG=726393
==========

[erikchen, 2017-06-23 01:14:25]

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

[commit-bot: I haz the power, 2017-06-23 01:14:47]

Dry run: CQ is trying da patch.

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

[erikchen, 2017-06-23 01:15:17]

erikchen@chromium.org changed reviewers:
+ primiano@chromium.org, ssid@chromium.org

[erikchen, 2017-06-23 01:15:17]

primiano: Please review.

Compared to https://codereview.chromium.org/2951413002/, we no longer need to
modify content/browser/tracing/tracing_ui.cc since all consumers that want
periodic dumps now send...memory dump configs!

Magical.

https://codereview.chromium.org/2949423002/

[commit-bot: I haz the power, 2017-06-23 02:31:04]

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

[commit-bot: I haz the power, 2017-06-23 02:31:05]

Dry run: This issue passed the CQ dry run.

[Primiano Tucci (use gerrit), 2017-06-23 09:38:40]

LGTM assuming we land the UI changes first :)
Thanks for fixing all this.

[erikchen, 2017-06-23 17:47:00]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2017-06-23 17:47:23]

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-06-23 17:53:14]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498240020652500, "parent_rev":
"4731a33c05082442a2096eaa19ebb7662351559a", "commit_rev":
"c72902d0a30ee1fc71e7890d14c9ca79c3d387b3"}

[commit-bot: I haz the power, 2017-06-23 17:53:29]

Description was changed from

==========
Change memory dumps to not use periodic dumps by default.

All consumers that require periodic dumps now explicitly specify a dump config.
Remove by default the periodic dump config.

BUG=726393
==========

to

==========
Change memory dumps to not use periodic dumps by default.

All consumers that require periodic dumps now explicitly specify a dump config.
Remove by default the periodic dump config.

BUG=726393

Review-Url: https://codereview.chromium.org/2952083004
Cr-Commit-Position: refs/heads/master@{#481951}
Committed:
https://chromium.googlesource.com/chromium/src/+/c72902d0a30ee1fc71e7890d14c9...
==========

[commit-bot: I haz the power, 2017-06-23 17:53:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c72902d0a30ee1fc71e7890d14c9...