diff options
author | matthewyuan@chromium.org <matthewyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-11 21:02:45 +0000 |
---|---|---|
committer | matthewyuan@chromium.org <matthewyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-11 21:02:45 +0000 |
commit | 847494a0e6dcd4ca032ac96972f06ed0242dab20 (patch) | |
tree | c25df597bc82ef9ccf42778221f7cb235ce9f348 /chrome/VERSION | |
parent | d13e7238154a56d43a1a02187b07b0cda902b765 (diff) | |
download | chromium_src-847494a0e6dcd4ca032ac96972f06ed0242dab20.zip chromium_src-847494a0e6dcd4ca032ac96972f06ed0242dab20.tar.gz chromium_src-847494a0e6dcd4ca032ac96972f06ed0242dab20.tar.bz2 |
Merge 222436 "Attempt 2: makes BrowserStatusMonitor remove obser..."
> Attempt 2: makes BrowserStatusMonitor remove observers in destructor
>
> First patch was reverted. Need to destroy BrowserStatusMonitor early on in ChromeLauncherController as BrowserStatusMonitor calls back into ChromeLauncherController during destruction.
>
> Original description:
>
> I'm seeing a crash where we have a dangling TabStripModelObserver,
> this is one possible place. At the same time I'm making the class only
> look for HOST_DESKTOP_TYPE_ASH browsers.
>
> BUG=286162
> TEST=none
> R=skuhne@chromium.org
>
> Review URL: https://chromiumcodereview.appspot.com/23464053
TBR=sky@chromium.org
Review URL: https://codereview.chromium.org/23658038
git-svn-id: svn://svn.chromium.org/chrome/branches/1626/src@222628 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/VERSION')
0 files changed, 0 insertions, 0 deletions