Commit e4e06e29 authored by Michael Lippautz's avatar Michael Lippautz Committed by Commit Bot

heap: Fix BlinkGC.MainThreadMarkingThroughput

This histogram should be computed from foreground marking time only.

Bug: 986235
Change-Id: I9ca3471a7fee2547ac1a89dcae2c9515b6e173a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2074878
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Commit-Queue: Omer Katz <omerkatz@chromium.org>
Auto-Submit: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: default avatarOmer Katz <omerkatz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744684}
parent fb0b87c1
...@@ -953,7 +953,7 @@ void UpdateHistograms(const ThreadHeapStatsCollector::Event& event) { ...@@ -953,7 +953,7 @@ void UpdateHistograms(const ThreadHeapStatsCollector::Event& event) {
"BlinkGC.TimeForGlobalWeakProcessing", "BlinkGC.TimeForGlobalWeakProcessing",
event.scope_data[ThreadHeapStatsCollector::kMarkWeakProcessing]); event.scope_data[ThreadHeapStatsCollector::kMarkWeakProcessing]);
base::TimeDelta marking_duration = event.marking_time(); base::TimeDelta marking_duration = event.foreground_marking_time();
constexpr size_t kMinObjectSizeForReportingThroughput = 1024 * 1024; constexpr size_t kMinObjectSizeForReportingThroughput = 1024 * 1024;
if (base::TimeTicks::IsHighResolution() && if (base::TimeTicks::IsHighResolution() &&
(event.object_size_in_bytes_before_sweeping > (event.object_size_in_bytes_before_sweeping >
......
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