New issue
Advanced search Search tips

Issue 865569 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

support_webgl2_compute_context not supported on android

Project Member Reported by timvolod...@chromium.org, Jul 19

Issue description

https://cs.chromium.org/chromium/src/third_party/blink/renderer/config.gni?l=41

The WebGL2ComputeRenderingContext interface is conditionally compiled now based on "support_webgl2_compute_context" value [1]. This is causing the Android WebView bots (L,M,N) on the main waterfall to fail [2]. With the reason being that "interface WebGL2ComputeRenderingContext" is missing in WebView.


[1] https://chromium-review.googlesource.com/c/chromium/src/+/1137987
[2] e.g. https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Android%20WebView%20N%20%28dbg%29/18732

 
It looks like this interface should not be exposed in WebView at all, because it's not supported on Android anyway.
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 19

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/962f2ba983294406a1e2064111fd8428775052be

commit 962f2ba983294406a1e2064111fd8428775052be
Author: Tim Volodine <timvolodine@chromium.org>
Date: Thu Jul 19 18:46:48 2018

[WebViewLayoutTests] Unexpose WebGL2ComputeRenderingContext

This interface is not supported on android and should not be
exposed. Currently it's causing some WebView bots to fail.

BUG= 865569 
NOTRY=true
TBR=paulmiller@chromium.org,tobiasjs@chromium.org

Change-Id: I1884dafea2d3db72fd5f0b91eb710dec9ff29bb6
Reviewed-on: https://chromium-review.googlesource.com/1144143
Reviewed-by: Tim Volodine <timvolodine@chromium.org>
Commit-Queue: Tim Volodine <timvolodine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576577}
[modify] https://crrev.com/962f2ba983294406a1e2064111fd8428775052be/android_webview/tools/system_webview_shell/test/data/webexposed/not-webview-exposed.txt

Cc: paulmiller@chromium.org tobiasjs@chromium.org
Components: Mobile>WebView
cc'ing sheriffs FYI

Cc: -paulmiller@chromium.org ctzsm@chromium.org
Owner: paulmiller@chromium.org
Status: Assigned (was: Untriaged)
Looks the test is still crashing after the change in #c2, passing this issue to current Android Sheriff.
Oh, the builders are still behind Tim's CL, feel free to close it if bots turns green then, thanks!
Status: Verified (was: Fixed)
Link mentioned in #6 is green marking verified

Sign in to add a comment