Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(156)

Issue 2505223002: Roll WebGL 6b0934a..f1cbc14 (Closed)

Created:
4 years, 1 month ago by Zhenyao Mo
Modified:
4 years, 1 month ago
CC:
chromium-reviews, Geoff Lang
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Roll WebGL 6b0934a..f1cbc14 https://chromium.googlesource.com/external/khronosgroup/webgl.git/+log/6b0934a..f1cbc14 BUG= TEST=bots NOTRY=true TBR=kbr@chromium.org CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Committed: https://crrev.com/62fd8f8b1160ce61fa3b031fffa8626609ccdd29 Cr-Commit-Position: refs/heads/master@{#432722}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 21 (11 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2505223002/1
4 years, 1 month ago (2016-11-16 23:34:25 UTC) #4
commit-bot: I haz the power
Try jobs failed on following builders: android_cronet on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_cronet/builds/37386)
4 years, 1 month ago (2016-11-17 01:13:49 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2505223002/1
4 years, 1 month ago (2016-11-17 01:19:17 UTC) #8
Zhenyao Mo
On 2016/11/17 01:19:17, commit-bot: I haz the power wrote: > CQ is trying da patch. ...
4 years, 1 month ago (2016-11-17 02:38:29 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2505223002/1
4 years, 1 month ago (2016-11-17 02:39:39 UTC) #13
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-11-17 03:24:40 UTC) #15
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/62fd8f8b1160ce61fa3b031fffa8626609ccdd29 Cr-Commit-Position: refs/heads/master@{#432722}
4 years, 1 month ago (2016-11-17 03:28:06 UTC) #17
Geoff Lang
I think that since this roll, we're seeing failures in WebglConformance_conformance2_textures_misc_tex_mipmap_levels and WebglConformance_deqp_functional_gles3_shadertexturefunction_texturesize on Linux ...
4 years, 1 month ago (2016-11-17 14:45:32 UTC) #18
Zhenyao Mo
On 2016/11/17 14:45:32, Geoff Lang wrote: > I think that since this roll, we're seeing ...
4 years, 1 month ago (2016-11-17 14:53:33 UTC) #19
Ken Russell (switch to Gerrit)
4 years, 1 month ago (2016-11-17 17:32:06 UTC) #21
Message was sent while issue was closed.
On 2016/11/17 14:53:33, Zhenyao Mo wrote:
> On 2016/11/17 14:45:32, Geoff Lang wrote:
> > I think that since this roll, we're seeing failures in
> > WebglConformance_conformance2_textures_misc_tex_mipmap_levels and
> > WebglConformance_deqp_functional_gles3_shadertexturefunction_texturesize on
> > Linux Release (New Intel).
> > 
> > They first fail here:
> >
>
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28N...
> > and continue here
> >
>
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28N...
> 
> It can't be as there is only a minor fix in this roll.  Flakiness has to come
> from somewhere else.

Geoff: please file a bug about these failures. Since they are happening only on
Linux Intel, they look like possible driver bugs to me.

https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28N...
contains
https://chromium.googlesource.com/chromium/src/+/5e6600b56cad269e4a8640c33084...
, which is a longstanding issue only just discovered, and which seems to be
working well on all the other platforms.

Powered by Google App Engine
This is Rietveld 408576698