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

Issue 895467 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 894852
Owner: ----
Closed: Oct 16
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

DCHECK when disabling Wallet during initial Sync setup

Project Member Reported by treib@chromium.org, Oct 15

Issue description

Repro steps:
- Fresh profile.
- Sign in.
- In the Sync confirmation dialog, click "Settings".
- Turn off "Use sync and all services" (so that the data type toggles become available).
- Disable "Payment methods ...". (Also happens when disabling "Autofill" since that includes payments.)

Stack trace:
[52528:52528:1015/173943.189585:FATAL:model_association_manager.cc(194)] Check failed: error.IsSet() || (dtc->state() != DataTypeController::NOT_RUNNING). 
#0 0x7f72b47dd64f base::debug::StackTrace::StackTrace()
#1 0x7f72b4706f1b logging::LogMessage::~LogMessage()
#2 0x561459b78435 syncer::ModelAssociationManager::StopDatatypeImpl()
#3 0x561459b7853e syncer::ModelAssociationManager::StopDatatype()
#4 0x561459b70275 syncer::DataTypeManagerImpl::ReadyForStartChanged()
#5 0x561459ba14c1 browser_sync::ProfileSyncService::ReadyForStartChanged()
#6 0x561459b6eaaf browser_sync::AutofillWalletModelTypeController::OnUserPrefChanged()
#7 0x7f72af7c4dec PrefChangeRegistrar::InvokeUnnamedCallback()
#8 0x7f72af7c50c2 PrefChangeRegistrar::OnPreferenceChanged()
#9 0x7f72af7c7ef9 PrefNotifierImpl::FireObservers()
#10 0x7f72af7d060a PrefValueStore::PrefStoreKeeper::OnPrefValueChanged()
#11 0x561458900104 SegregatedPrefStore::AggregatingObserver::OnPrefValueChanged()
#12 0x7f72af7bf614 JsonPrefStore::ReportValueChanged()
#13 0x7f72af7be1dd JsonPrefStore::SetValue()
#14 0x561458900ca3 SegregatedPrefStore::SetValue()
#15 0x7f72af7cd7a4 PrefService::SetUserPrefValue()
#16 0x7f72af7cd993 PrefService::SetBoolean()
#17 0x561459508e62 autofill::prefs::SetPaymentsIntegrationEnabled()
#18 0x561459fab446 settings::PeopleHandler::HandleSetDatatypes()
#19 0x7f72b20073f5 content::WebUIImpl::ProcessWebUIMessage()
#20 0x7f72b20065c4 content::WebUIImpl::OnWebUISend()
#21 0x7f72b2006376 _ZN3IPC8MessageTI27FrameHostMsg_WebUISend_MetaNSt3__15tupleIJNS2_12basic_stringIcNS2_11char_traitsIcEENS2_9allocatorIcEEEEN4base9ListValueEEEEvE8DispatchIN7content9WebUIImplESG_NSF_15RenderFrameHostEMSG_FvPSH_RKS9_RKSB_EEEbPKNS_7MessageEPT_PT0_PT1_T2_
#22 0x7f72b2006299 content::WebUIImpl::OnMessageReceived()
#23 0x7f72b1fb5241 content::WebContentsImpl::OnMessageReceived()
#24 0x7f72b1c08e45 content::RenderFrameHostImpl::OnMessageReceived()
#25 0x7f72b1e43ccb content::RenderProcessHostImpl::OnMessageReceived()
#26 0x7f72b2cce3b1 IPC::ChannelProxy::Context::OnDispatchMessage()
#27 0x7f72b2cd10c8 _ZN4base8internal7InvokerINS0_9BindStateIMN3IPC12ChannelProxy7ContextEFvRKNS3_7MessageEEJ13scoped_refptrIS5_ES6_EEEFvvEE3RunEPNS0_13BindStateBaseE
#28 0x7f72b46e7b22 base::debug::TaskAnnotator::RunTask()
#29 0x7f72b4714f2f base::MessageLoop::RunTask()
#30 0x7f72b47153c2 base::MessageLoop::DoWork()
#31 0x7f72b471975f base::(anonymous namespace)::WorkSourceDispatch()
#32 0x7f72a7d07fc7 g_main_context_dispatch
#33 0x7f72a7d08200 <unknown>
#34 0x7f72a7d0828c g_main_context_iteration
#35 0x7f72b4719482 base::MessagePumpGlib::Run()
#36 0x7f72b4714a01 base::MessageLoop::Run()
#37 0x7f72b4748ac6 base::RunLoop::Run()
#38 0x561458cefc7a ChromeBrowserMainParts::MainMessageLoopRun()
#39 0x7f72b1a472e7 content::BrowserMainLoop::RunMainMessageLoopParts()
#40 0x7f72b1a4a3d3 content::BrowserMainRunnerImpl::Run()
#41 0x7f72b1a43609 content::BrowserMain()
#42 0x7f72b2580c20 content::ContentMainRunnerImpl::Run()
#43 0x7f72b4a53b19 service_manager::Main()
#44 0x7f72b257ecf1 content::ContentMain()
#45 0x5614584f21b3 ChromeMain
#46 0x7f72a63f92b1 __libc_start_main
#47 0x5614584f202a _start

 
Components: Services>Sync UI>Browser>Autofill
Mergedinto: 894852
Status: Duplicate (was: Available)
Thanks, Marc, for reporting.

Although the repro steps are different, the DCHECK (and probably the root cause) is the same.

Sign in to add a comment