New issue
Advanced search Search tips

Issue 782385 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

DCHECK triggered in data_use_recorder.cc

Project Member Reported by tbansal@chromium.org, Nov 7 2017

Issue description

Not sure what's the best way to trigger it. I got it when navigating to a new page on Linux desktop.


Stacktrace:

7772:7814:1107/130904.416945:FATAL:data_use_recorder.cc(45)] Check failed: other->pending_url_requests_.find(request) == other->pending_url_requests_.end(). 
#0 0x7fbd3c73eead base::debug::StackTrace::StackTrace()
#1 0x7fbd3c73d2dc base::debug::StackTrace::StackTrace()
#2 0x7fbd3c7c499a logging::LogMessage::~LogMessage()
#3 0x007aa6b20b91 data_use_measurement::DataUseRecorder::MovePendingURLRequestTo()
#4 0x007aa360520c data_use_measurement::ChromeDataUseAscriber::DidFinishMainFrameNavigation()
#5 0x007aa38cd50f _ZN4base8internal13FunctorTraitsIMN20data_use_measurement21ChromeDataUseAscriberEFviiRK4GURLbjNS_9TimeTicksEEvE6InvokeIPS3_JiiS4_bN2ui14PageTransitionES7_EEEvS9_OT_DpOT0_
#6 0x007aa38cd3c4 _ZN4base8internal12InvokeHelperILb0EvE8MakeItSoIMN20data_use_measurement21ChromeDataUseAscriberEFviiRK4GURLbjNS_9TimeTicksEEJPS5_iiS6_bN2ui14PageTransitionES9_EEEvOT_DpOT0_
#7 0x007aa38cd2bc _ZN4base8internal7InvokerINS0_9BindStateIMN20data_use_measurement21ChromeDataUseAscriberEFviiRK4GURLbjNS_9TimeTicksEEJNS0_17UnretainedWrapperIS4_EEiiS5_bN2ui14PageTransitionES8_EEEFvvEE7RunImplISA_NSt3__15tupleIJSC_iiS5_bSE_S8_EEEJLm0ELm1ELm2ELm3ELm4ELm5ELm6EEEEvOT_OT0_NSJ_16integer_sequenceImJXspT1_EEEE
#8 0x007aa38ccfd9 _ZN4base8internal7InvokerINS0_9BindStateIMN20data_use_measurement21ChromeDataUseAscriberEFviiRK4GURLbjNS_9TimeTicksEEJNS0_17UnretainedWrapperIS4_EEiiS5_bN2ui14PageTransitionES8_EEEFvvEE7RunOnceEPNS0_13BindStateBaseE
#9 0x7fbd3c6eb051 _ZNO4base12OnceCallbackIFvvEE3RunEv
#10 0x7fbd3c7436ba base::debug::TaskAnnotator::RunTask()
#11 0x7fbd3c7e37e5 base::internal::IncomingTaskQueue::RunTask()
#12 0x7fbd3c7ec740 base::MessageLoop::RunTask()
#13 0x7fbd3c7ec9f6 base::MessageLoop::DeferOrRunPendingTask()
#14 0x7fbd3c7ecd40 base::MessageLoop::DoWork()
#15 0x7fbd3c7f2b29 base::MessagePumpLibevent::Run()
#16 0x7fbd3c7ebebc base::MessageLoop::Run()
#17 0x7fbd3c898d0b base::RunLoop::Run()
#18 0x7fbd3c95b194 base::Thread::Run()
#19 0x7fbd36ca8d46 content::BrowserThreadImpl::IOThreadRun()
#20 0x7fbd36ca904b content::BrowserThreadImpl::Run()
#21 0x7fbd3c95bdbb base::Thread::ThreadMain()
#22 0x7fbd3c9416e1 base::(anonymous namespace)::ThreadFunc()
#23 0x7fbd3cc90184 start_thread
#24 0x7fbd242a7ffd clone
 
Owner: rajendrant@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/47cbf8de6899390eaac9f41f50ccea8c9bc766ad

commit 47cbf8de6899390eaac9f41f50ccea8c9bc766ad
Author: rajendrant <rajendrant@chromium.org>
Date: Wed Nov 08 00:54:30 2017

Fix urlrequest move for same document navigation case

Fix a dcheck that is hit due to url request added to the data use recorder
before removing it from the old data use recorder

Bug:  782385 
Change-Id: I0618013ee648ca10acb9501ec0a4414f7a804143
Reviewed-on: https://chromium-review.googlesource.com/757778
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Commit-Queue: rajendrant <rajendrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#514684}
[modify] https://crrev.com/47cbf8de6899390eaac9f41f50ccea8c9bc766ad/chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc
[modify] https://crrev.com/47cbf8de6899390eaac9f41f50ccea8c9bc766ad/chrome/browser/data_use_measurement/chrome_data_use_ascriber_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment