Commit 5cfd9d9d authored by Michael Thiessen's avatar Michael Thiessen Committed by Commit Bot

Add bug # to GURL pre-native usage trace to help sheriffs.

Bug: 1065377
Change-Id: Id51e30bb496513a30d639fc62537ffd525f10da8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2326450Reviewed-by: default avatarTheresa  <twellington@chromium.org>
Reviewed-by: default avatarCharlie Harrison <csharrison@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#793602}
parent 7029263c
......@@ -84,9 +84,6 @@ public class GURL {
/**
* Enables debug stack trace gathering for GURL.
*
* TODO(https://crbug.com/783819): Remove this when the the fraction of users hitting this
* drops.
*/
public static void setReportDebugThrowableCallback(ReportDebugThrowableCallback callback) {
sReportCallback = callback;
......@@ -107,7 +104,8 @@ public class GURL {
RecordHistogram.recordTimesHistogram("Startup.Android.GURLEnsureMainDexInitialized",
SystemClock.elapsedRealtime() - time);
if (sReportCallback != null && new Random().nextInt(100) < DEBUG_REPORT_PERCENTAGE) {
final Throwable throwable = new Throwable("This is not a crash, please ignore.");
final Throwable throwable =
new Throwable("This is not a crash, please ignore. See crbug.com/1065377.");
// This isn't an assert, because by design this is possible, but we would prefer
// this path does not get hit more than necessary and getting stack traces from the
// wild will help find issues.
......
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