New issue
Advanced search Search tips

Issue 708372 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

DCHECK about no local border box properties in fast/frames/location-change-no-file-access.html

Project Member Reported by pdr@chromium.org, Apr 5 2017

Issue description

In fast/frames/location-change-no-file-access.html, we'll hit the following DCHECK:
failed: m_layer.layoutObject().localBorderBoxProperties(). 
blink::PaintLayerClipper::initializeCommonClipRectState
blink::PaintLayerClipper::calculateBackgroundClipRectWithGeometryMapper
blink::PaintLayerClipper::calculateRectsWithGeometryMapper
blink::PaintLayerClipper::calculateRects
 
Owner: chrishtr@chromium.org
Status: Assigned (was: Available)
Labels: PaintTeamTriaged-20170404 BugSource-Team
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 2 2017

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

commit 47f019304a77cd50ce7b43c10a064e7f9dc8f34a
Author: Chris Harrelson <chrishtr@chromium.org>
Date: Wed Aug 02 23:23:04 2017

Remove workaround for missing LocalBorderBoxProperties.

This was added to prevent crashes. Let's remove it
now to see if those crashes were fixed, and to uncover
more failure cases, which may shed light on similar crash
bugs we cannot reproduce.

Bug:  708372 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I1e89c3fcc859af65d82c9945a14cbf5aac8ef8c8
Reviewed-on: https://chromium-review.googlesource.com/598452
Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491548}
[modify] https://crrev.com/47f019304a77cd50ce7b43c10a064e7f9dc8f34a/third_party/WebKit/Source/core/paint/PaintLayerClipper.cpp

Status: Fixed (was: Assigned)

Sign in to add a comment