Bug 39036 - Alias WebGLArray type names to TypedArray type names
Summary: Alias WebGLArray type names to TypedArray type names
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Kenneth Russell
URL:
Keywords:
Depends on: 39080
Blocks: 39091
  Show dependency treegraph
 
Reported: 2010-05-12 17:59 PDT by Kenneth Russell
Modified: 2010-05-13 15:55 PDT (History)
7 users (show)

See Also:


Attachments
Patch (35.99 KB, patch)
2010-05-12 18:09 PDT, Kenneth Russell
dglazkov: review+
kbr: commit-queue-
Details | Formatted Diff | Diff
Revised patch (39.50 KB, patch)
2010-05-13 13:43 PDT, Kenneth Russell
dglazkov: review+
kbr: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kenneth Russell 2010-05-12 17:59:07 PDT
The WebGL specification is changing from the WebGL<T>Array type names to using the names in the separate TypedArray spec. Because this is a reasonably large change for developers (including ourselves) to adapt to, we want to give them a grace period during which both type names work. The current nightly builds of Firefox, for example, support both naming conventions.

To address this we aim to provide new attributes on the DOMWindow object which follow the TypedArray spec's naming convention. Under the hood they will actually instantiate WebGL<T>Array C++ objects. A subsequent patch will rename the implementing classes to use the TypedArray naming convention. In the near future the aliases to the old WebGL<T>Array constructor functions will be deleted.
Comment 1 Kenneth Russell 2010-05-12 18:09:53 PDT
Created attachment 55920 [details]
Patch

From the ChangeLog:

Exposed the existing WebGL<T>Array constructor functions on the DOMWindow under the naming convention in the TypedArray spec. The implementations will be renamed in a subsequent bug, and the aliases to the WebGL<T>Array type names eventually removed. Ran all WebGL layout tests in Safari and Chromium.
Comment 2 Dimitri Glazkov (Google) 2010-05-13 08:55:26 PDT
Comment on attachment 55920 [details]
Patch

I wanted to make you change code generator to auto-generate these, but since they're temporary, ok :)
Comment 3 Kenneth Russell 2010-05-13 11:52:18 PDT
Committed r59376: <http://trac.webkit.org/changeset/59376>
Comment 4 WebKit Review Bot 2010-05-13 12:20:17 PDT
http://trac.webkit.org/changeset/59376 might have broken SnowLeopard Intel Release (Tests)
Comment 5 Kenneth Russell 2010-05-13 13:08:22 PDT
Reopening because initial patch broke tests and was rolled out.
Comment 6 Kenneth Russell 2010-05-13 13:43:45 PDT
Created attachment 56017 [details]
Revised patch

Revised patch including updates to global prototype and constructor tests.
Comment 7 Dimitri Glazkov (Google) 2010-05-13 13:50:03 PDT
Comment on attachment 56017 [details]
Revised patch

r=me again.
Comment 8 Kenneth Russell 2010-05-13 15:07:51 PDT
Committed r59388: <http://trac.webkit.org/changeset/59388>
Comment 9 WebKit Review Bot 2010-05-13 15:55:57 PDT
http://trac.webkit.org/changeset/59388 might have broken Qt Windows 32-bit Release
The following changes are on the blame list:
http://trac.webkit.org/changeset/59387
http://trac.webkit.org/changeset/59388
http://trac.webkit.org/changeset/59389