New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 649250 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
OOO until 2019-01-24
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 295792
issue 639145



Sign in to add a comment

conformance2/buffers/getBufferSubData.html failed on all platforms

Project Member Reported by yunchao...@intel.com, Sep 22 2016

Issue description

OS: All
Frequency: 100%

What steps will reproduce the problem?
(1)Run webgl 2 conformance test getBufferSubData.html

What is the expected output?
Pass

What do you see instead?
Fail

There should be some error in WebKit to support ArrayBufferView if the test case itself is correct. 

 
Cc: yang...@intel.com

Comment 2 by kbr@chromium.org, Sep 22 2016

Blocking: 639145
Labels: -Pri-3 Pri-2
Owner: kbr@chromium.org
Status: Assigned (was: Available)
It looks like this is as simple as deleting the compatibility code paths.

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 23 2016

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

commit f32c007a212163867271de976999a32a02161209
Author: kbr <kbr@chromium.org>
Date: Fri Sep 23 19:56:19 2016

Remove obsolete getBufferSubData compatibility entry points.

BUG= 639145 ,  649250 
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
NOTRY=true

Review-Url: https://codereview.chromium.org/2360353002
Cr-Commit-Position: refs/heads/master@{#420706}

[modify] https://crrev.com/f32c007a212163867271de976999a32a02161209/content/test/gpu/gpu_tests/webgl2_conformance_expectations.py
[modify] https://crrev.com/f32c007a212163867271de976999a32a02161209/third_party/WebKit/Source/modules/webgl/WebGL2RenderingContextBase.cpp
[modify] https://crrev.com/f32c007a212163867271de976999a32a02161209/third_party/WebKit/Source/modules/webgl/WebGL2RenderingContextBase.h
[modify] https://crrev.com/f32c007a212163867271de976999a32a02161209/third_party/WebKit/Source/modules/webgl/WebGL2RenderingContextBase.idl

Comment 4 by kbr@chromium.org, Sep 24 2016

Status: Fixed (was: Assigned)

Comment 5 by kbr@chromium.org, Sep 24 2016

 Issue 649905  has been merged into this issue.

Sign in to add a comment