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

Issue 606117 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit 24 days ago
Closed: Apr 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Do not add already failed image decode tasks as dependencies.

Project Member Reported by prashan...@samsung.com, Apr 23 2016

Issue description

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 23 2016

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

commit 68f3b80bd5f76377e0b4bb067fd8e759d33cefc3
Author: prashant.n <prashant.n@samsung.com>
Date: Sat Apr 23 05:17:50 2016

cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG= 606117 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

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

[modify] https://crrev.com/68f3b80bd5f76377e0b4bb067fd8e759d33cefc3/cc/tiles/gpu_image_decode_controller.cc

Status: Verified (was: Started)
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 25 2016

Labels: merge-merged-2716
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/68f3b80bd5f76377e0b4bb067fd8e759d33cefc3

commit 68f3b80bd5f76377e0b4bb067fd8e759d33cefc3
Author: prashant.n <prashant.n@samsung.com>
Date: Sat Apr 23 05:17:50 2016

cc: Do not add already failed image decode tasks as dependencies.

When gpu decoding is used, do not add image tasks which have already
failed in decoding as dependencies for raster task.

BUG= 606117 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

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

[modify] https://crrev.com/68f3b80bd5f76377e0b4bb067fd8e759d33cefc3/cc/tiles/gpu_image_decode_controller.cc

Project Member

Comment 4 by bugdroid1@chromium.org, May 3 2016

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

commit d43afe33ce9ac38139432f0ac7aee09a5f822588
Author: prashant.n <prashant.n@samsung.com>
Date: Tue May 03 02:57:49 2016

cc: Unittest for checking already failed image decode task.

Add unittest for checking already failed image decode task does not
get added for decoding next time.

BUG= 606117 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/1933723002
Cr-Commit-Position: refs/heads/master@{#391159}

[modify] https://crrev.com/d43afe33ce9ac38139432f0ac7aee09a5f822588/cc/tiles/gpu_image_decode_controller.cc
[modify] https://crrev.com/d43afe33ce9ac38139432f0ac7aee09a5f822588/cc/tiles/gpu_image_decode_controller.h
[modify] https://crrev.com/d43afe33ce9ac38139432f0ac7aee09a5f822588/cc/tiles/gpu_image_decode_controller_unittest.cc

Sign in to add a comment