Issue metadata
Sign in to add a comment
|
DCHECK failed in DrawingRecorder
Reported by
ana...@yandex-team.ru,
Aug 1
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.103 YaBrowser/18.7.1.514 (beta) Yowser/2.5 Safari/537.36 Steps to reproduce the problem: 1. Open https://www.wikipedia.org 2. Try print page What is the expected behavior? Correct creation of preview What went wrong? Crashes with check failed in DrawingRecorder::~DrawingRecorder: 0u == picture->size() (0 vs. 1)"LayoutBlockFlow A class='link link_theme_normal organic__url link_cropped_no i-bem'" Did this work before? Yes Chrome version: 67.0.3396.103 Channel: n/a OS Version: 10.0 Flash Version: Shockwave Flash 28.0 r0 I guess it doesn't work after CL: https://chromium.googlesource.com/chromium/src/+/dc7889d8b9027cc1b7ce22790fbf74bc0ab5d7a2%5E%21/#F0
,
Aug 1
,
Aug 2
Unable to reproduce the issue on Win-10 using chrome latest stable #68.0.3440.84 and latest canary #70.0.3508.0. Attached a screen cast for reference. Following are the steps followed to reproduce the issue. ------------ 1. Opened https://www.wikipedia.org 2. Printed page. 3. Observed correct creation of preview as expected and did not observe any error. anakir@ - Could you please check the issue on latest stable #68.0.3440.84 by creating a new profile without any apps and extensions and please let us know if the issue still persists or not. Also please check the attached screen cast and please let us know if anything missed from our end. Thanks...!!
,
Aug 2
,
Aug 2
,
Aug 2
Hello! Sorry, I did not specify, that it's must be debug build. In stable and canary builds dcheck-s don't work. Attach video with crash. Thanks!
,
Aug 2
vmpstr@, would you again care to be the wangxianzhu@ replacement? I think it's fine to reassign to him if you're overloaded because this only affects debug builds and it can wait.
,
Aug 2
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0dd3f68274a4dc9e86398dae9d3e4862e644e543 commit 0dd3f68274a4dc9e86398dae9d3e4862e644e543 Author: Vladimir Levin <vmpstr@chromium.org> Date: Thu Aug 02 21:15:40 2018 Make PaintController::IsSkippingCache return true for transient types. This patch ensures that IsSkippingCache, which is the only public indicator that we might skip a cache, to return true if the paint controller is transient. This ensures that DCHECKs that verify the caches don't fire if they are guarded by IsSkippingCache checks. R=pdr@chromium.org, chrishtr@chromium.org Bug: 869809 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I1051609f65588dbef31e6d2b81e77d16752d2982 Reviewed-on: https://chromium-review.googlesource.com/1161057 Reviewed-by: Chris Harrelson <chrishtr@chromium.org> Commit-Queue: vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#580322} [modify] https://crrev.com/0dd3f68274a4dc9e86398dae9d3e4862e644e543/third_party/blink/renderer/platform/graphics/paint/paint_controller.cc [modify] https://crrev.com/0dd3f68274a4dc9e86398dae9d3e4862e644e543/third_party/blink/renderer/platform/graphics/paint/paint_controller.h
,
Aug 2
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by ana...@yandex-team.ru
, Aug 1