Change-Id: If012660d4b895399fe5e3226dea25a8c7f78c8ac
Reviewed-on: https://chromium-review.googlesource.com/239504Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Cooper Partin <coopp@microsoft.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

BUG=angle:658

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

The implementation uses asint, asuint and asfloat in the HLSL output.

BUG=angle:865

Change-Id: I6b48298e18c7b3b0bdeac522b375ebc4eab7cf4e
Reviewed-on: https://chromium-review.googlesource.com/239520Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Olli Etuaho <oetuaho@nvidia.com>

Change-Id: I1b02ed7209f30549c17f46aeb3726ba1254df3ea
Reviewed-on: https://chromium-review.googlesource.com/239270Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:858

Change-Id: Ib2b2b2b54ba25654ea80b5cb1c59b74ed0379b68
Reviewed-on: https://chromium-review.googlesource.com/239271Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:858

Change-Id: I0fe3baf811d561584f7d24f6da665c6934e32b2a
Reviewed-on: https://chromium-review.googlesource.com/239272Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

BUG=angle:731

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

BUG=angle:849

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

We can switch this on for Debug only. This will help developers
catch bugs more easily, and will only show up for internal
errors and major performance caveats. Currently it's far too easy
to miss these messages - it requires manually modifying debug.h,
looking at a text file and potentially also running Chromium with
an special flag.

BUG=angle:663

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

We use this format as a backbuffer texture for the EGL surface in
some configs. Thus, the app needs to be able to use it, since they
don't know when they are secretly using BGRA instead of RGBA,
similar to our existing logic which allows BGRA in CopyTexImage.

BUG=angle:811

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

BUG=angle:861

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

BUG=angle:861

Change-Id: I7cd2d1a56772fdf18bcf926456399322d13e7a4f
Reviewed-on: https://chromium-review.googlesource.com/236305Reviewed-by: Brandon Jones <bajones@chromium.org>
Tested-by: Geoff Lang <geofflang@chromium.org>

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

BUG=angle:849

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

Change-Id: Ic5f2aca0100a842be193cec9e89872187742f2c6
Reviewed-on: https://chromium-review.googlesource.com/238821Tested-by: Cooper Partin <coopp@microsoft.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

This c++11 standard library class isn't available on Chromium/OSX.
For now we can replace it with std::map.

BUG=angle:773

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

These classes has base RendererD3D which already have "mDeviceLost".
This can lead to errors because this variable is used in the methods
from RendererD3D and his derived classes

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

BUG=angle:862

Change-Id: I8e7fa9b0d00bb1b2a32a8d60d8ceda998cea8e8c
Reviewed-on: https://chromium-review.googlesource.com/239160Tested-by: Gregoire Payen de La Garanderie <Gregory.Payen@imgtec.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

BUG=angle:858

Change-Id: I6197d0f86536cb4613aa1aeb2d73a8173b4e4631
Reviewed-on: https://chromium-review.googlesource.com/238530Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

This fixes the compile errors in loadImageSSE2.cpp.

BUG=angle:773

This reverts commit a93192369eb448e932bcdd1e08ae791aacfbe0c4.

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

The _rotl method is unavailable on any platform but Windows,
so we can implement the method ourselves.

BUG=angle:773

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

This fixes compile errors on OSX.

BUG=angle:773

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

We will need this functionality for index validation, so we likely
can assume an implemenation will need to support such a call. We
might redesign this requirement, but for now we can't call though
to BufferD3D on OSX.

BUG=angle:773

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

We were missing one warning diable flag, for bison compatiblity,
and were missing the appropriate flag to enable c++11 support.

BUG=angle:773

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

We can't use this method on OSX due to missing c++11 STL support.
Instead use a helper function to replace it.

BUG=angle:773

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

In deleteTransformFeedback, we were using a const iterator, in a
non-const method. Fix the comparison by using auto, which delegates
more work to the compiler's static type checking.

BUG=angle:773

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

These changes we'll have to upstream at a later time.

BUG=angle:773

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

std::tuple currently isn't available on Chromium/OSX.

BUG=angle:773

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

This fixes the compile on OSX.

BUG=angle:773

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

We were using the c++11 iterator return value from std::map::erase.
Instead use the post-increment operator to keep a valid iterator
after the call to erase, even on older stl.

BUG=angle:773

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

We need the functionality of this extension to support dEQP. Likely
we only needed the minor version enum to not generate an error, but
we can take the time to implement the few cases for the other enums
as well.

We don't support any debug context functionality currently, but we
may want to support it in the future.

BUG=angle:501

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

We were not advertising the extension, though we require it for
ANGLE_platform_angle. This was blocking dEQP functionality.

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

It is quite valid for the app to pass in NULL for the attribs list.
In this case we currently crash. We could trigger this error using
dEQP.

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

*_TYPELESS formats weren't supported in D3D10Level9 until
Windows 8. Some Win8 D3D9 drivers don't support them either.
To workaround this, we avoid _TYPELESS formats on D3D11 FL9_3.

BUG=angle:856
BUG=435726

Change-Id: I280dc7f87e3a2c737c14284ebb744188e7f10616
Reviewed-on: https://chromium-review.googlesource.com/237292Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Tested-by: Austin Kinross <aukinros@microsoft.com>
Tested-by: Jamie Madill <jmadill@chromium.org>

Like D3D9, D3D11 Feature Level 9_3 doesn't support large or negative
viewports. We have to emulate these in the vertex shader.

BUG=angle:858

Change-Id: I2bd53e3921dc3590cc7193164d73596deafca9ea
Reviewed-on: https://chromium-review.googlesource.com/236040Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

D3D11 Feature Level 9_3 supports instancing, but slot 0 in the input
layout must not be instanced. D3D9 has a similar restriction, where
stream 0 must not be instanced. This restriction can be worked around
by remapping any non-instanced slot to slot 0. This works because
HLSL uses shader semantics to match the vertex inputs to the elements
in the input layout, rather than the slots.

BUG=angle:858

Change-Id: I67b2be9095afc206a4b9f107ed61356820551afe
Reviewed-on: https://chromium-review.googlesource.com/237270Tested-by: Austin Kinross <aukinros@microsoft.com>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

ANGLE's testing code recently got enabled in Chromium's builds. While it
builds fine with cl.exe, it isn't quite standards-conformant and doesn't
build with clang. Fix this.

There were three issues:

1. ANGLE_TYPED_TEST_CASE() is a variadic macro that tries to use
   __VA_ARGS__ as argument to a variadic template and then pass
   that template to another macro. However, [cpp.replace] describes
   that ANGLE_TYPED_TEST_CASE(Test, int, float) should be expanded
   to TYPED_TEST_CASE(Test, ::testing::Types<int, float>) which
   should be interpreted as a "call" of TYPED_TEST_CASE with the
   3 macro arguments `Test`, `::testing::Types<int`, and `float>`.
   As a fix, use a typedef for the variadic template and refer to
   it through the typedef in the macro call.

2. `#version` was used on its own line in a substitution of the
   SHADER_SOURCE macro. [cpp]p1 says that every line starting with
   a `#` is a preprocessing directive, and [cpp.replace]p11 says
   "If there are sequences of preprocessing tokens within the list
   of arguments that would otherwise act as preprocessing directives,
   the behavior is undefined" (with a footnote that this includes
   non-directives -- # followed by unknown text). As a fix, merge the
   `#version` line with the previous line. Now the line doesn't start
   with `#` and things are fine.

3. Unqualified lookup usually doesn't look into dependent bases. If
   this is desired, one usually has to make the call qualified, a
   good explanation for this is at
   http://eli.thegreenplace.net/2012/02/06/dependent-name-lookup-for-c-templates
   cl.exe doesn't implement this rule, and clang tries to emulate
   cl.exe's behavior to a certain extend when in Microsoft mode.
   However, that seems to not work for member templates with explicit
   types (filed http://llvm.org/PR22066 for this, but since it's not
   needed to parse Microsoft headers and not standards-conformant,
   I'm not sure if we'll fix that). As a fix, don't provide an explicit
   type, the inferred type is the same. This is also consistent with
   all the other tests in this file. (We might clean up -Wmicrosoft
   warnings in the future; if so I'll add the explicit this->s that
   are missing in this file when we do.)

BUG=chromium:445406

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

BUG=angle:658

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

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

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