Commit 33f85e07 authored by Eugene But's avatar Eugene But Committed by Commit Bot

[android] Set heap collection interval to 30 minutes

Currently HeapProfilerController uses 24 hours interval for Desktop and
Android. While 24h is good for Desktop, the interval is too big for
Android, which likely has median process up time of less than an hour.

iOS uses 30 minutes interval, which is equal to median process up time
duration.  30m would be more appropriate interval for Android.

Bug: 1090017
Change-Id: Ia8dfc09a75e036265298e3d69ad04fb8cc1b1124
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2225359
Commit-Queue: Erik Chen <erikchen@chromium.org>
Auto-Submit: Eugene But <eugenebut@chromium.org>
Reviewed-by: default avatarErik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#774370}
parent 61ec8f7c
......@@ -37,8 +37,9 @@ base::TimeDelta RandomInterval(base::TimeDelta mean) {
// - get from finch if available to allow experiment with different intervals
// - return default interval that is best suited for current OS
int GetCollectionIntervalInMinutes() {
#if defined(OS_IOS)
// Default on iOS is equal to mean value of up process time.
#if defined(OS_IOS) || defined(OS_ANDROID)
// Default on iOS is equal to mean value of up process time. Android is more
// similar to iOS than to Desktop.
const int kDefaultValueInMinutes = 30;
#else
const int kDefaultValueInMinutes = 24 * 60;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment