Bug 108959 - [WEBGL] Rename WEBKIT_WEBGL_depth_texture to WEBGL_depth_texture.
Summary: [WEBGL] Rename WEBKIT_WEBGL_depth_texture to WEBGL_depth_texture.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nayan Kumar K
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-05 10:44 PST by Nayan Kumar K
Modified: 2013-02-06 12:13 PST (History)
7 users (show)

See Also:


Attachments
Patch (6.86 KB, patch)
2013-02-05 11:01 PST, Nayan Kumar K
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nayan Kumar K 2013-02-05 10:44:03 PST
WEBGL_depth_texture is official now - http://www.khronos.org/registry/webgl/extensions/WEBGL_depth_texture/. Hence remove the support for vendor prefix of extension string.
Comment 1 Nayan Kumar K 2013-02-05 11:01:38 PST
Created attachment 186662 [details]
Patch
Comment 2 Kenneth Russell 2013-02-05 12:42:34 PST
Comment on attachment 186662 [details]
Patch

Looks fine. Please submit a pull request to the Khronos repository with the change to the conformance test. Thanks.
Comment 3 WebKit Review Bot 2013-02-05 13:18:18 PST
Comment on attachment 186662 [details]
Patch

Clearing flags on attachment: 186662

Committed r141922: <http://trac.webkit.org/changeset/141922>
Comment 4 WebKit Review Bot 2013-02-05 13:18:22 PST
All reviewed patches have been landed.  Closing bug.
Comment 5 Nayan Kumar K 2013-02-05 19:11:01 PST
(In reply to comment #2)
> (From update of attachment 186662 [details])
> Looks fine. Please submit a pull request to the Khronos repository with the change to the conformance test. Thanks.

WebGL conformance tests uses "getExtensionWithKnownPrefixes" function as extension name look-up. This look-up already covers non-prefix'ed version of string, hence I think it needs no modification.
Comment 6 Kenneth Russell 2013-02-05 22:05:56 PST
(In reply to comment #5)
> (In reply to comment #2)
> > (From update of attachment 186662 [details] [details])
> > Looks fine. Please submit a pull request to the Khronos repository with the change to the conformance test. Thanks.
> 
> WebGL conformance tests uses "getExtensionWithKnownPrefixes" function as extension name look-up. This look-up already covers non-prefix'ed version of string, hence I think it needs no modification.

I see. OK.
Comment 7 Nayan Kumar K 2013-02-06 11:39:58 PST
> 
> I see. OK.

As a side note, instead of maintaining subset of WebGL conformance tests in LayoutTests/fast/canvas/webgl, shouldn't we try to
a. Let build bots run conformance tests run directly from Khronos WebGL repository? This method deviates from existing test framework (as the tests are maintained outside WebKit repository), but avoids maintaining similar tests in 2 places.
b. Pull-in whole of Khronos WebGL conformance tests into LayoutTests and keep them in sync at regular intervals?

Above alternatives may require us to write some sort of filters to filter out executing subset of tests (which are not applicable or which fails). Just a thought, let me know your opinion.
Comment 8 Kenneth Russell 2013-02-06 11:45:54 PST
(In reply to comment #7)
> > 
> > I see. OK.
> 
> As a side note, instead of maintaining subset of WebGL conformance tests in LayoutTests/fast/canvas/webgl, shouldn't we try to
> a. Let build bots run conformance tests run directly from Khronos WebGL repository? This method deviates from existing test framework (as the tests are maintained outside WebKit repository), but avoids maintaining similar tests in 2 places.
> b. Pull-in whole of Khronos WebGL conformance tests into LayoutTests and keep them in sync at regular intervals?
> 
> Above alternatives may require us to write some sort of filters to filter out executing subset of tests (which are not applicable or which fails). Just a thought, let me know your opinion.

Gregg Tavares is incorporating the entire Khronos WebGL conformance suite as layout tests. Follow https://bugs.webkit.org/show_bug.cgi?id=94643 and related bugs for progress.
Comment 9 Nayan Kumar K 2013-02-06 12:13:41 PST
> 
> Gregg Tavares is incorporating the entire Khronos WebGL conformance suite as layout tests. Follow https://bugs.webkit.org/show_bug.cgi?id=94643 and related bugs for progress.

Great. Thanks.