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

Issue 734982 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Setup eviction of cc software image decode cache for unused images.

Project Member Reported by sohan.jy...@huawei.com, Jun 20 2017

Issue description

Setup eviction of cc software image decode cache for unused images.

https://codereview.chromium.org/2945813002/ plans to implement the interface, and functionality for cc gpu image decode cache. So, this would be a follow-up.

 
Status: Assigned (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 22 2017

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

commit 353ed830c053da0569e476ee65990ba88c2ed9fb
Author: sohan <sohan.jyoti@huawei.com>
Date: Sat Jul 22 01:34:33 2017

cc: Implement eviction from SW image decode cache.

This implements logic to evict from SW image decode cache
in cc, based on skia unique image id. We add a auxiliary
map of image id and key, and use it for optimizing search
and access. New api's to handle cache addition and deletion
is also added.

BUG= 734982 
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

patch from issue 2951093002 at patchset 80001 (http://crrev.com/2951093002#ps80001)

Change-Id: Iab533d140802a0308bf2143765708257a465bd31
Reviewed-on: https://chromium-review.googlesource.com/571459
Reviewed-by: Eric Karl <ericrk@chromium.org>
Reviewed-by: Vladimir Levin <vmpstr@chromium.org>
Reviewed-by: Sohan Jyoti Ghosh <sohan.jyoti@huawei.com>
Commit-Queue: Sohan Jyoti Ghosh <sohan.jyoti@huawei.com>
Cr-Commit-Position: refs/heads/master@{#488829}
[modify] https://crrev.com/353ed830c053da0569e476ee65990ba88c2ed9fb/cc/tiles/software_image_decode_cache.cc
[modify] https://crrev.com/353ed830c053da0569e476ee65990ba88c2ed9fb/cc/tiles/software_image_decode_cache.h
[modify] https://crrev.com/353ed830c053da0569e476ee65990ba88c2ed9fb/cc/tiles/software_image_decode_cache_unittest.cc

Status: Fixed (was: Assigned)

Sign in to add a comment