New issue
Advanced search Search tips

Issue 859586 link

Starred by 1 user
Status: Fixed
Owner:
chrishtr@chromium.org
Closed: Jul 3
Cc:
vmp...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

CHECK failure: !new_cached_subsequences_.Contains(&client). Multiple subsequences for client: "

Project Member Reported by ClusterFuzz, Jul 2 
Detailed report: https://clusterfuzz.com/testcase?key=6551386570096640

Fuzzer: bj_broddelwerk
Job Type: linux_debug_chrome
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  !new_cached_subsequences_.Contains(&client). Multiple subsequences for client: "
  blink::PaintController::EndSubsequence
  blink::SubsequenceRecorder::~SubsequenceRecorder
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_debug_chrome&range=559400:559401

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6551386570096640

Issue filed automatically.

See https://github.com/google/clusterfuzz-tools for more information.
Project Member

Comment 1 by ClusterFuzz, Jul 2

Components: Blink>Paint Internals>Core
Labels: Test-Predator-Auto-Components
Automatically applying components based on crash stacktrace and information from OWNERS files.

If this is incorrect, please apply the Test-Predator-Wrong-Components label.
Project Member

Comment 2 by ClusterFuzz, Jul 2

Labels: Test-Predator-Auto-Owner
Owner: chrishtr@chromium.org
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/chromium/src/+/02a5799e0e02d7e7274d78833f897c81ae2cc434 ([ForeignObject] Paint foreign objects during the foreground phase of normal-flow).

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label. If you aren't the correct owner for this issue, please unassign yourself as soon as possible so it can be re-triaged.
Project Member

Comment 3 by ClusterFuzz, Jul 3

Labels: -Reproducible Unreproducible
ClusterFuzz testcase 6551386570096640 appears to be flaky, updating reproducibility label.
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 3 

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

commit e27679b15b4bf6837ed43675aab3dbbfe0761f9e
Author: Chris Harrelson <chrishtr@chromium.org>
Date: Tue Jul 03 14:14:11 2018

Don't allow subsequence caching for foreignObject under multi-col

SVG content can't fragment (see LayoutFlowThread::LocateFlowThreadContainingBlockOf)
but since it is painted as a replaced-normal-flow stacking context, multiple
calls into its paint can happen anyway.

Bug:  859586 
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ied0b869d37373499e6a784dee8bd620d7b535fa9
Reviewed-on: https://chromium-review.googlesource.com/1123765
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572200}
[modify] https://crrev.com/e27679b15b4bf6837ed43675aab3dbbfe0761f9e/third_party/WebKit/LayoutTests/TestExpectations
[add] https://crrev.com/e27679b15b4bf6837ed43675aab3dbbfe0761f9e/third_party/WebKit/LayoutTests/fragmentation/foreignobject-no-pagination-expected.html
[add] https://crrev.com/e27679b15b4bf6837ed43675aab3dbbfe0761f9e/third_party/WebKit/LayoutTests/fragmentation/foreignobject-no-pagination.html
[modify] https://crrev.com/e27679b15b4bf6837ed43675aab3dbbfe0761f9e/third_party/blink/renderer/core/paint/svg_foreign_object_painter.cc

Comment 5 by chrishtr@chromium.org, Jul 3

Status: Fixed (was: Assigned)
►

Sign in to add a comment