Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in

Issue 3052043: Fix crasher with download bar. (Closed)

8 years, 5 months ago by Jay Civelli
7 years, 8 months ago
chromium-reviews, Paul Godavari,
Base URL:


Fix crasher with download bar. When dragging out a file, the download bar is shown and disappear automatically once the download is complete. If the download item menu was opened at that point, we would crash. This was because showing the menu runs an inner message loop that would process the hide for the download bar, leading to the download item that was on the call-stack to be deleted, causing a crasher once the stack-unwinded and the deleted object was accessed. BUG=51187 TEST=In GMail, open an email with a large attachment (like 2 MB). Drag the file from GMail to your desktop. Click on the arrow on the item on the download bar to bring up the download menu. Keep the menu open until the download is completed and the bar gets hidden. It should not crash. Committed:

Patch Set 1 #

Total comments: 1

Patch Set 2 : Nulling deleted_ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -1 line) Patch
M chrome/browser/views/download_item_view.h View 1 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/browser/views/ View 1 3 chunks +11 lines, -1 line 0 comments Download


Total messages: 3 (0 generated)
Jay Civelli
8 years, 5 months ago (2010-08-06 23:28:28 UTC) #1
LGTM to with the following change. File chrome/browser/views/ (right): chrome/browser/views/ Set deleted_ to ...
8 years, 5 months ago (2010-08-06 23:36:55 UTC) #2
Jay Civelli
8 years, 5 months ago (2010-08-07 00:28:40 UTC) #3
> chrome/browser/views/
> Set deleted_ to NULL here.
Oops! Thanks for catching this!

Powered by Google App Engine
This is Rietveld 408576698