Merge "Relax line width verification in primitive bbox tests" into nougat-cts-dev am: 312a550657 am: f3d81a3cd9
am: 3b0479c233

Change-Id: I71d4fc5fc3e81382b5d2e0a09bb49762b1c0478f

Relax line width verification in primitive bbox tests am: ba45591407 am: b4fbc02e3c
am: b9b63c17d2

Change-Id: I2aa14779ebae6f023eb9cef9e162ab45c506a0fb

Merge "Fix eglBindAPI negative test" into nougat-cts-dev am: 6068dfbf3a am: 365f81e68c
am: 15cde0b466

Change-Id: I6aa3c1cc622ec777f5841b0fca2bea2ef2db724a

Fix eglBindAPI negative test am: 09784f8510 am: 3342150f5a
am: 1af587d644

Change-Id: I17dbd8e798c13e7057cdeab73da429d75782ae39

Merge "Drop incorrect framebuffer texture subtests." into nougat-cts-dev am: 3baf7e3be2 am: a580ae91f8
am: 816d6bdb88

Change-Id: Ib3832fe6f652adf148535c38deb52405fe2c996e

Drop incorrect framebuffer texture subtests. am: 625d685874 am: 25b6d76b01
am: 6f6f3d78fc

Change-Id: Id18abd5bf542b38b13175fca635b2165f4d6a1e5

Merge "Optimize swapchain OOM tests" into nougat-cts-dev am: c811d42d99 am: 6d4c0dca61
am: d64bb66820

Change-Id: I0860b4a288e7f56d462411892eae21e19d1a462b

Optimize swapchain OOM tests am: 1614827a71 am: 2cfe132119
am: 00ab17bd23

Change-Id: Id303077bc77b4fa039a9ce6a74f35a15d5e26210

Merge "Fix internal format/type for ES3 3D + depth/stencil negative API tests." into nougat-cts-dev am: 0991459ed1 am: e19d6a6859
am: e0e2260a4a

Change-Id: I225d74dcb7a2806bdbfed8aebc7109ce2887ec36

Fix internal format/type for ES3 3D + depth/stencil negative API tests. am: 2ed3a5a66f am: 6a6f549150
am: f9b870fb5b

Change-Id: Ifba5e2d1705ea8f8f9dc79afe58a6e9a727d3a85

Merge "Add support for EGL_EXT_pixel_format_float" into nougat-cts-dev am: 13a05434cc am: 5f30178321
am: 764cd6e60b

Change-Id: I0f8b9eb0c1fa7ce4fdc26e5c8fbbefa3aac04bb6

Add support for EGL_EXT_pixel_format_float am: 521bf8a7c3 am: 4b7baeace7
am: 86683f95c7

Change-Id: I54718e0c412ba01c43e357f99f247f55ed139ed3

Merge "Remove tests affected by NV/OES_viewport_array extension" into nougat-cts-dev am: e98444b9bf
am: ce3144c3e5

Change-Id: If580320c0105fcc4087ef873f89de3350019c719

Remove tests affected by NV/OES_viewport_array extension am: 267c765690
am: 0004ce1767

Change-Id: I4d38d14737546246b00830294efe9366a36e51b8

Merge "Generate flat triangles in polygon offset tests." into nougat-cts-dev am: 61d6d95a66
am: 73b7f271a9

Change-Id: I81f005b1da8dabe780d5f16e2b0b24f587286546

Generate flat triangles in polygon offset tests. am: 975a59ebca
am: 02c67b12c1

Change-Id: I86b9659a38fb029def1ab9c7de15c73f1e60473a

Merge "Relax line width verification in primitive bbox tests" into nougat-cts-dev am: 312a550657
am: f3d81a3cd9

Change-Id: I9e49b6481fa9f157901c9a1d36c15505c4cf9098

Merge "Relax line width verification in primitive bbox tests" into nougat-cts-dev am: 312a550657
am: 36d3dd2ab1

Change-Id: I1328497673c38aefc8617ee7c555ae2b242edb53

Relax line width verification in primitive bbox tests am: ba45591407
am: a11fd8b487

Change-Id: I559fa411bd3e9ab117e4fdacd466c03ead6c73da

Relax line width verification in primitive bbox tests am: ba45591407
am: b4fbc02e3c

Change-Id: I5177fb99eeae802415968a9745195ddc3a6bd5c9

Merge "Fix eglBindAPI negative test" into nougat-cts-dev am: 6068dfbf3a
am: 839be6fae4

Change-Id: I15e8699e526124806d63a299d9c5ce73fffd696f

Merge "Fix eglBindAPI negative test" into nougat-cts-dev am: 6068dfbf3a
am: 365f81e68c

Change-Id: I20c34cbb3726ec1f018d0f79a654abc9b7682780

Fix eglBindAPI negative test am: 09784f8510
am: b7b2f00530

Change-Id: I87f03d37402abea5854ea8db14f847943ac306ba

Fix eglBindAPI negative test am: 09784f8510
am: 3342150f5a

Change-Id: I8d0ecfb53bd90769f15d656cd066ce5c17a67067

Merge "Drop incorrect framebuffer texture subtests." into nougat-cts-dev am: 3baf7e3be2
am: b3f5df9d8f

Change-Id: Ie130b6d34ae0dcd24b2fa867d3631cc9428ef4ba

Merge "Drop incorrect framebuffer texture subtests." into nougat-cts-dev am: 3baf7e3be2
am: a580ae91f8

Change-Id: I297decc0ca64dd782697ae74cfa7fcf15c178bb9

Drop incorrect framebuffer texture subtests. am: 625d685874
am: 25b6d76b01

Change-Id: I198ba68c6ba2919220194ff8c3c5d0bdddd78f56

Drop incorrect framebuffer texture subtests. am: 625d685874
am: 9df734d698

Change-Id: Ideece41c3411c8a01ea403395edc851dc734c4db

Merge "Optimize swapchain OOM tests" into nougat-cts-dev am: c811d42d99
am: 6d4c0dca61

Change-Id: I92282e9341ff46ff5ee48bfa2c5f54385e3a41ed

Merge "Optimize swapchain OOM tests" into nougat-cts-dev am: c811d42d99
am: 27b237e149

Change-Id: Id9d29d51ace7767e0b2a57a32d3f38379ed32900

Optimize swapchain OOM tests am: 1614827a71
am: 2cfe132119

Change-Id: I6e3f2cc8f18add32f8710196b9adbedd71d51ee2

Optimize swapchain OOM tests am: 1614827a71
am: c4cd6ba5e7

Change-Id: I61acabdc7d5696b4565e346143a189267ae0aeb8

Merge "Fix internal format/type for ES3 3D + depth/stencil negative API tests." into nougat-cts-dev am: 0991459ed1
am: e19d6a6859

Change-Id: Ib4ee4be626ba638dbf12707710cb86c9f276ec99

Merge "Fix internal format/type for ES3 3D + depth/stencil negative API tests." into nougat-cts-dev am: 0991459ed1
am: 0a2fc88750

Change-Id: I893215e862509b29f8fd3477fd513cb0ae87970d

Fix internal format/type for ES3 3D + depth/stencil negative API tests. am: 2ed3a5a66f
am: 6a6f549150

Change-Id: I549bf24bb7add8dd7e0d3f1aefe6f83c3319b032

Fix internal format/type for ES3 3D + depth/stencil negative API tests. am: 2ed3a5a66f
am: ad563bc887

Change-Id: I290e18b7ab69769c08fab421f022eeccc25a5e26

Merge "Add support for EGL_EXT_pixel_format_float" into nougat-cts-dev am: 13a05434cc
am: 5f30178321

Change-Id: I1c8e7901ff7203a242baa374f4f0d8c3c111dadd

Merge "Add support for EGL_EXT_pixel_format_float" into nougat-cts-dev am: 13a05434cc
am: 4fc6c2e455

Change-Id: I5e6511ecd304ecf0374cfb2078740c74629145cc

Add support for EGL_EXT_pixel_format_float am: 521bf8a7c3
am: 4b7baeace7

Change-Id: Ia42668945a64ea86b6357fb34cfb9b47a32719b5

Add support for EGL_EXT_pixel_format_float am: 521bf8a7c3
am: 28a6ef5df2

Change-Id: I737dc392ec39fa022166905d7af7f029da7e0c36

Merge "Relax line width verification in primitive bbox tests" into nougat-cts-dev
am: 312a550657

Change-Id: I710c20679327116888d493649b6a24183c2d705b

Relax line width verification in primitive bbox tests
am: ba45591407

Change-Id: Iaf6257f99741756a8a8e4b3f9edd463125486c34

Merge "Fix eglBindAPI negative test" into nougat-cts-dev
am: 6068dfbf3a

Change-Id: I50cc593a39875b4b91616e4dcd4a10ed21b3e25f

Fix eglBindAPI negative test
am: 09784f8510

Change-Id: I4e1a96d510ec7d9d26be1e4ff0eb0a80840137b2

Merge "Drop incorrect framebuffer texture subtests." into nougat-cts-dev
am: 3baf7e3be2

Change-Id: Ia80478eb3127eb595158f1a7f0c60e3fa11dc8d0

Drop incorrect framebuffer texture subtests.
am: 625d685874

Change-Id: I16d0bba267d3c4aabee476e442c01151efafd1a4

Merge "Optimize swapchain OOM tests" into nougat-cts-dev
am: c811d42d99

Change-Id: I80f4d73072eb4e310428b008e89bd41d62fffee4

Optimize swapchain OOM tests
am: 1614827a71

Change-Id: I7fce434527e387f9c14f332eddf99e2a885624d8

Merge "Remove tests affected by NV/OES_viewport_array extension" into nougat-cts-dev
am: e98444b9bf

Change-Id: I03190aa34e911cb4a8b174d1251a354ff6e34718

Remove tests affected by NV/OES_viewport_array extension
am: 267c765690

Change-Id: I74f0255f38b618d9c72e1e48fabbda9636daf14d

Merge "Fix internal format/type for ES3 3D + depth/stencil negative API tests." into nougat-cts-dev
am: 0991459ed1

Change-Id: Id5201c4bf95d6127d9ee71da302c64b5d6a2c1b1

Fix internal format/type for ES3 3D + depth/stencil negative API tests.
am: 2ed3a5a66f

Change-Id: I247338350d1d8cbb7dfcde284bbd4c29d90561b3

Merge "Generate flat triangles in polygon offset tests." into nougat-cts-dev
am: 61d6d95a66

Change-Id: Ia7141732269df847c13bbb8b36e9f911c76d8753

Generate flat triangles in polygon offset tests.
am: 975a59ebca

Change-Id: If91ab1b34f3215705636abaa4dc52ebad5d3d3a7

Merge "Add support for EGL_EXT_pixel_format_float" into nougat-cts-dev
am: 13a05434cc

Change-Id: Id211d01ef9006c6fcd2866ddc087f71fcd5500eb

Add support for EGL_EXT_pixel_format_float
am: 521bf8a7c3

Change-Id: I5328e30c4dcf07283541aaa5f9842597e56c4036

Merge "Relax line width verification in primitive bbox tests" into nougat-cts-dev
am: 312a550657

Change-Id: Ia8465e3ae1fb9d80125305134c157857e5e339b1

Relax line width verification in primitive bbox tests
am: ba45591407

Change-Id: Ia0f71e5511d1bf4bf449ef71ecf5bb822311cf08

Merge "Fix eglBindAPI negative test" into nougat-cts-dev
am: 6068dfbf3a

Change-Id: I4ca862cbe3c17c0887437a98570f16e1bfc19d34

Fix eglBindAPI negative test
am: 09784f8510

Change-Id: I9f1246736de39cfc6bc6593bad465fa881f55e29

Merge "Drop incorrect framebuffer texture subtests." into nougat-cts-dev
am: 3baf7e3be2

Change-Id: I2f95c748059d7aea5f1899ff6ee6c0fe96ea7245

Drop incorrect framebuffer texture subtests.
am: 625d685874

Change-Id: I7b1cab296c2a61ed3e05ae52dd21fd62bc6d3c8a

Merge "Optimize swapchain OOM tests" into nougat-cts-dev
am: c811d42d99

Change-Id: I06e852cc34ee156dbd66ac36fbb052a2db8ec36f

Optimize swapchain OOM tests
am: 1614827a71

Change-Id: I5ac33602999153d04624eb32504437ffff9b3b51

Merge "Fix internal format/type for ES3 3D + depth/stencil negative API tests." into nougat-cts-dev
am: 0991459ed1

Change-Id: If9f75801b7ea26bec470e4dd11b318431ed56352

Fix internal format/type for ES3 3D + depth/stencil negative API tests.
am: 2ed3a5a66f

Change-Id: I9439b037614e52368b771e1b7a84bd4930bfbf6a

Merge "Add support for EGL_EXT_pixel_format_float" into nougat-cts-dev
am: 13a05434cc

Change-Id: Ic956c9c223aef327c2614b3f52ecf5c1166bec22

Add support for EGL_EXT_pixel_format_float
am: 521bf8a7c3

Change-Id: I64bda78561b2eb598e78bb2ba13b485602ea96e7

Merge "Remove tests affected by NV/OES_viewport_array extension" into nougat-cts-dev

Merge "Generate flat triangles in polygon offset tests." into nougat-cts-dev

Merge "Relax line width verification in primitive bbox tests" into nougat-cts-dev

Merge "Fix eglBindAPI negative test" into nougat-cts-dev

Merge "Drop incorrect framebuffer texture subtests." into nougat-cts-dev

Merge "Optimize swapchain OOM tests" into nougat-cts-dev

Merge "Fix internal format/type for ES3 3D + depth/stencil negative API tests." into nougat-cts-dev

Merge "Add support for EGL_EXT_pixel_format_float" into nougat-cts-dev

Remove tests affected by NV/OES_viewport_array extension

Tests were not properly taking into account viewport clamping behavior
introduced by GL_NV/OES_viewport_array extensions.

Bug: 34349411
Change-Id: I3eae36359e6408191e0fbefb20cededec2a5bd39
Merged-In: I3d3c2e32d7dd96d1db960aa53dc18cc690187ec0

Generate flat triangles in polygon offset tests.

The tests setup triangles with a depth slope and then call
glPolygonOffset(factor, units) with variable units and
always 0 slope scaled factor.

According to the GLES 3.2 spec 13.7.2 Depth Offset:
"void PolygonOffset( float factor, float units );
factor scales the maximum depth slope of the polygon, and units scales an
implementation-dependent constant that relates to the usable resolution of the
depth buffer."

If the polygon has a depth slope,
the depth slope factor has to be not equal to zero.
The rationale of the slope scaled factor is to ensure that the offset
(equation 13.11) is greater than max(dz/dx, dz/dy) for all resolutions,
which may not be the case for the constant offset since it doesn't take
the screen resolution into account.

Changed tests:

dEQP-GLES2.functional.polygon_offset.*_displacement_with_units
dEQP-GLES3.functional.polygon_offset.*_displacement_with_units

Bug: 34386628

Change-Id: Ie8009ba1766eb50c3c5a750122aa8bb231144210

Drop incorrect framebuffer texture subtests.

This patch removes two broken subcases of:
dEQP-GLES31.functional.debug.negative_coverage.log.buffer.framebuffer_texture_layer

These were fixed in dEQP master by commit f28187809176f5eeb1ee25ccfa70d3,
but we cannot backport that to release branches because it changes the
existing expectations, which would break previously passing
implementations.  See buganizer 30287182 and 28090252.

The first subtest incorrectly checked layer (array slice) when it should
have checked miplevel.  The subtest description says:

"GL_INVALID_VALUE is generated if texture is a 2D multisample array
 texture and layer not 0."

which is wrong - array textures can certainly have multiple slices.
The ES 3.1 specification says that:

"An INVALID_VALUE error is generated if texture is not zero and level is
 not a supported texture level for texture, as described above."

and miplevel 0 is the only valid miplevel for a multisample texture.

The second subtest is no longer valid according to the resolution of
https://cvs.khronos.org/bugzilla/show_bug.cgi?id=15968.

Bug: 34281627
Merged-In: I500074c1777d88291c0b395498bb0f7d5613d27d

Relax line width verification in primitive bbox tests

Originally contributed by ARM to Khronos OpenGL ES CTS. Cherry-picked
with permission.

Google bug: 22713865
Khronos OpenGL issue: #19

Change-Id: I95359847611144e179819aaab6ac5dfe4bf4e4da

Skip commit '302bd7c6b3d468e33b19b693a25d67f2b54b961c' on nougat-mr1-cts-dev.
am: c5bbebc46a  -s ours

Change-Id: I9bfe818beb07950ca63f6f1d2e4efbfaaf7eee8d

Merge commit '09e8b66d03077e9a9b310145112cfd0e747e06f3' from nougat-cts-dev to nougat-cts-mr1-dev.
am: 2a9d42a651

Change-Id: I1a7c1e97354c43a561d91413191fb3b65f5d4f5a

Skip commit '302bd7c6b3d468e33b19b693a25d67f2b54b961c' on
nougat-mr1-cts-dev.

BUG: 33090058
Change-Id: Ife934b6694e07b2231070cc5d05865fed78e67c4

Merge commit '09e8b66d03077e9a9b310145112cfd0e747e06f3' from
nougat-cts-dev to nougat-cts-mr1-dev.

BUG: 33090058
Change-Id: I7a8bacd16b5d98255987c5632e73dd823d8643be

Optimize swapchain OOM tests

Swapchain OOM tests were hitting timeouts on some platforms. This commit
optimizes the tests in two ways:

 * VkInstance, VkDevice, and VkSurface are now re-used across all
sub-cases.

 * Loop is re-written to avoid re-creating swapchains for parameters
0..N-1 when testing parameter combination N.

Affects: dEQP-VK.wsi.*.swapchain.simulate_oom.*

Bug: 33555898
(cherry picked from commit 2beac9057d9113ac306632d819ded852691a6842)

Change-Id: I19e9c7c8012a82cd12322a733afab961f4c7a1ae

Fix internal format/type for ES3 3D + depth/stencil negative API tests.

This is a port of commit ae7f8e0a07730e693b24d3dc7a23d2372319145e from
the ES 3.1 tests to the ES 3.0 tests.

According to the ES 3.2 specification:

   "Textures with a base internal format of DEPTH_COMPONENT, DEPTH_STENCIL
    or STENCIL_INDEX are supported by texture image specification commands
    only if target is TEXTURE_2D, TEXTURE_2D_MULTISAMPLE, TEXTURE_2D_ARRAY,
    TEXTURE_2D_MULTISAMPLE_ARRAY, TEXTURE_CUBE_MAP or TEXTURE_CUBE_MAP_ARRAY.
    Using these formats in conjunction with any other target will result in
    an INVALID_OPERATION error."

This subtest tried to check the above error condition, but it specified
GL_DEPTH_STENCIL / GL_DEPTH_COMPONENT as format, rather than internalFormat.
Since the above text calls out "base internal format", we should specify
it as internalFormat.

We also change GL_DEPTH_STENCIL to use GL_UNSIGNED_INT_24_8 rather than
GL_UNSIGNED_BYTE, as that combination was illegal for a different reason
than the one the test intended to check.

Affects dEQP-GLES3.functional.negative_api.texture.teximage3d.

Bug: 34103293

Change-Id: Ie01e2d130bb1cadc821153487e3e41593e3ca15e

Fix eglBindAPI negative test

It is ok for an implementation to not fail eglBindAPI if the current
display does not support the specified client API. However, in that
case, getConfig must not return any configuration for that specific
API.

Bug: 33278378
Affects: dEQP-EGL.functional.negative_api.bind_api
Test: Deploy to elm Chromebook, run with Cherry, test passes.
Change-Id: Idb86971de7a30e1aa78bb0951f971ed2829e2794

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

Change-Id: I7fcc1d4237db9ea395b85bcdedc7acf340751c8d

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

Change-Id: Id9d584f16b6b43190528aa93945ae479520ada47

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

Change-Id: Ie61b8831c62b3559950b721707830f3fed3cd5c4

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

Change-Id: Iac6064074cf31d05563b47e90167fde94a641fde

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

Change-Id: I452742de754cbfad9bbd8633f7ecbc911c6dc172

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

Change-Id: Ibf74d17b9c30903c3c9fca20443330ba4f72ae7e

Add support for EGL_EXT_pixel_format_float

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

Bug: 32969314
Change-Id: Iacfdf6b1c32c86ea0e6f80ca7ebaaeef583d1135
Merged-In: 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

Cherry-pick the following missing commits from nyc-mr1-dev prior to
b/33090058, which was lost as a result of merge
01c922fc267ea41cc1c6cb83cde7ba6da200dc38:

ea5589 Kenneth Graunke: Fix sample_mask_in.bit_count_per_two_samples
       tests for 2x MSAA.
d2f3b4 Kenneth Graunke: Fix internal format/type for 3D + depth/stencil
       negative API tests.
bf0076 Aaron Holden: Add runtime to CtsDeqpTestCases

BUG: 33090058
Change-Id: I88a47ddebd9dff35a3fa65967cca6fe21b739a6e

Add runtime to CtsDeqpTestCases

Test: cts-tradefed run cts -m CtsDeqpTestCases
Bug:32843094
Change-Id: I6ea71f0d6658b1f06354374ece54743609472345
(cherry picked from commit bf0076a42408b330a5117c61a0b0d8d6d2b18660)

Fix internal format/type for 3D + depth/stencil negative API tests.

According to the ES 3.2 specification:

   "Textures with a base internal format of DEPTH_COMPONENT, DEPTH_STENCIL
    or STENCIL_INDEX are supported by texture image specification commands
    only if target is TEXTURE_2D, TEXTURE_2D_MULTISAMPLE, TEXTURE_2D_ARRAY,
    TEXTURE_2D_MULTISAMPLE_ARRAY, TEXTURE_CUBE_MAP or TEXTURE_CUBE_MAP_ARRAY.
    Using these formats in conjunction with any other target will result in
    an INVALID_OPERATION error."

This subtest tried to check the above error condition, but it specified
GL_DEPTH_STENCIL / GL_DEPTH_COMPONENT as format, rather than internalFormat.
Since the above text calls out "base internal format", we should specify
it as internalFormat.

We also change GL_DEPTH_STENCIL to use GL_UNSIGNED_INT_24_8 rather than
GL_UNSIGNED_BYTE, as that combination was illegal for a different reason
than the one the test intended to check.

Change-Id: I44bf607533b59f93fa42415bd6cad8783352582d
(cherry picked from commit ae7f8e0a07730e693b24d3dc7a23d2372319145e)
(cherry picked from commit d2f3b468db45e8deb130de84bf194eebe37d5fff)

Fix sample_mask_in.bit_count_per_two_samples tests for 2x MSAA.

The dEQP-GLES31.functional.shaders.sample_variables.sample_mask_in.
bit_count_per_two_samples.multisample_{texture,renderbuffer}_{1,2}
tests report:

    Verifying gl_SampleMaskIn.
    Fragment shader may be invoked [ceil(numSamples/2), numSamples] times.
    => gl_SampleMaskIn should have the number of bits set in range
       [1, numSamples - ceil(numSamples/2) + 1]:

which suggests that maxBitCount should be ceil(2/2) + 1 = 1 + 1 = 2.

However, it then says:

    Setting minBitCount = 1, maxBitCount = 1.

Having two bits should be acceptable for 2x MSAA.

This patch drops the special case for 1x/2x MSAA, making them work
like all the other MSAA levels.

Change-Id: Ie615c82150588dab53226dc46670182a29d85ef5
(cherry picked from commit 724519b8fda9c2a92d646c3dfd1478e0daa282e9)
(cherry picked from commit ea5589c748c0f399e478a570f9cc3c063fe55bf1)