Commits · 8dd592bf2a5ace66d12dc6a4e381fd345fd42da1 · Chen Yisong / angle

31 Mar, 2014 6 commits

Port the SimpleInstancing sample to the new sample framework. · 8dd592bf

authored Mar 03, 2014

Add a Vector2 type for storing texcoords and improve the sample by having
it draw several thousand instances instead of two.

BUG=angle:521

Change-Id: I87996a90000a83bc66d091c7537ea411080d7e9f
Reviewed-on: https://chromium-review.googlesource.com/188723Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

8dd592bf

Fix D3D11 resource leaks. · 8b4f8f84

authored Mar 26, 2014

A few bugs, like missing destructors or Release calls were leaking
D3D resources in a few places on exit. This patch should fix all the
unreleased D3D object warnings on exit when running a sample app.

BUG=angle:589

Change-Id: Ib50df3229998938cddc2859d6f2d19ddd3a4db47
Reviewed-on: https://chromium-review.googlesource.com/190079Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

8b4f8f84

Fix D3D9 short attributes to int component type. · 0979e7aa

authored Mar 27, 2014

In the component type lexicon, we only use float/int/uint/norm,
and we were marking the D3D9 short vertex attributes as short.

This fixes the WebGL Test gl-vertexattribpointer-offsets.

BUG=angle:550

Change-Id: I3853df8bad5c98b8d0986bfa4ea73d254264ad56
Reviewed-on: https://chromium-review.googlesource.com/191950Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

0979e7aa

Use correct input stride for disabled attribs. · 1e194d31

authored Mar 27, 2014

We were using the input stride from the disabled vertex attribute.
This was breaking the gl-disabled-vertex-attrib test in D3D9.

BUG=angle:550

Change-Id: Ia5a37dc39105fe8ec92f4adbee3cfb2905e82676
Reviewed-on: https://chromium-review.googlesource.com/191932Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

1e194d31

Fix stack overflow when parsing huge expressions. · 6654bc93

authored Mar 26, 2014

The expression limit validation check needs to be in front
of other tree traversal to prevent stack overflows. Also,
call depths of sufficient size (80k+) could overflow the
depth check itself, necessitating an upper bound on initial
tree traversal.

This fixes crashes in the WebGL long-expressions bug test.

BUG=angle:584

Change-Id: Ib48294bf77a5923d230f237fbd63a36a5662e317
Reviewed-on: https://chromium-review.googlesource.com/191931Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

6654bc93

Use a heap-memory traversal to free compiler resources. · dd0d3426

authored Mar 26, 2014

The stack-memory traversal is prone to stack overflow. See the
WebGL conformance test long-expressions-should-not-crash.

BUG=angle:584

Change-Id: I02d72bc2e4101b7141d609c50303403ea8298e60
Reviewed-on: https://chromium-review.googlesource.com/191930Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

dd0d3426

28 Mar, 2014 5 commits

Port the ParticleSystem sample to the new sample framework. · d19fae55

authored Feb 28, 2014

BUG=angle:521

Change-Id: I6d00333ac7761f15ee83a47dc3e6e3cc13c675cd
Reviewed-on: https://chromium-review.googlesource.com/188195Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

d19fae55

Compile shaders with the best profile available. · 6e05c272

authored Mar 17, 2014

Instead of always compiling D3D11 shaders with shader model 4, compile
with the latest profile that the current feature level allows.

BUG=angle:495

Change-Id: I478d3d4a1aadf24d893508eee1a8c11e501bc084
Reviewed-on: https://chromium-review.googlesource.com/190340Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

6e05c272

Add support for drawing with transform feedback. · 4c5c6bbb

authored Feb 05, 2014

BUG=angle:495

Change-Id: Ib9c19130dfbc44a99998c5d6d8160bfc2b683eb9
Reviewed-on: https://chromium-review.googlesource.com/185037Tested-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>

4c5c6bbb

Port the MultiTexture sample to the new sample framework. · 1b3e4b80

authored Feb 28, 2014

BUG=angle:521

Change-Id: I72f063664b43d3cfafe03c8f060076596bedea0f
Reviewed-on: https://chromium-review.googlesource.com/188194Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

1b3e4b80

Port the MipMap2D sample to the new sample framework. · 9adef9ba

authored Feb 28, 2014

BUG=angle:521

Change-Id: I3b1d3a02214850bf52c1d5c468f5696076065b88
Reviewed-on: https://chromium-review.googlesource.com/188193Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

9adef9ba

27 Mar, 2014 4 commits

Adds support for EXT_map_buffer_range · 916e769a

authored Mar 27, 2014

BUG=angle:581

Change-Id: I0ab637db35afd51ff4c516bb97e954f4f2cdad3d
Reviewed-on: https://chromium-review.googlesource.com/191710Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

916e769a

Adds support for OES_mapbuffer · b3801744

authored Mar 27, 2014

BUG=angle:581

Change-Id: I05824051789a77c8169ec5830731e6155c507fc4
Reviewed-on: https://chromium-review.googlesource.com/191650Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

b3801744

Port the Hello_Triangle sample to the new sample framework. · 8776b258

authored Feb 28, 2014

BUG=angle:521

Change-Id: I887257193af3130447c4691bb72303cc978b96fe
Reviewed-on: https://chromium-review.googlesource.com/188192Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

8776b258

Add an ANGLE sample framework. · 49be2ad9

authored Feb 28, 2014

BUG=angle:521

Change-Id: If1fc6198dbbe44f0ade5539cff540e3e086f2a3b
Reviewed-on: https://chromium-review.googlesource.com/188722Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

49be2ad9

26 Mar, 2014 4 commits

Make D3D9 default again. · f81e69b9

authored Mar 26, 2014

BUG=angle:568

Change-Id: I4b60084f2be0c04bb2f7dd78e78e2ef2825adc00
Reviewed-on: https://chromium-review.googlesource.com/191552Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

f81e69b9

Added transform feedback shader generation. · 48dcae7b

authored Feb 05, 2014

BUG=angle:495

Change-Id: I41a0177fd3eb43c9f4ab9e54faeadac3eb483c2c
Reviewed-on: https://chromium-review.googlesource.com/185035Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

48dcae7b

Add remaining transform feedback queries. · 1b6edcb6

authored Feb 03, 2014

BUG=angle:493

Change-Id: I8841af50dcb90e7fb8a811b95d015bfdd2c5d226
Reviewed-on: https://chromium-review.googlesource.com/191035Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

1b6edcb6

Allow creating fixed-size window surfaces. · 3dc300a5

authored Jan 28, 2014

BUG=320021

Change-Id: I97cdd65ac17ee142700dbf0f363891500e967571
Reviewed-on: https://chromium-review.googlesource.com/184205Reviewed-by: John Bauman <jbauman@chromium.org>
Tested-by: John Bauman <jbauman@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/190563Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

3dc300a5

25 Mar, 2014 1 commit

Clean up gyp build scripts. · f0b40601

authored Mar 24, 2014

We can use angle_path instead of angle_build_scripts_path to locate
enumerate_files.py. Also pass angle_path to the post-build script
hook, as this information is necessary to locate the root angle
folder.

BUG=angle:592

Change-Id: I06b3f0201751d90b7c1efdefe8db07d114bd87dd
Reviewed-on: https://chromium-review.googlesource.com/191082Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

f0b40601

24 Mar, 2014 1 commit

Fixed error message for exceeding maximum struct nesting. · 41a49272

authored Mar 18, 2014

It got broken in r2423.
We were referring to field-name as struct-name.
This patch fixes the regression and also improves the error message
by adding the field-name.

BUG=angle:459
R=kbr@chromium.org

Review URL: https://codereview.appspot.com/12891043

Change-Id: I9ac9a8dafcad1857f7d39e6d494e457d1fc50f70
Reviewed-on: https://chromium-review.googlesource.com/190530Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

41a49272

18 Mar, 2014 7 commits

Revert "Add remaining transform feedback queries." · 1e40d1c9

authored Mar 18, 2014

This reverts commit 4251b758.

Change-Id: I95afebcaf91ee38a121df20f403eb06f19b2510a
Reviewed-on: https://chromium-review.googlesource.com/190560Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

1e40d1c9

Validate target parameter in CopyTexImage calls before using to check mip level · 4dfed837

authored Mar 17, 2014

Would cause UNREACHABLE to be triggered if target was invalid, and INVALID_VALUE to
be returned instead of INVALID_ENUM if it were the only erroneous argument.

Change-Id: I2606e77379fa2832a50b3a299e6474a4b2f68afa
Reviewed-on: https://chromium-review.googlesource.com/189701Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

4dfed837

Add remaining transform feedback queries. · 4251b758

authored Feb 03, 2014

BUG=angle:493

Change-Id: Ibeae41da2b83ab63d237912602ba5d2aad0bbec2
Reviewed-on: https://chromium-review.googlesource.com/185036Tested-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>

4251b758

Only use map-write instead of map-write-discard. · e708fc02

authored Mar 17, 2014

In D3D11, map-discard is reserved for DYNAMIC usage buffer,
which are resources that live on the GPU. Staging buffers
don't have the notion of map-discard, and using them with
discard gives a D3D11 error.

This patch fixes crashes in dEQP buffer.map.write.invalidate.

BUG=angle:587

Change-Id: Ie225e72d4e226c69e73826c9fb67bbb940bbc466
Reviewed-on: https://chromium-review.googlesource.com/190072Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

e708fc02

Fix unsigned integer R10G10B10A2 vertex formats. · 0c66720a

authored Mar 17, 2014

We were incorrectly setting the signed parameter to true
in the vertex format table. This fixes a dEQP test in
vertex_arrays.output_types.

BUG=angle:586

Change-Id: I84dcdee89a95369cc8e41d451f96dbb8daa37b6d
Reviewed-on: https://chromium-review.googlesource.com/190056Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

0c66720a

Fix linking of struct varyings. · a53ab517

authored Mar 17, 2014

The dEQP varying struct linkage tests were triggering
asserts when we tried to query component info of struct types.

BUG=angle:580

Change-Id: I61e33573c5577a327c58fec7a07ae7718690ac42
Reviewed-on: https://chromium-review.googlesource.com/189194Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

a53ab517

Fix matrix input attributes and formats. · 3b7e205c

authored Mar 17, 2014

We can't assume we have a simple mapping between input attribs
and shader attribs. The app specifies matrix input attribs as
several vector inputs, while the shader uses a single matrix type.

BUG=angle:576

Change-Id: I4e3cdde4a14a5f06b86052dfe0325113cbe8e599
Reviewed-on: https://chromium-review.googlesource.com/189193Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

3b7e205c

17 Mar, 2014 5 commits

Amends CopyTexImage validation to conform to new language in the ES 3.0.3 spec. · 4d161bad

authored Mar 17, 2014

bug=angle:582

Also performs format checks for non-subimage copies, and permits unsized format
textures as destination for the copy.

Change-Id: Id647bb968416fbd35e1c70e5da21c81367e08f07
Reviewed-on: https://chromium-review.googlesource.com/189431Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

4d161bad

Draft spec for ANGLE_platform_angle_d3d · 3e29695d

authored Feb 04, 2014

This is the initial draft of a spec that extends
EGL_EXT_platform_base to allow choosing a particular ANGLE D3D
backend.

Change-Id: I1d80fa8b3c4a59fe998f6d15aa0f393bdb9d26c1
Reviewed-on: https://chromium-review.googlesource.com/184975Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
Tested-by: Scott Graham <scottmg@chromium.org>
(cherry picked from commit dc18dc30)
Reviewed-on: https://chromium-review.googlesource.com/190026Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

3e29695d

Disable MRT support when using D3D10 feature levels. · 626d54e8

authored Feb 07, 2014

D3D10 cards perform poorly when the pixel shader outputs to multiple
render targets that are not bound.

BUG=325581

Change-Id: I13f6f98861ffeb77cd21f7401ded4425e4ea7841
Reviewed-on: https://chromium-review.googlesource.com/185400Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/190341

626d54e8

Update gclient configuation for new git repository. · 0c0481a1

authored Mar 17, 2014

Cherry-pick of bd79517a.

Change-Id: I87d790c14d68789524df76f63a61fc30a8792e4e
Reviewed-on: https://chromium-review.googlesource.com/190350Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

0c0481a1

Cache bound vertex and index buffers by ID3D11Buffer instead of serial. · 7840b170

authored Mar 13, 2014

Since a DirectBuffer11 may return different buffers from getBuffer
depending on the required buffer type and if the internal buffers are
dirty or not, the current serial is not reliable.  Use the actual buffer
object so that there can be no missmatch.

BUG=350581

Change-Id: I6ea1616273346117a17646edb68e6518e584f2b1
Reviewed-on: https://chromium-review.googlesource.com/189435Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

7840b170

14 Mar, 2014 4 commits

Fix incorrect path usage in generate_projects. · 72b16b25

authored Mar 06, 2014

We should use the cross-platform os.path.join() to support
platforms other than Windows.

BUG=angle:577

Change-Id: I9d96fc1b13731a57ffcd97469632f0acedb8e5c7
Reviewed-on: https://chromium-review.googlesource.com/188879Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

Conflicts:

	generate_projects

Change-Id: I76cacb8ee3acf477c0e99cd73cf91622efc3c9b4
Reviewed-on: https://chromium-review.googlesource.com/189131Tested-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

72b16b25

Fix incorrect function forward declaration. · 9a2e5b94

authored Mar 13, 2014

Change-Id: I135c4457fdc95f2d331e2924104a30f90a40bd1a
Reviewed-on: https://chromium-review.googlesource.com/189456Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

9a2e5b94

Refactor ReadPixels validation. · 26e9195f

authored Mar 05, 2014

Move ReadPixels error generation out of the implementation and into
the API level.

BUG=angle:571

Change-Id: I0b32294f359fedd13d1af2c95baf37a3e5ac1d5b
Reviewed-on: https://chromium-review.googlesource.com/188014Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

26e9195f

Implement queries and enums for buffer mapping. · 70656a64

authored Mar 05, 2014

GetBufferPointerv, GetBufferParameteri64v, and new values
for GetBufferParameteriv.

BUG=angle:565

Change-Id: I179a3193987b2bff1b0628e47f6a0d627541b0cf
Reviewed-on: https://chromium-review.googlesource.com/188203Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

70656a64

12 Mar, 2014 1 commit

Don't attempt to perform constant folding with an uninitialized constant variable. · bd0ea9c7

authored Mar 04, 2014

BUG=348205

Change-Id: I9be17832b1a35d2995725ab147977992c2732274
Reviewed-on: https://chromium-review.googlesource.com/188711Tested-by: Nicolas Capens <nicolascapens@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/189471

bd0ea9c7

11 Mar, 2014 2 commits

Regenerate public projects. · 3f9e7eae

authored Mar 11, 2014

Change-Id: I0a385ff823ef492b4fced89a734d8053fdb0ffd0
Reviewed-on: https://chromium-review.googlesource.com/189474Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Zhenyao Mo <zmo@chromium.org>

3f9e7eae

Remove SH_MAP_LONG_VARIABLE_NAMES · 3cdfcce8

authored Mar 07, 2014

We use hashing to map all variables/strcuture field names, etc,
so we no longer need this option.

Checked with Firefox and WebKit, they no longer use this option
either. Time to remove it.

Change-Id: Ie3e79b91a05258b04af419a9c42b2fd1b00e67c4
Reviewed-on: https://chromium-review.googlesource.com/189236Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/189568

3cdfcce8