New issue
Advanced search Search tips

Issue 791280 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature

Blocked on:
issue 731727

Blocking:
issue 430155
issue 791281


Participants' hotlists:
AnimationWorklet-MVP


Sign in to add a comment

[animationworklet] Schedule LayerTreeMutator::Mutate calls asynchronously.

Project Member Reported by flackr@chromium.org, Dec 2 2017

Issue description

As described on the e-mail thread and document here:
https://groups.google.com/a/chromium.org/forum/#!topic/animations-dev/Uwx-y6rOJXs

When we post for mutations in  issue 731727  we should continue doing other compositing work, and receive the result of mutation asynchronously.
 
Blocking: 791281
Status: Available (was: Untriaged)
Marking available, doesn't seem "untriaged" to me.
Status: Assigned (was: Available)
Actually, assigned, because it has an owner.
Labels: -Type-Bug Type-Feature
Blocking: 430155
Owner: ----
Status: Available (was: Assigned)
Labels: Hotlist-Experimental
Owner: kevers@chromium.org
Status: Assigned (was: Available)
Cc: flackr@chromium.org petermayo@chromium.org
 Issue 834452  has been merged into this issue.
Labels: -Pri-3 Pri-2
Cc: -petermayo@chromium.org
Project Member

Comment 13 by bugdroid1@chromium.org, Jan 14

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

commit ac5834799e95fa56c0d308ab38115a50fac15801
Author: Kevin Ellis <kevers@chromium.org>
Date: Mon Jan 14 21:14:49 2019

Refactor synchronous dispatching of mutation requests in anticipation of async mutations.

* Splits out reusable functionality in the mutate method in order to share between the sync and async versions.

Bug: 791280
Change-Id: I2aefe1699954a5808ea28332c7a9cadb7808b68d
Reviewed-on: https://chromium-review.googlesource.com/c/1399022
Reviewed-by: Robert Flack <flackr@chromium.org>
Reviewed-by: Majid Valipour <majidvp@chromium.org>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Kevin Ellis <kevers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622603}
[modify] https://crrev.com/ac5834799e95fa56c0d308ab38115a50fac15801/third_party/blink/renderer/platform/graphics/DEPS
[modify] https://crrev.com/ac5834799e95fa56c0d308ab38115a50fac15801/third_party/blink/renderer/platform/graphics/animation_worklet_mutator_dispatcher_impl.cc
[modify] https://crrev.com/ac5834799e95fa56c0d308ab38115a50fac15801/third_party/blink/renderer/platform/graphics/animation_worklet_mutator_dispatcher_impl.h
[modify] https://crrev.com/ac5834799e95fa56c0d308ab38115a50fac15801/third_party/blink/renderer/platform/graphics/animation_worklet_mutator_dispatcher_impl_test.cc

Sign in to add a comment