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

Issue 23494022: Change active root window before closing old root windows' children. (Closed)

Created:
7 years, 3 months ago by flackr
Modified:
7 years, 3 months ago
Reviewers:
James Cook, oshima
CC:
chromium-reviews, sadrul, ben+watch_chromium.org, oshima
Visibility:
Public.

Description

Change active root window before closing old root windows' children. BUG=282582 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221497

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -1 line) Patch
M ash/root_window_controller.cc View 1 chunk +6 lines, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
flackr
PTAL, this should fix the crash mentioned in the bug. The circumstances under which the ...
7 years, 3 months ago (2013-09-05 13:13:49 UTC) #1
James Cook
LGTM +oshima just as FYI
7 years, 3 months ago (2013-09-05 15:47:46 UTC) #2
oshima
lgtm too
7 years, 3 months ago (2013-09-05 16:34:35 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/flackr@chromium.org/23494022/1
7 years, 3 months ago (2013-09-05 17:23:10 UTC) #4
commit-bot: I haz the power
7 years, 3 months ago (2013-09-05 20:01:44 UTC) #5
Message was sent while issue was closed.
Change committed as 221497

Powered by Google App Engine
This is Rietveld 408576698