|
|
Created:
4 years, 2 months ago by Geoff Lang Modified:
4 years, 2 months ago Reviewers:
Zhenyao Mo CC:
chromium-reviews, jam, darin-cc_chromium.org, piman+watch_chromium.org Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
DescriptionReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES.
BUG=607283
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Committed: https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa
Committed: https://crrev.com/860001120ed3a441f827f8a9850391903ddfaf0a
Cr-Original-Commit-Position: refs/heads/master@{#421614}
Cr-Commit-Position: refs/heads/master@{#421828}
Patch Set 1 #
Messages
Total messages: 27 (15 generated)
Description was changed from ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 ========== to ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel ==========
Description was changed from ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel ========== to ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ==========
The CQ bit was checked by geofflang@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Description was changed from ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ========== to ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ==========
geofflang@chromium.org changed reviewers: + zmo@chromium.org
PTAL
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
However, HALF_FLOAT does not exist on ES2, so it will fail on non-ANGLE ES2 drivers.
On 2016/09/28 19:34:31, Zhenyao Mo wrote: > However, HALF_FLOAT does not exist on ES2, so it will fail on non-ANGLE ES2 > drivers. That logic is preserved. The problem is that we were converting HALF_FLOAT_OES to HALF_FLOAT in ES3 which is not allowed unless it's returned as the implementation read type.
On 2016/09/28 19:41:56, Geoff Lang wrote: > On 2016/09/28 19:34:31, Zhenyao Mo wrote: > > However, HALF_FLOAT does not exist on ES2, so it will fail on non-ANGLE ES2 > > drivers. > > That logic is preserved. The problem is that we were converting HALF_FLOAT_OES > to HALF_FLOAT in ES3 which is not allowed unless it's returned as the > implementation read type. Apparently my brain is still muddy. LGTM
On 2016/09/28 19:48:47, Zhenyao Mo wrote: > On 2016/09/28 19:41:56, Geoff Lang wrote: > > On 2016/09/28 19:34:31, Zhenyao Mo wrote: > > > However, HALF_FLOAT does not exist on ES2, so it will fail on non-ANGLE ES2 > > > drivers. > > > > That logic is preserved. The problem is that we were converting > HALF_FLOAT_OES > > to HALF_FLOAT in ES3 which is not allowed unless it's returned as the > > implementation read type. > > Apparently my brain is still muddy. > > LGTM NP, glad you're starting to feel better!
The CQ bit was checked by geofflang@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Message was sent while issue was closed.
Description was changed from ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ========== to ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel ========== to ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa Cr-Commit-Position: refs/heads/master@{#421614} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa Cr-Commit-Position: refs/heads/master@{#421614}
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2374083003/ by geofflang@chromium.org. The reason for reverting is: Flaky results in WebglConformance_conformance_extensions_oes_texture_half_float on win_optional_gpu_tests_rel..
Message was sent while issue was closed.
Description was changed from ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa Cr-Commit-Position: refs/heads/master@{#421614} ========== to ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa Cr-Commit-Position: refs/heads/master@{#421614} ==========
The CQ bit was checked by geofflang@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Message was sent while issue was closed.
Description was changed from ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa Cr-Commit-Position: refs/heads/master@{#421614} ========== to ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa Cr-Commit-Position: refs/heads/master@{#421614} ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa Cr-Commit-Position: refs/heads/master@{#421614} ========== to ========== ReadPixels with GL_HALF_FLOAT_OES when targetting all versions of ES. BUG=607283 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/2d1b397c9a057651edbd3ef70266cb0727671baa Committed: https://crrev.com/860001120ed3a441f827f8a9850391903ddfaf0a Cr-Original-Commit-Position: refs/heads/master@{#421614} Cr-Commit-Position: refs/heads/master@{#421828} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/860001120ed3a441f827f8a9850391903ddfaf0a Cr-Commit-Position: refs/heads/master@{#421828} |