New issue
Advanced search Search tips

Issue 725351 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Feature


Sign in to add a comment

Enable checker-imaging by default.

Project Member Reported by khushals...@chromium.org, May 23 2017

Issue description

This is the tracking bug for all work necessary to enable checker-imaging by default. Enabling it on canary should only be blocked on 725344.
 
Blockedon: 722559 691087
vmpstr@, is there a bug tracking the work for decode API for image elements?

Comment 3 by cblume@chromium.org, May 23 2017

Do you mean the JS decode API or the thing I'm working on?
My work can be tracked here: https://bugs.chromium.org/p/chromium/issues/detail?id=663569
Oops sorry. This was for the JS decode API. That will be a blocking task for checker-imaging now...
Blockedon: 725358
Blockedon: 725517

Comment 7 by vmp...@chromium.org, May 23 2017

Blockedon: 725668
Project Member

Comment 8 by bugdroid1@chromium.org, May 26 2017

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

commit ab4ae0f9d8efe37e7357c8ada6c84b86b56cd1bd
Author: khushalsagar <khushalsagar@chromium.org>
Date: Fri May 26 02:13:04 2017

cc: Add finch feature flag for checker-imaging.

Add a base::Feature entry for checker-imaging finch trial and report
the status on chrome://gpu

BUG= 725351 
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

[modify] https://crrev.com/ab4ae0f9d8efe37e7357c8ada6c84b86b56cd1bd/content/browser/gpu/compositor_util.cc
[modify] https://crrev.com/ab4ae0f9d8efe37e7357c8ada6c84b86b56cd1bd/content/browser/gpu/compositor_util.h
[modify] https://crrev.com/ab4ae0f9d8efe37e7357c8ada6c84b86b56cd1bd/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/ab4ae0f9d8efe37e7357c8ada6c84b86b56cd1bd/content/browser/resources/gpu/info_view.js
[modify] https://crrev.com/ab4ae0f9d8efe37e7357c8ada6c84b86b56cd1bd/content/public/common/content_features.cc
[modify] https://crrev.com/ab4ae0f9d8efe37e7357c8ada6c84b86b56cd1bd/content/public/common/content_features.h
[modify] https://crrev.com/ab4ae0f9d8efe37e7357c8ada6c84b86b56cd1bd/content/test/gpu/gpu_tests/gpu_process_integration_test.py

Project Member

Comment 9 by bugdroid1@chromium.org, May 31 2017

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

commit 4e1be6ae8914f0eeed1df004b74a467bcb93241b
Author: khushalsagar <khushalsagar@chromium.org>
Date: Wed May 31 21:45:49 2017

cc: Enable perf testing for checker-imaging.

BUG= 725351 , 728273

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

[modify] https://crrev.com/4e1be6ae8914f0eeed1df004b74a467bcb93241b/testing/variations/fieldtrial_testing_config.json

Blockedon: 728410
Blockedon: 733247
Blockedon: -725358
Blockedon: -728410
Status: Fixed (was: Assigned)
All the work for getting the first version ready for a finch rollout is done, which is being tracked on the launch bug (728273).

Sign in to add a comment