DescriptionReland of Change memory dumps to not use periodic dumps by default. (patchset #1 id:1 of https://codereview.chromium.org/2952693002/ )
Reason for revert:
Relanding the original CL.
Periodic dumping on Android should be fixed:
https://codereview.chromium.org/2948023002/
Original issue's description:
> Revert of Change memory dumps to not use periodic dumps by default. (patchset #3 id:40001 of https://codereview.chromium.org/2912483002/ )
>
> Reason for revert:
> This breaks periodic dumping on Android. Reverting.
>
> Original issue's description:
> > Change memory dumps to not use periodic dumps by default.
> >
> > The only consumer that requires periodic dumps [and doesn't specify an explicit
> > dump config] is Chrome tracing, so explicitly specify a periodic dump config
> > there. Remove by default the periodic dump config.
> >
> > BUG=726393
> >
> > Review-Url: https://codereview.chromium.org/2912483002
> > Cr-Commit-Position: refs/heads/master@{#475608}
> > Committed: https://chromium.googlesource.com/chromium/src/+/0529413d84c06f3720bf4a56b28bb2c7358cefd1
>
> TBR=primiano@chromium.org,ssid@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=726393
>
> Review-Url: https://codereview.chromium.org/2952693002
> Cr-Commit-Position: refs/heads/master@{#480941}
> Committed: https://chromium.googlesource.com/chromium/src/+/8b0b2a16a1e141102620e7d173271197cd6b7f3d
TBR=primiano@chromium.org,ssid@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=726393
Patch Set 1 #
Messages
Total messages: 7 (3 generated)
|