New issue
Advanced search Search tips

Issue 841844 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

virtual/.../animation-worklet-scroll-timeline.html in webkit_layout_tests failing on chromium.webkit/WebKit Linux Trusty Leak

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, May 10 2018

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of dmurph@chromium.org

virtual/.../animation-worklet-scroll-timeline.html in webkit_layout_tests failing on chromium.webkit/WebKit Linux Trusty Leak

Builders failed on: 
- WebKit Linux Trusty Leak: 
  https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20Leak

Cl range:
https://chromium.googlesource.com/chromium/src/+log/c71fcaeb63583fcbf854f7c4a99a8a124037815a%5E..6358e09882ccfd713f02b93b968711d5de95bb24?pretty=fuller&n=


 

Comment 1 by dmu...@chromium.org, May 10 2018

output:

crash log for renderer (pid <unknown>):
STDOUT: layer at (0,0) size 800x600
STDOUT:   LayoutView at (0,0) size 800x600
STDOUT: layer at (0,0) size 800x316
STDOUT:   LayoutBlockFlow {HTML} at (0,0) size 800x316
STDOUT:     LayoutBlockFlow {BODY} at (8,8) size 784x300
STDOUT:       LayoutBlockFlow {DIV} at (0,100) size 100x100 [bgcolor=#FF8080]
STDOUT: layer at (8,208) size 100x100 clip at (8,208) size 85x100 scrollY 450.00 scrollHeight 1000
STDOUT:   LayoutBlockFlow {DIV} at (0,200) size 100x100
STDOUT:     LayoutBlockFlow {DIV} at (0,0) size 85x1000
STDOUT: layer at (8,8) size 100x100 transparent
STDOUT:   LayoutBlockFlow {DIV} at (0,0) size 100x100 [bgcolor=#00FF00]
STDERR: [1734:1741:0509/213010.047469:WARNING:crash_handler_host_linux.cc(334)] Could not translate tid, attempt = 1 retry ...

Comment 2 by dmu...@chromium.org, May 10 2018

Cc: smcgruer@chromium.org
Adding Stephen as the owner of the test - do any of the CLs in that range seem like they could cause this test to fail?
Components: Internals>Compositing>Animation
Project Member

Comment 4 by bugdroid1@chromium.org, May 10 2018

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

commit 35bab68abc419e3242d5388c7292c3fc1a338099
Author: Daniel Murphy <dmurph@chromium.org>
Date: Thu May 10 18:02:00 2018

Revert "Reland "Composite ScrollTimeline scrollSource when appropriate.""

This reverts commit 48f503a88abc242e8aba16ac1f6ec7eac6d104c3.

Reason for revert: Failing the Linux Trusty Leak bot

Original change's description:
> Reland "Composite ScrollTimeline scrollSource when appropriate."
> 
> This is a reland of ddd9f0eec9abf574e6995f9034fa519e069fa1fd
> 
> Original change's description:
> > Composite ScrollTimeline scrollSource when appropriate.
> >
> > This CL makes being the scrollSource of an active (i.e. the scrollSource is
> > scrollable) and attached (i.e. it is being used by at least one WorkletAnimation
> > as a timeline) ScrollTimeline a compositing reason. It does not change the
> > requirement that the scrollSource must be composited for the WorkletAnimation to
> > be sent to the compositor, nor does it change the case where a scrollSource stops
> > being composited whilst a WorkletAnimation is running. Those will be addressed in
> > follow-up CLs.
> >
> > Bug:  776533 
> > Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> > Change-Id: Icae43e1c9cfe2720cc739401c4766b592571f231
> > Reviewed-on: https://chromium-review.googlesource.com/1036934
> > Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
> > Reviewed-by: Majid Valipour <majidvp@chromium.org>
> > Reviewed-by: Philip Rogers <pdr@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#556541}
> 
> TBR=pdr@chromium.org
> 
> Bug:  776533 
> Change-Id: I36cde2bb53e4c24fc57caef1e33f8dcac159ec77
> Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> Reviewed-on: https://chromium-review.googlesource.com/1050547
> Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
> Reviewed-by: Stephen McGruer <smcgruer@chromium.org>
> Reviewed-by: Majid Valipour <majidvp@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#557347}

TBR=pdr@chromium.org,majidvp@chromium.org,smcgruer@chromium.org

Change-Id: Ia89e6d42ef89fceebb1630dcdcbe3b1af6c71964
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  776533 ,  841844 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Reviewed-on: https://chromium-review.googlesource.com/1054116
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557583}
[delete] https://crrev.com/0ae5186d1bf72bd375c5cf14c7a762806fe0f29d/third_party/WebKit/LayoutTests/virtual/threaded/fast/animationworklet/animation-worklet-scroll-timeline-dispose.html
[modify] https://crrev.com/35bab68abc419e3242d5388c7292c3fc1a338099/third_party/WebKit/LayoutTests/virtual/threaded/fast/animationworklet/animation-worklet-scroll-timeline.html
[modify] https://crrev.com/35bab68abc419e3242d5388c7292c3fc1a338099/third_party/blink/renderer/core/BUILD.gn
[modify] https://crrev.com/35bab68abc419e3242d5388c7292c3fc1a338099/third_party/blink/renderer/core/animation/scroll_timeline.cc
[modify] https://crrev.com/35bab68abc419e3242d5388c7292c3fc1a338099/third_party/blink/renderer/core/animation/scroll_timeline.h
[delete] https://crrev.com/0ae5186d1bf72bd375c5cf14c7a762806fe0f29d/third_party/blink/renderer/core/animation/scroll_timeline_test.cc
[modify] https://crrev.com/35bab68abc419e3242d5388c7292c3fc1a338099/third_party/blink/renderer/core/paint/compositing/compositing_reason_finder.cc
[modify] https://crrev.com/35bab68abc419e3242d5388c7292c3fc1a338099/third_party/blink/renderer/modules/animationworklet/worklet_animation.cc
[modify] https://crrev.com/35bab68abc419e3242d5388c7292c3fc1a338099/third_party/blink/renderer/platform/graphics/compositing_reasons.h

Comment 6 by dmu...@chromium.org, May 10 2018

Status: Fixed (was: Available)

Sign in to add a comment