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

Issue 59873011: Add threads stack dumping feature (through SIGUSR1) to test_runner.py. (Closed)

Created:
7 years, 1 month ago by Philippe
Modified:
7 years, 1 month ago
Reviewers:
bulach
CC:
chromium-reviews, craigdh+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org, klundberg+watch_chromium.org, frankf+watch_chromium.org
Visibility:
Public.

Description

Add threads stack dumping feature (through SIGUSR1) to test_runner.py. BUG=320445 R=bulach@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=235996

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -0 lines) Patch
M build/android/test_runner.py View 2 chunks +19 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Philippe
Hey Marcus, I would like to land this to investigate the issue on the bot.
7 years, 1 month ago (2013-11-19 14:06:15 UTC) #1
bulach
lgtm, great idea, go for it! :)
7 years, 1 month ago (2013-11-19 14:18:44 UTC) #2
Philippe
On 2013/11/19 14:18:44, bulach wrote: > lgtm, great idea, go for it! :) Thanks Marcus!
7 years, 1 month ago (2013-11-19 14:22:36 UTC) #3
Philippe
On 2013/11/19 14:22:36, Philippe wrote: > On 2013/11/19 14:18:44, bulach wrote: > > lgtm, great ...
7 years, 1 month ago (2013-11-19 14:59:41 UTC) #4
Philippe
Committed patchset #1 manually as r235996 (presubmit successful).
7 years, 1 month ago (2013-11-19 15:02:28 UTC) #5
frankf
On 2013/11/19 15:02:28, Philippe wrote: > Committed patchset #1 manually as r235996 (presubmit successful). pliard, ...
7 years, 1 month ago (2013-11-19 18:27:01 UTC) #6
craigdh
On 2013/11/19 18:27:01, frankf wrote: > On 2013/11/19 15:02:28, Philippe wrote: > > Committed patchset ...
7 years, 1 month ago (2013-11-19 19:50:40 UTC) #7
Philippe
7 years, 1 month ago (2013-11-20 08:24:50 UTC) #8
Yeah I had seen this reraiser thread but it was not immediately clear to
me. Thanks for cleaning this up!


On Tue, Nov 19, 2013 at 8:50 PM, <craigdh@chromium.org> wrote:

> On 2013/11/19 18:27:01, frankf wrote:
>
>> On 2013/11/19 15:02:28, Philippe wrote:
>> > Committed patchset #1 manually as r235996 (presubmit successful).
>>
>
>  pliard, we already have such functionality. Take a look at
>> pylib/utils/reraiser_thread.py and how it's used.
>>
>
> See http://crrev.com/76233005
>
> https://chromiumcodereview.appspot.com/59873011/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698