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

Issue 99195: Fix for in-progress download confirmation dialog (Closed)

Created:
11 years, 7 months ago by jcampan
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Closing the last tab with a download in-progress would cause the tab to be closed and become unusable if the user decides not to proceed with the browser shutdown. This is because we check for in-progress downloads when the browser is closed, and the tab is closed before that, leaving the tab-strip in a bad state. This CL ensures we also bring-up the confirmation dialog when the last tab is closed. BUG=10680 TEST=Start downloading a big file. While the file is downloading, close all tabs. When closing the last tab, the in-progress download dialog should be shown. Select the 'Wait for downloads', the download tab should be shown and the previous tab should still be displayed and functional. Close all tabs again, this time select the 'Close and cancel downloads' option, the browser should be closed. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=14948

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Patch Set 5 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -8 lines) Patch
M chrome/browser/browser.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/browser.cc View 1 1 chunk +9 lines, -0 lines 0 comments Download
M chrome/browser/tabs/tab_strip_model.h View 1 2 3 4 5 chunks +10 lines, -6 lines 0 comments Download
M chrome/browser/tabs/tab_strip_model.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/tabs/tab_strip_model_unittest.cc View 3 chunks +16 lines, -2 lines 0 comments Download
M chrome/browser/views/tabs/tab_strip.cc View 1 2 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jcampan
11 years, 7 months ago (2009-04-29 19:09:06 UTC) #1
jcampan
11 years, 7 months ago (2009-04-29 20:51:12 UTC) #2
Ben Goodger (Google)
11 years, 7 months ago (2009-04-30 06:38:44 UTC) #3
sorry got lost in the shuffle

meant to say LGTM earlier

Powered by Google App Engine
This is Rietveld 408576698