New issue
Advanced search Search tips

Issue 878379 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 28
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

NGPaintFragment is leak

Project Member Reported by markusheintz@chromium.org, Aug 28

Issue description

Maybe CL https://chromium-review.googlesource.com/1190625 is the cause

Since https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20Leak/23587

The following Layout test are failing on Linux:
Unexpected Failures:
* virtual/layout_ng/external/wpt/css/CSS2/floats-clear/floats-005.xht
* virtual/layout_ng/external/wpt/css/CSS2/floats-clear/floats-136.xht
* virtual/layout_ng/external/wpt/css/CSS2/linebox/inline-formatting-context-022.xht
* virtual/layout_ng/fast/inline/drawStyledEmptyInlines.html
* virtual/layout_ng/fast/inline/drawStyledEmptyInlinesWithWS.html
* virtual/layout_ng/fast/inline/inline-box-background-long-image.html
* virtual/layout_ng/fast/inline/inline-box-background-repeat-x.html
* virtual/layout_ng/fast/inline/inline-box-background-repeat-y.html
* virtual/layout_ng/fast/inline/inline-box-background.html
* virtual/layout_ng/fast/writing-mode/background-vertical-lr.html
* virtual/layout_ng/fast/writing-mode/background-vertical-rl.html
* virtual/layout_ng/fast/writing-mode/border-image-vertical-lr.html
* virtual/layout_ng/fast/writing-mode/border-image-vertical-rl.html
* virtual/layout_ng_experimental/external/wpt/css/css-multicol/multicol-fill-auto-003.xht
 
Sorry wasn't aware that Leak test should not be disabled but instead the offending CL should be reverted.

Reverted https://chromium-review.googlesource.com/1190625 in
https://chromium-review.googlesource.com/c/chromium/src/+/1193864
Status: Started (was: Assigned)
Summary: NGPaintFragment is leak (was: layout_ng Layout tests failing on Linux)
Status: Fixed (was: Started)
The patch causes leak is reverted.

Sign in to add a comment