New issue
Advanced search Search tips

Issue 859787 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Jul 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug


Participants' hotlists:
layoutng


Sign in to add a comment

[LayoutNG] Background painted for visibility hidden inlines

Project Member Reported by e...@chromium.org, Jul 3

Issue description

LayoutNG currently paints backgrounds for inline elements, even if they have visibility hidden. 

 
visibility_hidden.html
359 bytes View Download
There is a similar problem where background is painted twice:
/compositing/background-color/view-blending-base-background.html
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 4

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

commit e9a7dae9acb13c7d7f78c39636220d0e6c2dc866
Author: Aleks Totic <atotic@chromium.org>
Date: Wed Jul 04 01:35:06 2018

[LayoutNG] Inline background paint and visibility

Bug:  859787 
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I03e64594a44c3b7d4ab2fa757f21320df58838f1
Reviewed-on: https://chromium-review.googlesource.com/1125404
Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Aleks Totic <atotic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572437}
[modify] https://crrev.com/e9a7dae9acb13c7d7f78c39636220d0e6c2dc866/third_party/blink/renderer/core/paint/ng/ng_box_fragment_painter.cc

Status: Fixed (was: Available)

Sign in to add a comment