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

Issue 8879047: Fix crash during profile shutdown (Closed)

Created:
9 years ago by sail
Modified:
9 years ago
Reviewers:
Robert Sesek, SteveT
CC:
chromium-reviews
Visibility:
Public.

Description

Fix crash during profile shutdown With multiple profiles we would sometimes crash with the following backtrace: ProfileImpl::~ProfileImpl ProfileDependencyManager::DestroyProfileServices ... ProfileImp::InitSyncService() UpdateProfileUserNameCache profile_manager->GetProfileInfoCache( The problem was that at this point the profile manager and the profile sync service were both already deleted. Our lazy load of the profile sync service was causing us to create the service twice. In our other lazy load code we have a flag to prevent us from creating something twice. I added the same code to the profile sync service. BUG=106869 TEST=Reproduced the crash. Applied my fix and verified that the crash no longer reproduced. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113835

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
M chrome/browser/profiles/profile_impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/profiles/profile_impl.cc View 2 chunks +4 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
sail
9 years ago (2011-12-09 07:20:15 UTC) #1
SteveT
LGTM.
9 years ago (2011-12-09 15:52:59 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sail@chromium.org/8879047/1
9 years ago (2011-12-09 17:44:22 UTC) #3
commit-bot: I haz the power
9 years ago (2011-12-09 19:36:21 UTC) #4
Change committed as 113835

Powered by Google App Engine
This is Rietveld 408576698