New issue
Advanced search Search tips

Issue 735662 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Feature

Blocked on:
issue 735741
issue 735743
issue 735747



Sign in to add a comment

Move image animations to the compositor thread.

Project Member Reported by khushals...@chromium.org, Jun 21 2017

Issue description

This is the tracking bug for everything that needs to be done to tick image animations on the compositor thread.
 
Blockedon: 735741
Blockedon: 735743
Blockedon: 691735
Labels: -Type-Bug Type-Feature
Blockedon: 735747
Blockedon: 728359
Blockedon: 764189
Project Member

Comment 8 by bugdroid1@chromium.org, Sep 14 2017

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

commit b42fb24d343a7ff81cfb6b6bc16496930d2710eb
Author: Khushal <khushalsagar@chromium.org>
Date: Thu Sep 14 19:15:15 2017

cc: Track animated images during discardable image analysis.

Keep a vector of animated images when generating data for discardable
images. This will be required for tracking images to animate in cc.

Bug:  735662 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I27ff57af0f2d4a5dd60697d78787d84e62a63db4
Reviewed-on: https://chromium-review.googlesource.com/630816
Commit-Queue: Khushal <khushalsagar@chromium.org>
Reviewed-by: Vladimir Levin <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502010}
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/paint/discardable_image_map.cc
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/paint/discardable_image_map.h
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/paint/discardable_image_map_unittest.cc
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/paint/frame_metadata.h
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/paint/paint_image.cc
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/paint/paint_image.h
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/paint/paint_image_builder.cc
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/test/skia_common.cc
[modify] https://crrev.com/b42fb24d343a7ff81cfb6b6bc16496930d2710eb/cc/test/skia_common.h

Project Member

Comment 10 by bugdroid1@chromium.org, Oct 6 2017

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

commit 051a012c57a3fdfd6c933ec3df1c0e60d247a73a
Author: Khushal <khushalsagar@chromium.org>
Date: Fri Oct 06 04:42:46 2017

perf: Enable field trial testing for image animations in cc.

Bug:  735662 
Change-Id: I44d121a5109f7c76667630303200ba30329f7341
Reviewed-on: https://chromium-review.googlesource.com/704035
Reviewed-by: Steven Holte <holte@chromium.org>
Commit-Queue: Khushal <khushalsagar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506969}
[modify] https://crrev.com/051a012c57a3fdfd6c933ec3df1c0e60d247a73a/testing/variations/fieldtrial_testing_config.json

Blockedon: -764189
Blockedon: -728359
Blockedon: -691735
Status: Fixed (was: Assigned)
The feature is code complete. Tracking the finch controlled roll out on the launch bug: 776216.

Sign in to add a comment