New issue
Advanced search Search tips

Issue 845288 link

Starred by 3 users
Status: Fixed
Owner:
kainino@chromium.org
Closed: May 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Remove WEBGL_compressed_texture_atc

Project Member Reported by kainino@google.com, May 21 2018 
As described in:
https://www.khronos.org/webgl/public-mailing-list/public_webgl/1702/msg00054.php
https://github.com/KhronosGroup/WebGL/issues/2310

Apparently it was widely supported at the time the extension was created, but it looks like now it is no longer available on AMD (according to the OpenGL capabilities database & on my own machine) making it only possible to implement on Qualcomm devices. Since there are other, newer compressed texture formats available on Qualcomm, this extension seems pretty useless.

The AMD_compressed_ATC_texture spec is very old and underspecified. This is creating quite a bit of maintenance work for the conformance tests and browsers - the tests fail currently on Qualcomm (the only platform where they're available), despite Qualcomm seeming to conform to the underspecified spec.
Project Member

Comment 1 by bugdroid1@chromium.org, May 22 2018 

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

commit 344f339a3b379b4dab61a35be40c5effbbf468fb
Author: Kai Ninomiya <kainino@chromium.org>
Date: Tue May 22 00:46:56 2018

Remove WEBGL_compressed_texture_atc

Bug:  845288 
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: Ica88b645f9db95fa2baa9ab1c0ba51fde8093f6c
Reviewed-on: https://chromium-review.googlesource.com/1067700
Commit-Queue: Kai Ninomiya <kainino@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560435}
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/content/test/data/gpu/webgl2_conformance_tests_output.json
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/content/test/data/gpu/webgl_conformance_tests_output.json
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/content/test/gpu/gpu_tests/webgl2_conformance_expectations.py
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/content/test/gpu/gpu_tests/webgl_conformance_expectations.py
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/content/test/gpu/gpu_tests/webgl_conformance_integration_test.py
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/modules_idl_files.gni
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/webgl/BUILD.gn
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/webgl/webgl2_rendering_context.cc
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/webgl/webgl2_rendering_context.h
[delete] https://crrev.com/20b65d00ca3d8696430e22efad7485366f8c3a21/third_party/blink/renderer/modules/webgl/webgl_compressed_texture_atc.cc
[delete] https://crrev.com/20b65d00ca3d8696430e22efad7485366f8c3a21/third_party/blink/renderer/modules/webgl/webgl_compressed_texture_atc.h
[delete] https://crrev.com/20b65d00ca3d8696430e22efad7485366f8c3a21/third_party/blink/renderer/modules/webgl/webgl_compressed_texture_atc.idl
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/webgl/webgl_extension_name.h
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/webgl/webgl_rendering_context.cc
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/webgl/webgl_rendering_context.h
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/webgl/webgl_rendering_context_base.cc
[modify] https://crrev.com/344f339a3b379b4dab61a35be40c5effbbf468fb/third_party/blink/renderer/modules/webgl/webgl_rendering_context_base.h

Comment 2 by kainino@chromium.org, May 23 2018

Status: Fixed (was: Started)
►

Sign in to add a comment