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

Issue 650585 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Sep 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

CHECK error during "Remove this person" with Data Saver extension installed

Project Member Reported by ricea@chromium.org, Sep 27 2016

Issue description

Version: 55.0.2863.0
OS: Tested on Linux

What steps will reproduce the problem?
(0) Compile Chromium with DCHECK enabled.
(1) Add a person to Chromium and sync it to an account with the "Data Saver" extension installed.
(2) Open the "Switch Person" window
(3) From the drop-down menu on the Person's icon, select "Remove this person"
(4) Click the red "REMOVE THIS PERSON" button

What is the expected output?

Chrome does not crash.

What do you see instead?

Chrome crashed 2/3 times with a check failure:

[20212:20212:0927/183215:FATAL:data_usage_store.cc(228)] Check failed: !start_interval.is_null(). 
#0 0x7f2850e0fd9e base::debug::StackTrace::StackTrace()
#1 0x7f2850e31f5b logging::LogMessage::~LogMessage()
#2 0x7f285270af4b data_reduction_proxy::DataUsageStore::BucketOverlapsInterval()
#3 0x7f28526faf1b data_reduction_proxy::DataReductionProxyCompressionStats::DeleteBrowsingHistory()
#4 0x7f2851d3b7ed BrowsingDataRemover::RemoveImpl()
#5 0x7f2851d3ab0e BrowsingDataRemover::RunNextTask()
#6 0x7f2851d3aa5f BrowsingDataRemover::RemoveInternal()
#7 0x7f2851d3a5a9 BrowsingDataRemover::Remove()
#8 0x7f2851c44a63 profiles::RemoveBrowsingDataForProfile()
#9 0x7f2851c3c82b ProfileManager::FinishDeletingProfile()
#10 0x7f2851c3c160 ProfileManager::ScheduleProfileForDeletion()
#11 0x7f2851c3bf92 ProfileManager::MaybeScheduleProfileForDeletion()
#12 0x7f28528a1a21 webui::DeleteProfileAtPath()
#13 0x7f2852937cf3 UserManagerScreenHandler::HandleRemoveUser()
#14 0x7f284ee6a50e content::WebUIImpl::ProcessWebUIMessage()
#15 0x7f284ee695d8 content::WebUIImpl::OnWebUISend()
#16 0x7f284ee693d3 _ZN3IPC8MessageTI26ViewHostMsg_WebUISend_MetaSt5tupleIJ4GURLSsN4base9ListValueEEEvE8DispatchIN7content9WebUIImplESA_vMSA_FvRKS3_RKSsRKS5_EEEbPKNS_7MessageEPT_PT0_PT1_T2_
#17 0x7f284ee692e7 content::WebUIImpl::OnMessageReceived()
#18 0x7f284ee3c070 content::WebContentsImpl::OnMessageReceived()
#19 0x7f284ed4476b content::RenderViewHostImpl::OnMessageReceived()
#20 0x7f284ed4b373 content::RenderWidgetHostImpl::OnMessageReceived()
#21 0x7f284ed38136 content::RenderProcessHostImpl::OnMessageReceived()
#22 0x7f284dddc835 IPC::ChannelProxy::Context::OnDispatchMessage()
#23 0x7f2850e108f4 base::debug::TaskAnnotator::RunTask()
#24 0x7f2850e3d0a5 base::MessageLoop::RunTask()
#25 0x7f2850e3d478 base::MessageLoop::DeferOrRunPendingTask()
#26 0x7f2850e3d89b base::MessageLoop::DoWork()
#27 0x7f2850e3f86a base::(anonymous namespace)::WorkSourceDispatch()
#28 0x7f284a90fe04 g_main_context_dispatch
#29 0x7f284a910048 <unknown>
#30 0x7f284a9100ec g_main_context_iteration
#31 0x7f2850e3f5c6 base::MessagePumpGlib::Run()
#32 0x7f2850e3cba1 base::MessageLoop::RunHandler()
#33 0x7f2850e69de0 base::RunLoop::Run()
#34 0x7f2851e76cda ChromeBrowserMainParts::MainMessageLoopRun()
#35 0x7f284ea78059 content::BrowserMainLoop::RunMainMessageLoopParts()
#36 0x7f284ea7b718 content::BrowserMainRunnerImpl::Run()
#37 0x7f284ea7338e content::BrowserMain()
#38 0x7f284f37a86e content::RunNamedProcessTypeMain()
#39 0x7f284f37b323 content::ContentMainRunnerImpl::Run()
#40 0x7f284f379b80 content::ContentMain()
#41 0x7f285187c4cb ChromeMain
#42 0x7f2847f15f45 __libc_start_main
#43 0x7f285187c39d <unknown>


 
Status: Started (was: Untriaged)
Status: Fixed (was: Started)
Fixed in https://codereview.chromium.org/2378463002/
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 28 2016

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

commit 7d019e768800fe08d9391003702f46622e0bfdab
Author: kundaji <kundaji@chromium.org>
Date: Tue Sep 27 23:35:40 2016

Remove unnecessary DCHECK in BucketOverlapsInterval().

BUG= 650585 

Review-Url: https://codereview.chromium.org/2378463002
Cr-Commit-Position: refs/heads/master@{#421383}

[modify] https://crrev.com/7d019e768800fe08d9391003702f46622e0bfdab/components/data_reduction_proxy/core/browser/data_usage_store.cc

Sign in to add a comment