New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 605190 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 605189



Sign in to add a comment

Stop Image Animations on the Engine

Project Member Reported by khushals...@chromium.org, Apr 20 2016

Issue description

This includes the GIF animations which request a commit for every frame.
 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 21 2016

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

commit 03a74c3c04ff24b43fbeaf5dc0c3e013c364e500
Author: khushalsagar <khushalsagar@chromium.org>
Date: Thu Apr 21 21:38:32 2016

blimp: Update animation policy for images on the blimp engine.

The Blimp Engine currently uses the default animation policy in blink,
which means for an image like a GIF the engine would trigger regular
commits to drive the animation. Change this to NO_ANIMATION, so only
the first frame of the animation is shown.

BUG= 605190 

Review URL: https://codereview.chromium.org/1903873003

Cr-Commit-Position: refs/heads/master@{#388908}

[modify] https://crrev.com/03a74c3c04ff24b43fbeaf5dc0c3e013c364e500/blimp/engine/BUILD.gn
[delete] https://crrev.com/c1a9e6d597840d7f479cfc9563c8fc69038819de/blimp/engine/app/engine_settings.cc
[modify] https://crrev.com/03a74c3c04ff24b43fbeaf5dc0c3e013c364e500/blimp/engine/app/engine_settings.h
[modify] https://crrev.com/03a74c3c04ff24b43fbeaf5dc0c3e013c364e500/blimp/engine/app/settings_manager.cc
[modify] https://crrev.com/03a74c3c04ff24b43fbeaf5dc0c3e013c364e500/blimp/engine/app/settings_manager_unittest.cc

Status: Fixed (was: Started)
Labels: Archive-Blimp

Sign in to add a comment