Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(203)

Issue 155357: Add 16276 to crash known list. (Closed)

Created:
11 years, 5 months ago by huanr
Modified:
9 years ago
Reviewers:
eroman
CC:
chromium-reviews_googlegroups.com, Kasper Lund
Visibility:
Public.

Description

Add 16276 to crash known list. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=20390

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M chrome/test/data/reliability/known_crashes.txt View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
huanr
11 years, 5 months ago (2009-07-10 18:22:47 UTC) #1
eroman
LGTM
11 years, 5 months ago (2009-07-10 18:23:28 UTC) #2
Kasper Lund
11 years, 5 months ago (2009-07-14 08:53:24 UTC) #3
Huan,

Thanks a lot for adding this. Do we have a policy on when we remove
such entries again? I believe I have (finally) fixed the issue (in
r20485), but this is a rather delicate area of the V8 system, and the
lookup routines are used a lot, so I'd really like to get feedback
from ChromeBot as early as possible.

Unless it makes you feel uneasy, I propose we remove the entry as soon
as possible.

Cheers,
Kasper

On Fri, Jul 10, 2009 at 8:23 PM, <eroman@chromium.org> wrote:
> LGTM
>
> http://codereview.chromium.org/155357
>

Powered by Google App Engine
This is Rietveld 408576698