android: Extend Startup.Android.Cold.TimeToFirstContentfulPaint3
Extend the lifetime of Startup.Android.Cold.TimeToFirstContentfulPaint3. The histogram as it is mentioned in the histograms.xml has no data on the UMA dashboard because the collected histogram adds the suffix ".Tabbed". Register this suffix in the histogram_suffixes_list to make it searchable in the dashboard. Mark the histogram without the suffix as base="true" to make it non-searchable (to avoid confusions). Adjust the doc and the test accordingly. Bug: 40926074, 384509264 Change-Id: Id7653a49fd8fff67df5fd046011b2d233d93b779 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6110414 Owners-Override: Nafis Abedin <nafisabedin@google.com> Reviewed-by: Andrew Grieve <agrieve@chromium.org> Commit-Queue: Egor Pasko <pasko@chromium.org> Cr-Commit-Position: refs/heads/main@{#1398663}
This commit is contained in:

committed by
Chromium LUCI CQ

parent
4e409e485d
commit
b9941a6100
chrome/android/javatests/src/org/chromium/chrome/browser/metrics
docs/speed/startup
tools/metrics/histograms/metadata
@ -167,7 +167,8 @@ public class StartupLoadingMetricsTest {
|
||||
Assert.assertTrue(coldStartFirstCommit4Samples < 2);
|
||||
|
||||
int coldStartFirstContentfulPaintSamples =
|
||||
RecordHistogram.getHistogramTotalCountForTesting(FIRST_CONTENTFUL_PAINT_HISTOGRAM3);
|
||||
RecordHistogram.getHistogramTotalCountForTesting(
|
||||
FIRST_CONTENTFUL_PAINT_HISTOGRAM3 + histogramSuffix);
|
||||
Assert.assertTrue(coldStartFirstContentfulPaintSamples < 2);
|
||||
|
||||
int firstCommitSamples =
|
||||
|
@ -26,7 +26,7 @@ Here are some UMA metrics that measure start-up (UMA's UI has descriptions):
|
||||
* `Startup.Android.Cold.TimeToFirstVisibleContent2`
|
||||
* `Startup.Android.Cold.TimeToFirstNavigationCommit2.Tabbed`
|
||||
* `Browser.PaintPreview.TabbedPlayer.TimeToFirstBitmap`
|
||||
* `Startup.Android.Cold.TimeToFirstContentfulPaint.Tabbed`
|
||||
* `Startup.Android.Cold.TimeToFirstContentfulPaint3.Tabbed`
|
||||
|
||||
This one can be useful for measuring dex optimization changes:
|
||||
|
||||
|
@ -55,6 +55,7 @@ chromium-metrics-reviews@google.com.
|
||||
label="Chrome starts up with intent to ChromeTabbedActivity"/>
|
||||
<suffix name="WebApk" label="Chrome starts up with intent to WebApkActivity"/>
|
||||
<affected-histogram name="Startup.Android.Cold.TimeToFirstContentfulPaint"/>
|
||||
<affected-histogram name="Startup.Android.Cold.TimeToFirstContentfulPaint3"/>
|
||||
<affected-histogram name="Startup.Android.Cold.TimeToFirstNavigationCommit"/>
|
||||
</histogram_suffixes>
|
||||
|
||||
|
@ -116,8 +116,8 @@ chromium-metrics-reviews@google.com.
|
||||
</summary>
|
||||
</histogram>
|
||||
|
||||
<histogram name="Startup.Android.Cold.TimeToFirstContentfulPaint3" units="ms"
|
||||
expires_after="2025-01-26">
|
||||
<histogram base="true" name="Startup.Android.Cold.TimeToFirstContentfulPaint3"
|
||||
units="ms" expires_after="2025-07-26">
|
||||
<owner>pasko@chromium.org</owner>
|
||||
<owner>nafisabedin@google.com</owner>
|
||||
<summary>
|
||||
|
Reference in New Issue
Block a user