Commits · 3dc300a592756d292225141eded4a3a61bd511da · Chen Yisong / angle

26 Mar, 2014 1 commit

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

10 Mar, 2014 1 commit

Regenerate default Windows projects · 276abc85

authored Mar 07, 2014

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

276abc85

07 Mar, 2014 5 commits

Improvement on loop unrolling with loops indexing sampler arrays · 550c600b

authored Feb 26, 2014

    1) Before this workaround is hardwired on mac, now we move it behind a compil
    2) Fix the issue where "break" inside the loop isn't handled while unrolled.

    BUG=338474
    TEST=webgl conformance test sampler-array-using-loop-index.html

Change-Id: I4996a42c2dea39a8a5af772c256f8e3cb383f59a
Reviewed-on: https://chromium-review.googlesource.com/188079Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Tested-by: Zhenyao Mo <zmo@chromium.org>

Conflicts:
	include/GLSLANG/ShaderLang.h
	src/compiler/translator/ValidateLimitations.cpp
Change-Id: I546197bd7df1634ebccdd380be14c3250cd56151
Reviewed-on: https://chromium-review.googlesource.com/189061Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Zhenyao Mo <zmo@chromium.org>

550c600b

Fix compile errors on pre-C++11 compilers. · 2515c520

authored Mar 07, 2014

BUG=angle:575

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

2515c520

Keep the debug log file open and flush with each message. · feee44bf

authored Mar 05, 2014

Instead of opening and closing the debug log file with each message, keep
it open permanently and flush it with each message, allowing the OS to
flush to disk as needed.  With debug logging enabled, performance is improved
by 100x!

Also, dynamically size the output buffer to the max message size.  When
running all the samples, the largest debug message was 318 characters so
an initial size of 512 (instead of 32768) seems reasonable.

BUG=angle:575

Change-Id: If4a1387431fc2d32a6b2d1f9ca1f2590473feb86
Reviewed-on: https://chromium-review.googlesource.com/188713Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

feee44bf

Implement buffer map related entry points and errors. · 7a5f738b

authored Mar 05, 2014

BUG=angle:565
Change-Id: I14cee4eec032d4b6b8d9479da4234b81e4ec6bec
Reviewed-on: https://chromium-review.googlesource.com/188202Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

7a5f738b

Add helper methods for buffer enum tagets. · 8c96d58a

authored Mar 05, 2014

A query helper method in Context and a validation helper
method reduce code redundancy quite a bit.

BUG=angle:565

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

8c96d58a

05 Mar, 2014 3 commits

Added methods for applying the current transform feedback buffers. · eeba6e1e

authored Feb 03, 2014

BUG=angle:495

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

eeba6e1e

Fix default bits for 3-channel un-normalized int. · ca442ba4

authored Mar 05, 2014

GPU conversion means we should be using a default of 1 as for
int types instead of the MAXINT value for normalized types.

Fixes the WebGL test for vertex attribs with offsets.

BUG=angle:550

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

ca442ba4

Combine widen and defaultBits params of copyVertexData. · 2f612290

authored Mar 05, 2014

Since we never, and never will use a default value of zero
for the alpha channel, we can use a value of zero to
indicate no widening. This has the benefit of not needing
a default value where it does not apply.

BUG=angle:550

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

2f612290

04 Mar, 2014 1 commit

Emulate integer cube texture sampling as an array of six 2D textures. · 0027fa9f

authored Feb 20, 2014

BUG=angle:525

Change-Id: I3c3ec2cecebf9e745f0c02a132433e3076a6fdea
Reviewed-on: https://chromium-review.googlesource.com/187534Tested-by: Nicolas Capens <nicolascapens@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

0027fa9f

03 Mar, 2014 1 commit

Update enumerate_files.py to ignore "hidden" files. · 9c90d7ed

authored Feb 27, 2014

Skip over files that begin with '.' since they are often hidden temporary
files.

BUG=angle:553

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

9c90d7ed

27 Feb, 2014 1 commit

Remove the second translator target, only using static. · d51df461

authored Feb 26, 2014

Because libGLESv2 needs a static translator to link against
internal types like Attribute and Varying, as well as using
some other internal methods, we can for now disable the
component build for the translator. In the future we can
support a proper component build with exports across library
boundaries.

BUG=angle:568

Change-Id: Ie3efc0a29342d02491e91952c3c2398568f83576
Reviewed-on: https://chromium-review.googlesource.com/187699Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

d51df461

26 Feb, 2014 5 commits

Fix crash on multiple calls to ShInitialize(). · 477bc781

authored Feb 26, 2014

The previous fix would allow the calling app to trigger
an ASSERT if it did not first call ShFinalize. Chrome
would do this in some cases, causing a test failure.

BUG=344043

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

477bc781

Prevent overflow in the variable packer by sanity checking each variable. · 1d1c1152

authored Feb 26, 2014

BUG=346489

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

1d1c1152

Re-initialize shader translator properly after one shutdown. · b4d192cd

authored Feb 26, 2014

Because of a local static, we could get stuck and never re-initialize
the translator after calling ShFinalize.

BUG=344043

Change-Id: Ib4d18da886eac28f3e1bc959a7eadf5b34d9410f
Reviewed-on: https://chromium-review.googlesource.com/186975Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/187848

b4d192cd

Implement BufferStorage11 mapping functions. · 9533dcd3

authored Feb 24, 2014

BUG=angle:565
Change-Id: I4f416981bc63098f0364653ff279b3f54eeb756e
Reviewed-on: https://chromium-review.googlesource.com/186979Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

9533dcd3

Implement state and interface for map in gl::Buffer. · 103a11f4

authored Feb 24, 2014

Mostly these calls pass through to BufferStorage.

BUG=angle:565
Change-Id: I38ab7d4f49bfad940773428f5bb330de55d3f573
Reviewed-on: https://chromium-review.googlesource.com/187080Reviewed-by: Nicolas Capens <nicolascapens@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

103a11f4

25 Feb, 2014 1 commit

Update codereview.settings to point to gerrit. · dd6badf0

authored Feb 25, 2014

BUG=angle:570

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

dd6badf0