Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: I0e30246d8afce9f4b3704c941c57e9fe88098268

Merge aosp/master into gerrit/vulkan-cts-1.0-dev

Change-Id: I6d61156e336937d8c2f534a410b6f2a92987f037

Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: If9a06423b0fb4be658187ee4df49817dcc9b0394

Merge gerrit/vulkan-cts-1.0.2 into gerrit/vulkan-cts-1.0-dev

Change-Id: If550d50796597ec466efb30b1cbbd9aab4b13890

Merge gerrit/vulkan-cts-1.0.1 into gerrit/vulkan-cts-1.0.2

Change-Id: I4dc1513b1528b8182cfd75f4ed9184fa8f383e47

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: I3ace822f33fb20acd6109886458b0d16c1624bd5

Fix assert for combined depth/stencil formats

This was using tcu::isCombinedDepthStencilType(format.type) for this,
but this only checks the type and for 24bit depth formats with a
x8 d24 setup it will incorrectly identify them as combined
depth/stencil even though the underlying format is depth-only. Fix
this by testing if the format has depth and stencil components
instead.

Prevents crashes in tests with the following signature (144 tests):
dEQP-VK.texture.shadow.*x8_d24*

Fixes #563

Change-Id: Ie43e140a0461de49c02e0d97eeed38fa8ef24040

Fix buffer size and alignment issues in api.buffer

- Don't use maxTexelBufferElements, it doesn't apply here
- Remove unused throw/catch blocks (catch block was never reachable)
- Add a case that exercises huge buffer sizes
- Add a check for sparseAddressSpaceSize

Modified tests:
- dEQP-VK.api.buffer.create_buffer_*

Fixes #558

Change-Id: Ia6b46d8c34a4eef4a6136532d0cb626d0b64b560

Update zlib and libpng dependencies

Fixes #568

Change-Id: I8ee890a2176bf6d42d3dd8b119b7263034127960

Merge "Merge aosp/upstream-vulkan-cts-1.0-dev into aosp/master"

DO NOT MERGE: Remove changed texel buffer access tests from mustpass
am: 302bd7c6b3  -s ours

Change-Id: I1cefd7be538f766444576ae02a4b28b639a3e8cb

DO NOT MERGE: Remove a few changed Vulkan tests from mustpass
am: 09e8b66d03  -s ours

Change-Id: I595b0c3b589b73c806145602957875b57cffe84a

Merge aosp/upstream-vulkan-cts-1.0-dev into aosp/master

Change-Id: Id23cc2f5a2ff4238fbf56d8c9cd4330dd166971a

Add support for EGL_EXT_pixel_format_float

This updates the egl.xml version and identifies/skips the float configs

Bug: 32969314
Change-Id: Iadeb0d4a76a06b2be716f3e76a1cc9fb69aff16a

DO NOT MERGE: Remove changed texel buffer access tests from mustpass

NYC CTS bug-fix changes the SPIR-V for these tests causing drivers that
passed old CTS fail the new version.

Bug: 33041922
Change-Id: I7e23a700693d5e6ee7c4264e05d449f49f820516

DO NOT MERGE: Remove a few changed Vulkan tests from mustpass

Original NYC CTS was decorating struct member locations differently and
this was later fixed causing drivers that pass original NYC CTS fail.

Bug: 34060750
Bug: 33041922
Change-Id: I3fe33960a74fb8f568aa7e1327bc09de8bc3df41

Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: Icef189b9af29e090fbb4940af970d91951c2d6e6

Merge gerrit/vulkan-cts-1.0.2 into gerrit/vulkan-cts-1.0-dev

Change-Id: I598c6d52ece7b00a8dd0af10b31ea144ebc6ba80

Revert "Add new files to Android.mk"

This reverts commit 4ed5a72717f9dc776f7f63aa21aca2c727524d0f.

Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: Ieccd2e74ca11d5a4b7b06f78bae0094e5a870b69

Merge gerrit/vulkan-cts-1.0.2 into gerrit/vulkan-cts-1.0-dev

Change-Id: I33cca201217f9c9a258c61e0bb6178bb7694a96e

Add new files to Android.mk

Change-Id: Ib396f9504a3b5523314cd2d5ab9375a346dbb219

Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: I6ae311b56b24f91aae76a908b8086bcc8afeaacf

Merge gerrit/vulkan-cts-1.0.2 into gerrit/vulkan-cts-1.0-dev

Change-Id: I4b9cb490746a875501c5213b0a5899a3a487be03

Merge gerrit/vulkan-cts-1.0.1 into gerrit/vulkan-cts-1.0.2

Skip integer formats that expose a max. sample count of 1

Actually fixes #538

As agreed in the ticket we need to skip integer formats if they
need to be sampled and only expose a max. sample count of 1.

Change-Id: Ia67c8d84cedcae09804e83cfefeb279b329d07b9

install.py continues if adb uninstall fails

ADB uninstall fails with non-zero return code if the package to be
uninstalled does not exist. To allow the installation proceed when
there's no previous package, the non-zero return value should be ignored
for the uninstall step.

There are historical reasons for not relying on -r -d for
install (replace, allow downgrade), which I can't recall right now.

Test: Installed succesfully on device without deqp package.
Bug: 33961937
Change-Id: I401f3861f8d68f352747c041bd60cd372b48134b

Merge "Blacklist match_different_[member_]struct_names SSO validation tests." into nougat-cts-dev
am: 4acf725866

Change-Id: I2cc66f7fb73fecf721da7704aa5a35384c5f5d4d

resolve merge conflicts of b81edb4 to master

Change-Id: I16c7230358a1758afe1bc9ca2b22ae6eff2490b7

Merge "Change GL_MAX_TESS_CONTROL_TOTAL_OUTPUT_COMPONENTS minmax to 2048" into nougat-cts-dev
am: ce7ccaab4a

Change-Id: I36f440b4feadda51d9a47c856a820986800c387b

Change GL_MAX_TESS_CONTROL_TOTAL_OUTPUT_COMPONENTS minmax to 2048
am: a42836566d

Change-Id: I6cf9f214b1a66360823af5fb219860454bc5a275

Merge changes I7c2ffd66,Ib53b245d,I4862cca3 into nougat-cts-dev
am: 4812d6d47f

Change-Id: I6c5c6201bedac30702f65c2633454dc6bd27b4a5

Update glslang to d02dc5d
am: b0ce1f083e  -s ours

Change-Id: I12b4d9962c21073cfc89766b87049f4140d69f2a

Cherry-pick SPIR-V ClipDistance validation fixes
am: ef29b7735d  -s ours

Change-Id: I05ec2bc33b606a2500f7991e60a9823b08a5868d

Cherry-pick dEQP-VK.pipeline.cache fixes to nougat-cts-dev
am: e71e3bbacd

Change-Id: Id4477e4ab350a34791c9a59bec747d487b31d5c5

Merge "Blacklist match_different_[member_]struct_names SSO validation tests." into nougat-cts-dev

Merge "Change GL_MAX_TESS_CONTROL_TOTAL_OUTPUT_COMPONENTS minmax to 2048" into nougat-cts-dev

Merge changes I7c2ffd66,Ib53b245d,I4862cca3 into nougat-cts-dev

* changes:
  Update glslang to d02dc5d
  Cherry-pick SPIR-V ClipDistance validation fixes
  Cherry-pick dEQP-VK.pipeline.cache fixes to nougat-cts-dev

Correct a typo in blit image test case names

Modified test names:
- dEQP-VK.api.copy_and_blit.blit_image.simple_tests.*

Change-Id: I20b09407101eb3e395d10f324e9f402f5c1c4b77

Correct a typo in blit image test case names

Modified test names:
- dEQP-VK.api.copy_and_blit.blit_image.simple_tests.*

Change-Id: Ia62b9f616466b919f1e92d0a74714586ccc6e313

GetDevices: Also allow `\.` to be part of the host name am: dbf9715509
am: 133c610be6

Change-Id: I21054992a16e3e6847a8af086528f4172602d614

GetDevices: Also allow `\.` to be part of the host name
am: dbf9715509

Change-Id: I659ce579867c7ca0c9b47e3b53a734265e0fbb26

GL_ARB_arrays_of_arrays: unsized dimension only array's outermost

From the GL_ARB_arrays_of_arrays spec:

   "For unsized arrays, only the outermost dimension can be lacking a
    size. A type that includes an unknown array size cannot be formed
    into an array until it gets an explicit size."

This is later further detailed as per OpenGL® Shading Language 4.40.

From page 3 (page 9 of the PDF) of the GLSL 4.00 v9 spec:

  "* Bug 11987: Allow array-of-array constructors and initialized
     variables to have any dimension unsized (not just the outer
     dimension) and get their size from the content of the constructor
     or initializer."

From page 37 (page 43 of the PDF) of the GLSL 4.00 v9 spec:

  " vec4 a[][] = { vec4[2](vec4(0.0), vec4(1.0)), // okay, size to
                   vec4[2](vec4(0.0), vec4(1.0)), //       a[3][2]
                   vec4[2](vec4(0.0), vec4(1.0)) };"

From page 38 (page 44 of the PDF) of the GLSL 4.00 v9 spec:

  "For implicitly-sized or run-time-sized arrays, only the outermost
   dimension can be lacking a size. A type that includes an unknown
   array size cannot be formed into an array until it gets an explicit
   size, except for shader storage blocks where the only unsized array
   member is the last member of the block."

Hence, in the absence of a constructor/initializer, unsized arrays are
only allowed for the outermost dimension of an AoA.

Fixes #22.

Change-Id: I0ca0488d6fa29f37578df2270d87ee72b4ecb944

Moved GL_KHR_blend_equation_advanced tests to common module

Change-Id: Iaa2d4cc117d756db7226641abb20cc4db7cccf0f

ImageLoadStoreMultipleUniformsTest: fix reversed bit-field overflow check

This class has a templated Mask struct used to store and access
bit-fields. In includes a specialization that is used if the offset is
bigger that the bitfield, that raises assert.

Without this patch, the check is reversed, so it fails on cases where
the offset is correct.

This patch changes the condition. An alternative would be change the
value of the specialization with asserts. I found this option more
natural.

This fixes the following test:
GL45-CTS.shader_image_load_store.multiple-uniforms

on the Mesa driver for Intel Haswell hw. It makes it goes from
crash to fail on Broadwell/Skylake.

Change-Id: I7341a542ad925e55ab8a6c95373567b4ea58434f

Smarter verification of configs to be tested for ES

Change-Id: I153cdd317228574bece32b87525e20acb52e16ab

Move GLES mustpass to external/openglcts/data/mustpass/gles

Change-Id: I4d1ec0d32504adb59e513e3bec5445283fb530e9

Update GL CTS README

Passing Criteria
* Remove duplicate sub-item in item 4
* Replace spelled numbers with digits

Running the tests on Android
* Enumerate available acitivites

Creating a submission package
* Typos

Change-Id: I76afa20a257ef91357ca0ad0d135964d9fbf94ec

Merge "Non-compressed copy image target size (64,64,8)"

Reformatting glcViewportArrayTests.cpp

Change-Id: Ic5d6ca4dc5a0016592bb5fb492880c6cbb1ea037

Documentation updates in preparation for the new release.

README restructuring, more detailed process
of creating a submission package.

Change-Id: Ie20291d3dafdbffda990a010d7fe85a04f6571c3

Updates to submission verification script

Change-Id: I4c2a3c897ceeecdcac586fad022a34ca18041b0d

Do not run against EGL_EXT_yuv_surface configs

Change-Id: I95750b2318c00d7d47d23076ee812dc33ad6445f

Fix uninitialized char arrays in ProgramInterfaceQueryTests.

Replicated from OpenGL/CTS Merge Request 432.

This patch initializes all of the char arrays in
the ProgramInterfaceQueryTests in the OpenGL and OpenGL ES versions of
conformance. Since they are char arrays
we should initialize them to the null terminator.

The motivation here is to prevent the verification script from
encountering unparsable tokens in the qpa result files from conformance.

This patch fixesOpengGL OSS-CTS issue 4.

Change-Id: Ia95742db876520d911b91039402dee46bbc6ebfe

Avoid double registering of dEQP modules on Android

Change-Id: I0a5fad29f1bb831f891cdae0459779f513c7d282

Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: I061ec26cc7b0e645a5c226659d051e7e12857d90

Merge gerrit/vulkan-cts-1.0.2 into gerrit/vulkan-cts-1.0-dev

Change-Id: I92ebeef47df62646fb9fd2743991c43cd8893bda

Alloc callbacks: don't report a quality warning for pooled objects

Modified tests:
- dEQP-VK.api.object_management.alloc_callback_fail_multiple.descriptor_set
- dEQP-VK.api.object_management.alloc_callback_fail_multiple.command_buffer_primary
- dEQP-VK.api.object_management.alloc_callback_fail_multiple.command_buffer_secondary

Change-Id: I0daea167c4b0c1e0a00039286b85af8c02e410f5
(cherry picked from commit ca634a5ac0b5278736e012f43a59bb25a5234e6f)

Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: Idb21386b1b14b3a7302028f556da63bec86427ae

Merge gerrit/vulkan-cts-1.0.2 into gerrit/vulkan-cts-1.0-dev

Change-Id: If83e62fdad4430038ba6318b99997a128f2c6c30

Fix image memory barriers for depth/stencil formats

Affected tests:
- dEQP-VK.api.copy_and_blit.image_to_image.all_formats.depth_stencil*

Fixes #562

Change-Id: I59a851d2748f7ae0c2b56fdcd879978de4755ddf

Sparse resources: create correct device interface

Make sure correct function pointers are used with the a VkDevice.
This should make no actual difference on most implementations.

Affected tests:
- dEQP-VK.sparse_resources.*

Change-Id: Id39cf318c4ae24b47135e1f27308985bd20a0009

Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: I76f3c7db1863fe3fe69a114ff835968bdc3ea6d1

Merge gerrit/vulkan-cts-1.0.2 into gerrit/vulkan-cts-1.0-dev

Change-Id: I3151d603fa37a323629d83d27b7b687d3fe3693e

Merge gerrit/vulkan-cts-1.0.1 into gerrit/vulkan-cts-1.0.2

Change-Id: I5ee52225ed76297caba6ae717e897d7290c20684

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Change-Id: I82eb4eb290e5d85b44ac2f1f2efb1eb88aef062a

Add -Wno-delete-non-virtual-dtor to default cxxflags

That warning generates some false negatives that are hard to work
around.

Change-Id: Ib451eca3af7bf2a6bfd24aa966f89496257b9f0d

Merge gerrit/vulkan-cts-1.0.1 into gerrit/vulkan-cts-1.0.2

Merge gerrit/vulkan-cts-1.0.0 into gerrit/vulkan-cts-1.0.1

Synchronization: use the correct device interface

This is a correctness change, likely has no impact
on most implementations.

Affected tests:
- dEQP-VK.synchronization.op.multi_queue.*

Change-Id: Ic2687f60c1ef2c468908c6c7240205b9e77e944c

Synchronization: use the correct device interface

This is a correctness change, likely has no impact
on most implementations.

Affected tests:
- dEQP-VK.synchronization.op.multi_queue.*

Change-Id: I3eee09c13626e23261a7d0496aacab259265c3b9

add the viewport_array.* tests

to the KHR-GLES32.core and KHR-GLESEXT test groups.
This adds the following 16 tests which can all be run with the
KHR-GLESEXT.viewport_array.* or KHR-GLES31.core.viewport_array.*
wildcards:

viewport_array.api_errors
viewport_array.queries
viewport_array.viewport_api
viewport_array.scissor_api
viewport_array.depth_range_api
viewport_array.scissor_test_state_api
viewport_array.draw_to_single_layer_with_multiple_viewports
viewport_array.dynamic_viewport_index
viewport_array.draw_mulitple_viewports_with_single_invocation
viewport_array.draw_multiple_layers
viewport_array.scissor
viewport_array.scissor_zero_dimension
viewport_array.scissor_clear
viewport_array.depth_range
viewport_array.depth_range_depth_test
viewport_array.provoking_vertex

Change-Id: I864cd57b70900db7ec6712919e7282960ccc0686

Reformatting with clang-format

Change-Id: Ibdec4cf1cbb77f9d48787bcbbc770d9cf789f2c8

Update viewport array tests to work in GLES

- Use shader specialization to enable the correct extensions and version.
- add supported functionality check to all tests
- Use runtime selection to determine Core vs ES functionality
  - use readpixels from a fbo attached texture in ES instead of getTexImage
  - disable provoking vertex calls for ES, and just assume LAST_CONVENTION
  - skip the subroutine tests in ES
  - Use templatized helper functions and an overloaded DepthFuncWrapper class
    to select the appropriate version of depth functionality as GL depth
    functions use GLdouble data types and ES uses GLfloat data types.

Change-Id: I9b37213acbdf41b93447741cfa097bb734ed4099

es31: complete check for rgba32f internalformat

Right now several tests check for the extension
EXT_color_texture_buffer to know if that internalformat is
supported. But at this moment that internalformat is also supported on
core profiles, or if ARB_color_texture_buffer is available.

Note that it is not possible to just add a check for
ARB_color_texture_buffer. On core profiles, most of the functionality
got included on the spec, but deprecating some of the enums. As an
outcome, some drivers decide to not expose the extension, even if the
internalformat and other functionality is exposed.

For more info, take a look to the following commits on mesa:
3264c3e99700389f0a3958db7c9c19673107d67a
bd850cb4f2c77e2eb6716c865c40b9976633fc23

This commit gets 72 tests going from skipped to pass on
Intel Skylake and Broadwell:
GL45-CTS.sample_variables.mask.rgba32f.*
GL45-CTS.shader_multisample_interpolation.render.*.rgba32f.*

Also move es31cSampleVariableTests and es31cShaderMultisampleInterpolationTests
to common.

Tests defined on those files are used on both GLES and GL test case lists,
so it makes sense to move them to the common module.

Change-Id: I0ed09e9897f9f2a1b4efd1a40c22da00ec70bfdc

Skip non-testable float EGLconfigs exposed via EGL_EXT_pixel_format_float

OpenGL CTS issue #13

Change-Id: I6e68b03df7e823dfcba56e19c55b4618d0e41589

Merge "Allow ANGLE_depth_texture for FBO completeness."

Merge "Extend primitive_bounding_box negative tests"

Allow ANGLE_depth_texture for FBO completeness.

This extension enables the same depth attachment formats as
GL_OES_depth_texture. The few restrictions on usage aren't
relevant for the completeness tests.

Bug: 33453991

Extend primitive_bounding_box negative tests

    Tests groups added to
    - dEQP-GLES31.functional.debug.negative_coverage.*.

    Extended tests:
    - shader_directive.primitive_bounding_box

Change-Id: I38b698e3bd594f71b15d3bce4dc9a809298eb95e

Add shader storage negative coverage tests

Tests groups added to:
- dEQP-GLES31.functional.debug.negative_coverage.*.

New tests:
- shader_storage.block_number_limits
- shader_storage.max_combined_block_number_limit

Extended tests:
- state.get_integeri_v
- state.get_integer64i_v
- shader.compile_compute_shader

Fixed tests:
- buffer.bind_buffer_base
- buffer.bind_buffer_range

Change-Id: Id71fb997e14ad026f0d6634298993f1638c67dde

Add viewport_array support to glcts::TestCaseBase

Detect viewport_array functionality for GL 4.1 core (or later)
and for OES_viewport_array extension in ES.
Streamline code for detection of other extensions.
Add support for specialization of viewport_array shader functionality.

Change-Id: Ie07186163cfcc94988a1d83e91897812b27e59ae

Fix existing viewport_array bugs

- was querying viewport instead of scissor box
- was requiring upper viewport bounds range of 32768

(these were pointed out by jputsman in previous review)

Affects tests: *.viewport_array.queries

Change-Id: I1914fd51cad1870ba91ef3380088aa20757ce29d

Remove unused utility functions from ViewportArray

Change-Id: I8525051e3eb1b5518def586d4d8a3b620ee3346b

Rebase ViewportArray classes and add extParams

Rebase the ViewportArray classes onto TestCaseBase (which supports shader specialization)
and add the extParams so that we know which type of functionality to test and can
specialize different versions of shaders.

Change-Id: I38f07580a2ee71957565a2a7f93f97fcf3073bdf

Move Viewport array tests from gl/ to common/

Move gl/gl4cViewportArrayTests.* to common/glcViewportArrayTests.*
Adjust relevant makefiles, namespaces, and include guards.
No other code changes were made.

Change-Id: I2d0245717cd2c507025ba256b48c7ad545e483f1

Enable GL_OES_viewport_array in GL generation scripts

The actual change was in scripts/opengl/src_util.py
everything else was auto-generated.

Change-Id: Id9a18188f0e3f0d4eb7c6495adc50b6ae439582a

Get rid of invalid buffer create flags combinations

Removed tests:
- dEQP-VK.api.buffer.createBuffer_*_2
- dEQP-VK.api.buffer.createBuffer_*_6

(cherry picked from commit bdd0a62ee565d0e7e07f05ddb95217d532e407cb)

Change-Id: I6cc6ed365de9d710f07fbfd2dde9db7edaecd007

Add explicit OS to some configs

Change-Id: I9176062420f3dd122456f9cf73c5e2aa12df5ec5

Get rid of invalid buffer create flags combinations

Removed tests:
- dEQP-VK.api.buffer.createBuffer_*_2
- dEQP-VK.api.buffer.createBuffer_*_6

(cherry picked from commit bdd0a62ee565d0e7e07f05ddb95217d532e407cb)

Change-Id: I6cc6ed365de9d710f07fbfd2dde9db7edaecd007

Move common submission verification functions to scripts/verify

VulkanCTS submission verification functionality will be
signifcantly reused by OpenGL CTS.
It's nice to have commonly used functions in one place.

Change-Id: I61cd83556c6f961b50d1c27953dd57614c4f417a

Get rid of invalid buffer create flags combinations

Removed tests:
- dEQP-VK.api.buffer.createBuffer_*_2
- dEQP-VK.api.buffer.createBuffer_*_6

Change-Id: I6cc6ed365de9d710f07fbfd2dde9db7edaecd007

Merge "direct-state-access/texture-tests: Use a type compatible with the format" into opengl-cts-dev

Merge "esextcTessellationShaderXFB: use individual out varyings instead of trying to get the array" into opengl-cts-dev

esextcTessellationShaderXFB: use individual out varyings instead of trying to get the array

Fixes:
GL45-CTS.tessellation_shader.single.xfb_captures_data_from_correct_stage

The test has a tessellation shader with a out varying like this:
          "out BLOCK_INOUT { vec4 value; } user_out[];\n"

The test uses transform feedback to get the content of user_out.

But the test is trying to use BLOCK_INOUT.value, instead of the
individual (4 in this case) elements of the array.

Additionally, this commit adds an OUT_PER_VERTEX_TCS_DECL, needed
as the out gl_PerVertex on tessellation control shaders is slightly
different compared with other stages.

From the GLSL 4.40 spec, page 123 (page 129 on the pdf):

   "In the tessellation control language, built-in variables are
    intrinsically declared as:

    <skip>
    out gl_PerVertex {
        vec4 gl_Position;
        float gl_PointSize;
        float gl_ClipDistance[];
    } gl_out[];"

Change-Id: I185aa456c4a802d71650b710d939bd749793942b

Merge branch 'wip/apinheiro/memory-barriers' into 'master'

    Extra memory barriers needed on different tests

    We found several tests to consistently fail or to be flaky (not always failing/passing) on Intel Skylake GT2 model. Those get solved adding memory barriers. Those memory barriers are convenient as could prevent the tests failing on other hw.

    The branch includes 4 new commits, to make easier the review, but I think that they can be squashed without problems.

    See merge request !108

Change-Id: I4ec91df6c1d8721eee18f9aa0223d3c5ba6465ef

direct-state-access/texture-tests: Use a type compatible with the format

One of the test sub-cases in ImageQueryErrorsTest family of tests is using
an incompatible format/type, when it is just checking for format and
internal-format compatibility. This is adding another failure point to the
test, making it dependent of the order of validation of the arguments in
the OpenGL driver.

Specifically, the sub-case that checks that "format is DEPTH_STENCIL and the
base internal format is not DEPTH_STENCIL" is passing GL_DEPTH_STENCIL with
type GL_UNSIGNED_INT, which would fail with INVALID_ENUM:

From OpenGL 4.5, section '18.2 Reading Pixels', page 506 of the PDF states:

    "An INVALID_ENUM error is generated if format is DEPTH_STENCIL and
     type is not UNSIGNED_INT_24_8 or FLOAT_32_UNSIGNED_INT_24_8_REV."

This is for ReadPixels, but the same pixel storage modes apply for
GetTextureImage:

From OpenGL 4.5, section '8.11 Texture Queries', page 234 of the PDF states:

    "Pixel storage modes that are applicable to ReadPixels are applied, as
     described in table 18.1 and section 18.2.9."

This patch changes the type to GL_UNSIGNED_INT_24_8, which is compatible with
the format, and will allow the subcase to unambiguosly fail if format and
internal-format are incompatible.

Change-Id: I979e007aef6e09b1707605bf4b94aebaa2d21b1c

Fix protocol auto-detection for ssh

oss-cts issue 21

Change-Id: Id0bf39fc6d9eb64ef9415e5101a9a8bd042848aa

Merge "Update KC CTS ref" into opengl-cts-dev

Merge gerrit/vulkan-cts-1.0-dev into gerrit/opengl-cts-dev

Change-Id: I4d2ef04a4c3bb44a40888cc859d9288f8b4949ef