New issue
Advanced search Search tips

Issue 622314 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug

Blocking:
issue 6010
issue 478209



Sign in to add a comment

The out of process browser hang instrumentation should use the Crashpad reporter

Project Member Reported by manzagop@chromium.org, Jun 22 2016

Issue description

The out of process browser hang instrumentation currently uses the Kasko reporter, which is deprecated. It should use the Crashpad reporter instead.
 
Cc: scottmg@chromium.org
Temporarily removed the instrumentation to get rid of the Kasko reporter:
https://chromiumcodereview.appspot.com/2086403002/

(Not sure why the CL didn't show up here)
Blocking: 6010
Owner: ----
Status: Available (was: Assigned)
Didn't get to this. Focus has been on https://crbug.com/620813
Project Member

Comment 5 by sheriffbot@chromium.org, Jul 2

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: siggi@chromium.org
Owner: brucedaw...@chromium.org
Status: Assigned (was: Untriaged)
Seems similar to issue 643176, though I thought Kasko was already gone. Maybe this bug is meaningless now? Siggi: do you know?
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 17

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/97ba83ba5c7605c091965c1033983c234c9df97e

commit 97ba83ba5c7605c091965c1033983c234c9df97e
Author: Greg Thompson <grt@chromium.org>
Date: Wed Oct 17 06:23:11 2018

Remove base/win/wait_chain*.

This code has been unused since https://crrev.com/402189.

BUG=622314
R=pmonette@chromium.org

Change-Id: I375cfc911f0c05753476a58f3f5c9dff72b5f181
Reviewed-on: https://chromium-review.googlesource.com/c/1283070
Reviewed-by: Patrick Monette <pmonette@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#600299}
[modify] https://crrev.com/97ba83ba5c7605c091965c1033983c234c9df97e/base/BUILD.gn
[delete] https://crrev.com/6b616844c61448e9786b24187c1d4c513bb84c51/base/win/wait_chain.cc
[delete] https://crrev.com/6b616844c61448e9786b24187c1d4c513bb84c51/base/win/wait_chain.h
[delete] https://crrev.com/6b616844c61448e9786b24187c1d4c513bb84c51/base/win/wait_chain_unittest.cc

Sign in to add a comment