|
Revert of webgl: optimize webgl.texSubImage2D(video) path. (patchset #10 id:240001 of https://codereview.chromium.org/1315323006/ )
Reason for revert:
Reverting to address webgl_conformance_tests break in:
https://build.chromium.org/p/chromium.gpu/builders/Android%20Debug%20%28Nexus%209%29/builds/3534
Original issue's description:
> webgl: optimize webgl.texSubImage2D(video) path.
>
> Currently, texSubImage2D(video) requires glReadPixels to copy video texture to
> target texture. This CL makes infra structure to copy video texture to target
> texture directly. Blink will turn on the direct path.
>
> In addition, it will reduce WebGL video test flakiness significantly.
>
> TEST=video texture test on webgl_conformance
> ./content/test/gpu/run_gpu_test.py webgl_conformance --browser=release --story-filter=WebglConformance.conformance_textures_video_tex_image_and_sub_image_2d_with_video
> BUG= 349871, 504773
>
> Committed: https://crrev.com/01542ac6d0fbec6aa78e33e6c7ec49a582072ea9
> Cr-Commit-Position: refs/heads/master@{#356390}
TBR=piman@chromium.org,dalecurtis@chromium.org,kbr@chromium.org,oetuaho@nvidia.com,dpranke@chromium.org,philipj@opera.com,dongseong.hwang@intel.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 349871, 504773
Committed: https://crrev.com/bfa196022345aa9bd694d200f107b6d809178715
Cr-Commit-Position: refs/heads/master@{#356415}
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+101 lines, -261 lines) |
Patch |
 |
M |
content/renderer/media/android/webmediaplayer_android.h
|
View
|
|
1 chunk |
+5 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/renderer/media/android/webmediaplayer_android.cc
|
View
|
|
3 chunks |
+11 lines, -22 lines |
0 comments
|
Download
|
 |
M |
content/renderer/media/webmediaplayer_ms.h
|
View
|
|
1 chunk |
+5 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/renderer/media/webmediaplayer_ms.cc
|
View
|
|
2 chunks |
+7 lines, -13 lines |
0 comments
|
Download
|
 |
M |
media/blink/webmediaplayer_impl.h
|
View
|
|
1 chunk |
+5 lines, -1 line |
0 comments
|
Download
|
 |
M |
media/blink/webmediaplayer_impl.cc
|
View
|
|
2 chunks |
+7 lines, -13 lines |
0 comments
|
Download
|
 |
M |
media/renderers/skcanvas_video_renderer.h
|
View
|
|
1 chunk |
+5 lines, -24 lines |
0 comments
|
Download
|
 |
M |
media/renderers/skcanvas_video_renderer.cc
|
View
|
|
4 chunks |
+10 lines, -42 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/core/html/HTMLVideoElement.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/core/html/HTMLVideoElement.cpp
|
View
|
|
1 chunk |
+3 lines, -6 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.h
|
View
|
|
1 chunk |
+0 lines, -3 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
|
View
|
|
4 chunks |
+25 lines, -58 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/platform/graphics/ImageBuffer.h
|
View
|
|
2 chunks |
+1 line, -7 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/platform/graphics/ImageBuffer.cpp
|
View
|
|
2 chunks |
+4 lines, -27 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/Source/platform/graphics/gpu/Extensions3DUtil.cpp
|
View
|
|
1 chunk |
+2 lines, -17 lines |
0 comments
|
Download
|
 |
M |
third_party/WebKit/public/platform/WebMediaPlayer.h
|
View
|
|
1 chunk |
+10 lines, -25 lines |
0 comments
|
Download
|
Total messages: 4 (0 generated)
|