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

Issue 309573004: aura: Reland r266681 to debug WindowObserver crashes. (Closed)

Created:
6 years, 6 months ago by Daniel Erat
Modified:
6 years, 6 months ago
Reviewers:
flackr, sky
CC:
chromium-reviews, sadrul, ben+aura_chromium.org, kalyank
Visibility:
Public.

Description

aura: Reland r266681 to debug WindowObserver crashes. Add code to make the aura::WindowObserver interface crash when it's destroyed without having unregistered its interest in all aura::Windows that it was observing. This will hopefully help track down the source of crashes that are seen when closing video windows on Google Play. https://codereview.chromium.org/303163003/ hopefully fixes the test crashes that resulted in this change being reverted earlier. BUG=365364 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274086

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -2 lines) Patch
M ui/aura/aura.gyp View 1 chunk +1 line, -0 lines 0 comments Download
M ui/aura/window.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M ui/aura/window_observer.h View 2 chunks +18 lines, -1 line 0 comments Download
A ui/aura/window_observer.cc View 1 chunk +31 lines, -0 lines 1 comment Download
M ui/aura/window_unittest.cc View 2 chunks +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Daniel Erat
This is dependent on https://codereview.chromium.org/303163003/ going in first.
6 years, 6 months ago (2014-05-29 23:46:26 UTC) #1
flackr
lgtm https://codereview.chromium.org/309573004/diff/1/ui/aura/window_observer.cc File ui/aura/window_observer.cc (right): https://codereview.chromium.org/309573004/diff/1/ui/aura/window_observer.cc#newcode1 ui/aura/window_observer.cc:1: // Copyright (c) 2013 The Chromium Authors. All ...
6 years, 6 months ago (2014-05-30 00:12:22 UTC) #2
sky
LGTM
6 years, 6 months ago (2014-05-30 02:34:34 UTC) #3
Daniel Erat
The CQ bit was checked by derat@chromium.org
6 years, 6 months ago (2014-05-31 04:24:56 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/derat@chromium.org/309573004/1
6 years, 6 months ago (2014-05-31 04:28:40 UTC) #5
commit-bot: I haz the power
6 years, 6 months ago (2014-05-31 22:55:55 UTC) #6
Message was sent while issue was closed.
Change committed as 274086

Powered by Google App Engine
This is Rietveld 408576698