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

Issue 194095: Fix crash when closing tab with an open http auth sheet. (Closed)

Created:
11 years, 3 months ago by Nico
Modified:
9 years, 7 months ago
Reviewers:
Avi (use Gerrit)
CC:
chromium-reviews_googlegroups.com, viettrungluu, Mike Mammarella
Visibility:
Public.

Description

Fix crash when closing tab with an open http auth sheet. Parts of patch by mdm@chromium.org, taken from http://codereview.chromium.org/196104 BUG=19427, 21111, 21304 (not sure about last one) TEST=Open a page with an http auth sheet, open another tab, close tab with the open sheet -- should not crash. Also check that opening a tab with an open sheet, then hitting back, forward, reload still works and doesn't crash. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=26105

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -5 lines) Patch
M chrome/browser/login_prompt_mac.mm View 3 chunks +10 lines, -5 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Nico
11 years, 3 months ago (2009-09-12 07:11:50 UTC) #1
Avi (use Gerrit)
11 years, 3 months ago (2009-09-14 14:05:51 UTC) #2
LG

Powered by Google App Engine
This is Rietveld 408576698