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

Issue 465007: Unload the entire extension when any part of it crashes. (Closed)

Created:
11 years ago by Paweł Hajdan Jr.
Modified:
9 years, 7 months ago
Reviewers:
Aaron Boodman, jcampan
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Unload the entire extension when any part of it crashes. Also enable the task_manager_browsertest.cc (where I added the new test) on Linux. TEST=Covered by browser_tests. BUG=21635 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=33599

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+35 lines, -3 lines) Patch
M chrome/browser/extensions/extensions_service.cc View 2 chunks +12 lines, -0 lines 0 comments Download
M chrome/browser/task_manager_browsertest.cc View 1 chunk +21 lines, -0 lines 0 comments Download
M chrome/chrome.gyp View 3 chunks +2 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Paweł Hajdan Jr.
Hopefully no crashes this time.
11 years ago (2009-12-02 17:26:32 UTC) #1
Aaron Boodman
lgtm
11 years ago (2009-12-02 19:22:58 UTC) #2
jcampan
11 years ago (2009-12-02 19:43:24 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698