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

Issue 728736 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

DCHECK in chrome --mash when over page with custom cursors

Project Member Reported by sky@chromium.org, Jun 1 2017

Issue description

This is on my pixel and on zillow. Here's the stack:
#0 0x7f6b24acc99c base::debug::StackTrace::StackTrace()
#1 0x7f6b24ae7f4e logging::LogMessage::~LogMessage()
#2 0x7f6b251bbf31 skia::ImageOperations::Resize()
#3 0x7f6b251bc185 skia::ImageOperations::Resize()
#4 0x7f6b256a9d13 ui::ScaleAndRotateCursorBitmapAndHotpoint()
#5 0x7f6b227e442a content::WebCursor::GetPlatformCursor()
#6 0x7f6b227e423e content::WebCursor::GetNativeCursor()
#7 0x7f6b2308ab57 content::RenderWidgetHostViewAura::UpdateCursorIfOverSelf()
#8 0x7f6b2308aa21 content::RenderWidgetHostViewAura::UpdateCursor()
#9 0x7f6b230801d7 content::RenderWidgetHostImpl::SetCursor()
#10 0x7f6b2307b288 _ZN3IPC8MessageTI26ViewHostMsg_SetCursor_MetaSt5tupleIJN7content9WebCursorEEEvE8DispatchINS3_20RenderWidgetHostImplES8_vMS8_F
vRKS4_EEEbPKNS_7MessageEPT_PT0_PT1_T2_
#11 0x7f6b23078a17 content::RenderWidgetHostImpl::OnMessageReceived()
#12 0x7f6b23060278 content::RenderProcessHostImpl::OnMessageReceived()
#13 0x7f6b2511e941 IPC::ChannelProxy::Context::OnDispatchMessage()
#14 0x7f6b22c335e0 _ZN4base8internal7InvokerINS0_9BindStateINS0_18IgnoreResultHelperIMN7content20BrowserMessageFilter8InternalEFbRKN3IPC7Message
EEEEJ13scoped_refptrIS6_ES8_EEEFvvEE3RunEPNS0_13BindStateBaseE
#15 0x7f6b2269aacf _ZNO4base8CallbackIFvvELNS_8internal8CopyModeE1ELNS2_10RepeatModeE1EE3RunEv
#16 0x7f6b24b8a1e8 base::debug::TaskAnnotator::RunTask()
#17 0x7f6b24af0905 base::MessageLoop::RunTask()
#18 0x7f6b24af0c1b base::MessageLoop::DeferOrRunPendingTask()
#19 0x7f6b24af108a base::MessageLoop::DoWork()
#20 0x7f6b24af3849 base::MessagePumpLibevent::Run()
#21 0x7f6b24af047e base::MessageLoop::Run()
#22 0x7f6b24b1db37 base::RunLoop::Run()
#23 0x7f6b24638308 ChromeBrowserMainParts::MainMessageLoopRun()
#24 0x7f6b22d23584 content::BrowserMainLoop::RunMainMessageLoopParts()
#25 0x7f6b22d26443 content::BrowserMainRunnerImpl::Run()
#26 0x7f6b22d1ea2c content::BrowserMain()
#27 0x7f6b246053cf content::RunNamedProcessTypeMain()
#28 0x7f6b246061bd content::ContentMainRunnerImpl::Run()
#29 0x7f6b2462d89b service_manager::Main()
#30 0x7f6b24604ba1 content::ContentMain()
#31 0x7f6b2267d3ec ChromeMain
#32 0x7f6b200d1816 __libc_start_main
#33 0x7f6b2267d1f9 _start

 
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 6 2017

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

commit f5ed5f60a73da4d69a1c87dfe98523bba659a7f0
Author: erg <erg@chromium.org>
Date: Tue Jun 06 18:26:58 2017

Fix a DCHECK ordering problem in ImageOperations.

This moves a DCHECK below an ordered early out. The subset of an empty bitmap
with an empty rectangle is empty, and we should return an SkBitmap() before
performing a more general intersection check.

BUG= 728736 

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

[modify] https://crrev.com/f5ed5f60a73da4d69a1c87dfe98523bba659a7f0/skia/ext/image_operations.cc

Comment 3 by e...@chromium.org, Jun 6 2017

Status: Fixed (was: Assigned)

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment