New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 740484 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: 2017-08-11
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Lower minidump crash reporting fraction in time for 61 Stable.

Project Member Reported by gsennton@chromium.org, Jul 10 2017

Issue description

We did this for 60 in crbug.com/725074

For 58 and 59 we lowered the crash reporting rate to 0.1% before Stable, for 60 we lowered it to 1% after noticing that we had some slack in 59.
 
The NextAction date has arrived: 2017-08-07
NextAction: 2017-08-11
The NextAction date has arrived: 2017-08-11

Comment 4 by cma...@chromium.org, Aug 18 2017

gsennton@ I think this has been done already right?
The CL is up:
https://chromium-review.googlesource.com/c/612090
but I won't land it until next week because we want as much data from Beta releases as possible :)
Let's just make sure that if our final beta is good we can use it without having to respin to take this patch per normal process.
Cc: -amineer@chromium.org cma...@chromium.org
Handing off to cmasso@.
Alright, I'm gonna land this on Friday unless there are any objections :)

Comment 9 by cma...@chromium.org, Aug 22 2017

Sounds good!
Project Member

Comment 10 by bugdroid1@chromium.org, Aug 25 2017

Labels: merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/69288437f43fc0a39a9d5142a190fee4900f958e

commit 69288437f43fc0a39a9d5142a190fee4900f958e
Author: Gustav Sennton <gsennton@google.com>
Date: Fri Aug 25 10:32:07 2017

[Android WebView] Lower minidump generation fraction to 1% for Stable

To avoid hitting the cap on the number of uploaded crash reports per day
we lower the fraction of uploaded reports from 100% for Beta to 1% for
Stable.
Note that there should be at least one Beta version with the new lowered
fraction since we do not want to re-push Beta just for this change.

BUG= 740484 
NOTRY=true

Change-Id: I58a8394ab1cdcdab4fd8a72e38cfef50cdfa6a00
Reviewed-on: https://chromium-review.googlesource.com/612090
Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Reviewed-by: Gustav Sennton <gsennton@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#887}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/69288437f43fc0a39a9d5142a190fee4900f958e/android_webview/common/crash_reporter/aw_microdump_crash_reporter.cc

Status: Fixed (was: Assigned)

Sign in to add a comment