Commits · aed9f94ea3e6355a249e523461945c5bf5249ecd · Chen Yisong / angle

19 Nov, 2014 1 commit

Limit D3D11 Feature Level 9 to GL ES 2.0, and say it's not conformant · aed9f94e

authored Oct 23, 2014

Change-Id: I2ce88217c9b78e83bac6df5975d9edfbbf90e557
Reviewed-on: https://chromium-review.googlesource.com/225251Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

aed9f94e

18 Nov, 2014 1 commit

Change the component type of STENCIL_INDEX8 to UNSIGNED_NORMALIZED · e0902644

authored Nov 04, 2014

That way RenderbufferStorageMultisample accepts STENCIL_INDEX8 as
an internal format for a multisampled buffer (samples > 0) as it
isn't concerned by the restriction on integer component types anymore.
This is consistent with the component type returned by the NVIDIA driver.

BUG=angle:812

Change-Id: Ic03f502ffa082b1011e8127213a5c1df0617ef43
Reviewed-on: https://chromium-review.googlesource.com/227470Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

e0902644

17 Nov, 2014 2 commits

Copy the D3D compiler dll in the GN build. · ea0c84d1

authored Nov 11, 2014

Change-Id: I773a411d07991d9ae160267eba8325c327146554
Reviewed-on: https://chromium-review.googlesource.com/229007Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: John Abd-El-Malek <jam@chromium.org>

ea0c84d1

Unset SRVs which resource are being bound to an RTV · ac6ff938

authored Nov 12, 2014

This is the extension of a previously debug-only workaround for
D3D doing the unsetting by itself (with a warning). It made our
state tracking of the bound SRVs out of sync with D3D's and caused
some rendering to not take place.

BUG=427245

Change-Id: Id378030b64fddafbb3d406872a7dfd2c84793221
Reviewed-on: https://chromium-review.googlesource.com/229066Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Corentin Wallez <cwallez@chromium.org>

ac6ff938

15 Nov, 2014 1 commit

Revert "Revert "Switch to 8.1 SDK and d3dcompiler_47.dll"" · 7e81c9d5

authored Nov 15, 2014

This reverts commit 3cbdc34d.

This reverts back to using 8.1 and d3dcompiler_47, as chrome is now rolled forward.

Change-Id: I7ef4ebdbc32146b9ba6f551e376ac03ac71131ba
Reviewed-on: https://chromium-review.googlesource.com/229990Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
Tested-by: Scott Graham <scottmg@chromium.org>

7e81c9d5

14 Nov, 2014 4 commits

Revert "Switch to 8.1 SDK and d3dcompiler_47.dll" · 3cbdc34d

authored Nov 14, 2014

This reverts commit e1425d95.

This is to keep the GPU fyi bots green until the chrome side is landed. Chrome will be rolling to e1425d95...

Change-Id: Iff1c60eeb7143fb72afa8525b31f8f0de9385897
Reviewed-on: https://chromium-review.googlesource.com/229497Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

3cbdc34d

Switch to 8.1 SDK and d3dcompiler_47.dll · e1425d95

authored Nov 13, 2014

Change-Id: If43c7861ab7f47448fd726fd0b5c35f3a6eeb362
Reviewed-on: https://chromium-review.googlesource.com/229616
Commit-Queue: Scott Graham <scottmg@chromium.org>
Tested-by: Scott Graham <scottmg@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

e1425d95

Added Trim support for Windows Store application certification requirement · e9bdeb25

authored Nov 04, 2014

Change-Id: Iff58863bf4af756d77521aadb1ccbaa977f0b78a
Reviewed-on: https://chromium-review.googlesource.com/227530Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

e9bdeb25

Fix precision tracking of constructor return values · 21203702

authored Nov 13, 2014

Precision should be set for constructor return values if they are of a
built-in type. Structs should not be precision qualified.

BUG=angle:787

Change-Id: Ie5efd5be25a788ff6f01c5b989254572c00231eb
Reviewed-on: https://chromium-review.googlesource.com/229560Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Nicolas Capens <capn@chromium.org>
Tested-by: Olli Etuaho <oetuaho@nvidia.com>

21203702

13 Nov, 2014 3 commits

Remove redundant rx:: scoping prefixes. · 30d6c255

authored Nov 13, 2014

In many places we can get rid of this scoping operator.

BUG=angle:789

Change-Id: If6c40c435a97473d2fd9d6f1fb5a2c709393eb00
Reviewed-on: https://chromium-review.googlesource.com/225566Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

30d6c255

Remove the unused Renderer client version methods. · b38f2e5d

authored Nov 13, 2014

Previous patches enable us to get rid of this workaround. We should
in subsequent patches also try to clean up code that uses the
internal Renderer-only version of the context caps.

BUG=angle:789

Change-Id: Ic054090303cdc3f7ee8e45e4577c157559955472
Reviewed-on: https://chromium-review.googlesource.com/227714Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

b38f2e5d

Pass GL context data into shader creation. · 245d3602

authored Nov 13, 2014

This allows us to remove the hack where we store the current GL
client version in the Renderer.

BUG=angle:789

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

245d3602

12 Nov, 2014 1 commit

Use the [[flatten]] attribute only when a loop is present. · 80bacde5

authored Nov 10, 2014

Flattening branch-heavy shaders that contained no loops caused regressions.
As a temporary workaround we only flatten ifs when there exists a loop.

BUG=395048

Change-Id: I95c40f0249643b98c62304a0f2a4563561d1fbbc
Reviewed-on: https://chromium-review.googlesource.com/228722Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Corentin Wallez <cwallez@chromium.org>

80bacde5

11 Nov, 2014 7 commits

Fix ProgramBinary being disabled when it should be enabled. · 560eef16

authored Nov 11, 2014

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

560eef16

Refactor Framebuffer attachment application. · ab75a056

authored Oct 15, 2014

Instead of passing texture and renderbufer IDs which requires a call to
gl::getNonLostContext to resolve the objects, pass the Texture and Renderbuffer
objects directly.

BUG=angleproject:733

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

ab75a056

Pass GL caps and version to compiler. · de8892b4

authored Nov 11, 2014

This allows us to get rid of some of the reliance on storing the
current context client version in the Renderer. A subsequent
patch will allow us to remove the client version ugly hack.

BUG=angle:789

Change-Id: I139e0f66e1d39e5cd41a484c841a7101b1f29540
Reviewed-on: https://chromium-review.googlesource.com/227712Reviewed-by: Brandon Jones <bajones@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

de8892b4

Move the Context ReadPixels logic into RendererD3D. · 87f6da1f

authored Nov 11, 2014

BUG=angle:789

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

87f6da1f

Move BlitFramebuffer logic into RendererD3D. · a93bef2e

authored Nov 11, 2014

BUG=angle:789

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

a93bef2e

Split the RenderTarget classes based on if they are backing a SwapChain. · 1bd642c5

authored Nov 04, 2014

bug=angle:824

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

1bd642c5

Fix compilation failure with Visual Studio 14. · df8fafef

authored Nov 11, 2014

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

df8fafef

10 Nov, 2014 5 commits

Pass 0 for the sample count to RenderTarget when there is no multisampling. · 23419d56

authored Nov 07, 2014

BUG=angle:520

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

23419d56

Move Clear logic into RendererD3D. · a153d4b0

authored Nov 06, 2014

BUG=angle:789

Change-Id: I56d4398e4c49fe980f649327b105cdba9d944a00
Reviewed-on: https://chromium-review.googlesource.com/226062Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>

a153d4b0

Move Context draw call logic into RendererD3D. · d9e58303

authored Nov 06, 2014

Also move a lot of supporting code.

BUG=angle:789

Change-Id: I098bf7d072ece1f414605783c32ec5354ba63e19
Reviewed-on: https://chromium-review.googlesource.com/226061Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>

d9e58303

Fixes a bug in program ID validation for GetUniform* calls. · 4de4fd6e

authored Nov 07, 2014

BUG=angle:571

Change-Id: I7080f0f7ea0def29d1b67c9d954ee5d9d41fb348
Reviewed-on: https://chromium-review.googlesource.com/228560Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

4de4fd6e

MANGLE the ANGLE Renderer class. · 93e13fbf

authored Nov 06, 2014

BUG=angle:789

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

93e13fbf

07 Nov, 2014 5 commits

Fix zero texture application. · e6382c3b

authored Nov 07, 2014

Patch https://chromium-review.googlesource.com/#/c/227711/ would
allow NULL Textures sometimes from State::getSamplerTexture. Fix
this by always setting the zero "Default" textures instead of NULL
in the State.

This was breaking the ES2-CTS test 'framebuffer_objects' on Windows.

BUG=angle:826

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

e6382c3b

Remove the gl::error function. · b28126ef

authored Oct 03, 2014

BUG=angle:520

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

b28126ef

Add error handling to RenderTarget creation. · bafefdf3

authored Oct 10, 2014

 * RenderTarget9/11 don't create any resources themselves anymore.
 * Pass the real internal format to RenderTarget creation so that it is
   not lost by converting to D3D formats and back.

BUG=angle:520

Change-Id: If420970d42f0bf6ce392d64f9cb7efa4df0b8f8e
Reviewed-on: https://chromium-review.googlesource.com/222838Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

bafefdf3

Regenerate projects. · 468d7a1e

authored Nov 07, 2014

Disabled warnings were reordered.

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

468d7a1e

Fix possible crash when the info log length is zero. · 981afd7f

authored Nov 05, 2014

BUG=angle:822

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

981afd7f

06 Nov, 2014 5 commits

Add RendererD3D as a layer between GL and Renderer9/11. · ad65a13c

authored Nov 06, 2014

BUG=angle:789

Change-Id: I6107e6cbdeb5e3466f96a4bdf98181b8e5a25467
Reviewed-on: https://chromium-review.googlesource.com/225471Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

ad65a13c

Use Context caps in Framebuffer completeness check. · 48faf80a

authored Nov 06, 2014

This frees the Framebuffer from any dependency on the Renderer.

BUG=angle:789

Change-Id: I0e9ae0edf93674e32ae4f6538a934cddbdc804d5
Reviewed-on: https://chromium-review.googlesource.com/225470Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Jamie Madill <jmadill@chromium.org>

48faf80a

Add a base for GL essential data. · 1e9ae079

authored Nov 06, 2014

The data holds the GL state, caps, extensions, texture caps, and
current client version.

BUG=angle:789

Change-Id: Icd15d806e14490f39041dea663ab2461a6a76090
Reviewed-on: https://chromium-review.googlesource.com/226060Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>

1e9ae079

Merge zero textures into the state texture map. · dedd7b95

authored Nov 05, 2014

We can treat these textures the same as the rest, saving a bit of
code and complexity cost.

BUG=angle:789

Change-Id: I14b426c817cbe3c5bb737410378b6a80e274689b
Reviewed-on: https://chromium-review.googlesource.com/227711Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>

dedd7b95

Fix platform if defined build error on Linux. · d46275f9

authored Nov 06, 2014

Using #if instead of #if defined() was giving an error when
ANGLE_PLATFORM_WINDOWS was undefined. Instead of defining it as
zero for all platforms, use #if defined() to ensure our legacy
usage of #ifdef doesn't get messed up by #define <value> 0.

BUG=angle:825

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

d46275f9

05 Nov, 2014 5 commits

Switch to new APIs in samples/translator/translator.cpp · 0bdc0f3e

authored Nov 04, 2014

Also, fix a few issuse for ninja build "all" to finish.

BUG=angle:775
TEST=translator.cc compiles OK

Change-Id: I6507e9143882a628a7737dd139efb6d8d358b0db
Reviewed-on: https://chromium-review.googlesource.com/227490Tested-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

0bdc0f3e

Fixes for Angle to build with Windows GN. · 657cd684

authored Nov 05, 2014

Change-Id: Id38c4b11dba31b2779efa7beade481cd50f3d935
Reviewed-on: https://chromium-review.googlesource.com/227692Reviewed-by: Shannon Woods <shannonwoods@chromium.org>
Tested-by: Shannon Woods <shannonwoods@chromium.org>

657cd684

Fix fuzzy color check in ClearTest. · 9abdc2d8

authored Nov 05, 2014

This test was failing on some systems because of overly stringent
color expectations.

BUG=angle:809

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

9abdc2d8

Fixes for Angle to build with Windows GN. · 330f0ade

authored Nov 05, 2014

Change-Id: I5af7a1305e5a190de5367ad940186edd6baee5bc
Reviewed-on: https://chromium-review.googlesource.com/227665Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

330f0ade

Use an ES3 shader spec for ES3 contexts. · 0af0b38e

authored Nov 05, 2014

BUG=angle:823

Change-Id: I377b4483b3835660e7fd45d923bf7f560cf2e5f6
Reviewed-on: https://chromium-review.googlesource.com/227593Tested-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Nicolas Capens <capn@chromium.org>

0af0b38e