Commit cb36da07 authored by Michael Crouse's avatar Michael Crouse Committed by Commit Bot

Change exponential bucketing for Media data use UKM.

Privacy doc: https://docs.google.com/document/d/133SsQTd3L5ryQaI-q1rZsh6EeTsE_HxDFlthjlXi4CU/edit?ts=5f3c1ca0#

Bug: 1117645
Change-Id: Ifc3229de5e19cc78269cd6e6c3ae2d4ae1f1c11c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363693Reviewed-by: default avatarRyan Sturm <ryansturm@chromium.org>
Commit-Queue: Michael Crouse <mcrouse@chromium.org>
Auto-Submit: Michael Crouse <mcrouse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#799316}
parent 683ae383
...@@ -604,10 +604,10 @@ void UkmPageLoadMetricsObserver::RecordTimingMetrics( ...@@ -604,10 +604,10 @@ void UkmPageLoadMetricsObserver::RecordTimingMetrics(
ukm::GetExponentialBucketMin(js_max_decoded_bytes_, 10)); ukm::GetExponentialBucketMin(js_max_decoded_bytes_, 10));
builder.SetNet_ImageBytes( builder.SetNet_ImageBytes(
ukm::GetExponentialBucketMin(image_total_bytes_, 1.3)); ukm::GetExponentialBucketMin(image_total_bytes_, 1.15));
builder.SetNet_ImageSubframeBytes( builder.SetNet_ImageSubframeBytes(
ukm::GetExponentialBucketMin(image_subframe_bytes_, 1.3)); ukm::GetExponentialBucketMin(image_subframe_bytes_, 1.15));
builder.SetNet_MediaBytes(ukm::GetExponentialBucketMin(media_bytes_, 1.3)); builder.SetNet_MediaBytes(ukm::GetExponentialBucketMin(media_bytes_, 1.15));
if (main_frame_timing_) if (main_frame_timing_)
ReportMainResourceTimingMetrics(timing, &builder); ReportMainResourceTimingMetrics(timing, &builder);
......
...@@ -1351,13 +1351,13 @@ TEST_F(UkmPageLoadMetricsObserverTest, ImageMediaSizeMetrics) { ...@@ -1351,13 +1351,13 @@ TEST_F(UkmPageLoadMetricsObserverTest, ImageMediaSizeMetrics) {
// 30 KB for all images, 20 KB for subframe images, and 50 KB for media. // 30 KB for all images, 20 KB for subframe images, and 50 KB for media.
tester()->test_ukm_recorder().ExpectEntryMetric( tester()->test_ukm_recorder().ExpectEntryMetric(
kv.second.get(), "Net.ImageBytes", kv.second.get(), "Net.ImageBytes",
ukm::GetExponentialBucketMin(30 * 1024, 1.3)); ukm::GetExponentialBucketMin(30 * 1024, 1.15));
tester()->test_ukm_recorder().ExpectEntryMetric( tester()->test_ukm_recorder().ExpectEntryMetric(
kv.second.get(), "Net.ImageSubframeBytes", kv.second.get(), "Net.ImageSubframeBytes",
ukm::GetExponentialBucketMin(20 * 1024, 1.3)); ukm::GetExponentialBucketMin(20 * 1024, 1.15));
tester()->test_ukm_recorder().ExpectEntryMetric( tester()->test_ukm_recorder().ExpectEntryMetric(
kv.second.get(), "Net.MediaBytes", kv.second.get(), "Net.MediaBytes",
ukm::GetExponentialBucketMin(50 * 1024, 1.3)); ukm::GetExponentialBucketMin(50 * 1024, 1.15));
} }
} }
......
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