New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 687247 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

DCHECK on browser shutdown in content::ServiceManagerConnectionImpl::IOThreadContext::RemoveConnectionFilterOnIOThread

Project Member Reported by jamescook@chromium.org, Jan 31 2017

Issue description

Chrome ToT r447004
Linux build with target_os="chromeos"
chrome --mash

Sometimes when content_browser shuts down we get the DCHECK below. It happens pretty much every time when I shut down the service manager in the root process in https://codereview.chromium.org/2646033002/

[2891:2911:0131/100904.918877:FATAL:service_manager_connection_impl.cc(217)] Check failed: it != connection_filters_.end(). 
#0 0x7fa2ee70ac9b base::debug::StackTrace::StackTrace()
#1 0x7fa2ee70931c base::debug::StackTrace::StackTrace()
#2 0x7fa2ee76f98c logging::LogMessage::~LogMessage()
#3 0x7fa2e7733234 content::ServiceManagerConnectionImpl::IOThreadContext::RemoveConnectionFilterOnIOThread()
#4 0x7fa2e772516e _ZN4base8internal13FunctorTraitsIMN7content21EmbeddedServiceRunner15InstanceManagerEFviEvE6InvokeIRK13scoped_refptrIS4_EJRKiEEEvS6_OT_DpOT0_
#5 0x7fa2e77250a6 _ZN4base8internal12InvokeHelperILb0EvE8MakeItSoIRKMN7content21EmbeddedServiceRunner15InstanceManagerEFviEJRK13scoped_refptrIS6_ERKiEEEvOT_DpOT0_
#6 0x7fa2e7733463 _ZN4base8internal7InvokerINS0_9BindStateIMN7content28ServiceManagerConnectionImpl15IOThreadContextEFviEJ13scoped_refptrIS5_EiEEEFvvEE7RunImplIRKS7_RKSt5tupleIJS9_iEEJLm0ELm1EEEEvOT_OT0_NS_13IndexSequenceIJXspT1_EEEE
#7 0x7fa2e773337c _ZN4base8internal7InvokerINS0_9BindStateIMN7content28ServiceManagerConnectionImpl15IOThreadContextEFviEJ13scoped_refptrIS5_EiEEEFvvEE3RunEPNS0_13BindStateBaseE
#8 0x7fa2ee710201 _ZNO4base8internal8RunMixinINS_8CallbackIFvvELNS0_8CopyModeE0ELNS0_10RepeatModeE0EEEE3RunEv
#9 0x7fa2ee70fc09 base::debug::TaskAnnotator::RunTask()
#10 0x7fa2ee796aa3 base::MessageLoop::RunTask()
#11 0x7fa2ee796d04 base::MessageLoop::DeferOrRunPendingTask()
#12 0x7fa2ee796fee base::MessageLoop::DoWork()
#13 0x7fa2ee7ace1c base::MessagePumpLibevent::Run()
#14 0x7fa2ee796692 base::MessageLoop::RunHandler()
#15 0x7fa2ee836394 base::RunLoop::Run()
#16 0x7fa2ee8d3612 base::Thread::Run()
#17 0x7fa2e7eca858 content::BrowserThreadImpl::IOThreadRun()
#18 0x7fa2e7ecaaf1 content::BrowserThreadImpl::Run()
#19 0x7fa2ee8d3e03 base::Thread::ThreadMain()
#20 0x7fa2ee8bc57a base::(anonymous namespace)::ThreadFunc()
#21 0x7fa2eeb37184 start_thread
#22 0x7fa2d8a7f37d clone

(This might be dupe.)

 
I filed the bug so I can put a bug number in a TODO, btw.

Cc: roc...@chromium.org
Owner: jamescook@chromium.org
Over to you james, since you found it and have the fix :)
Status: Fixed (was: Assigned)
This was also causing a shutdown hang in content_browser.

Fixed by https://codereview.chromium.org/2690453002/ which just landed.

Comment 4 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 5 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 6 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 7 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Comment 8 by laforge@google.com, Nov 1 2017

Components: -Internals>ServiceManager Internals>Services>ServiceManager

Sign in to add a comment