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

Unified Diff: third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp

Issue 2121043002: 16 bpp video stream capture, render and WebGL usage - Realsense R200 & SR300 support. Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
diff --git a/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp b/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
index 0dca9165b5c0d6bc54ff70dfccbba8dadcb2cdd6..6b0a69ad959142b547551752e2d26174d16253d1 100644
--- a/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
+++ b/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
@@ -500,6 +500,28 @@ class ScopedFramebufferRestorer {
Member<WebGLRenderingContextBase> m_context;
};
+class ScopedUnpackParametersResetRestore {
+ STACK_ALLOCATED();
+
+ public:
+ explicit ScopedUnpackParametersResetRestore(
+ WebGLRenderingContextBase* context,
+ bool enabled = true)
+ : m_context(context), m_enabled(enabled) {
+ if (enabled)
+ m_context->resetUnpackParameters();
+ }
+
+ ~ScopedUnpackParametersResetRestore() {
+ if (m_enabled)
+ m_context->restoreUnpackParameters();
+ }
+
+ private:
+ Member<WebGLRenderingContextBase> m_context;
+ bool m_enabled;
+};
+
static void formatWebGLStatusString(const StringView& glInfo,
const StringView& infoString,
StringBuilder& builder) {
@@ -4404,7 +4426,7 @@ void WebGLRenderingContextBase::texImageImpl(
}
}
- resetUnpackParameters();
+ ScopedUnpackParametersResetRestore temporaryResetUnpack(this);
if (functionID == TexImage2D) {
texImage2DBase(target, level, internalformat, imageExtractor.imageWidth(),
imageExtractor.imageHeight(), 0, format, type,
@@ -4421,7 +4443,6 @@ void WebGLRenderingContextBase::texImageImpl(
imageExtractor.imageHeight(), 1, format, type,
needConversion ? data.data() : imagePixelData);
}
- restoreUnpackParameters();
}
bool WebGLRenderingContextBase::validateTexFunc(
@@ -4607,16 +4628,14 @@ void WebGLRenderingContextBase::texImageHelperDOMArrayBufferView(
return;
}
- if (changeUnpackAlignment)
- resetUnpackParameters();
+ ScopedUnpackParametersResetRestore temporaryResetUnpack(
+ this, changeUnpackAlignment);
if (functionID == TexImage2D)
texImage2DBase(target, level, internalformat, width, height, border, format,
type, data);
else if (functionID == TexSubImage2D)
contextGL()->TexSubImage2D(target, level, xoffset, yoffset, width, height,
format, type, data);
- if (changeUnpackAlignment)
- restoreUnpackParameters();
}
void WebGLRenderingContextBase::texImage2D(GLenum target,
@@ -4690,7 +4709,7 @@ void WebGLRenderingContextBase::texImageHelperImageData(
return;
}
}
- resetUnpackParameters();
+ ScopedUnpackParametersResetRestore temporaryResetUnpack(this);
if (functionID == TexImage2D) {
texImage2DBase(target, level, internalformat, pixels->width(),
pixels->height(), border, format, type,
@@ -4706,7 +4725,6 @@ void WebGLRenderingContextBase::texImageHelperImageData(
pixels->height(), depth, format, type,
needConversion ? data.data() : pixels->data()->data());
}
- restoreUnpackParameters();
}
void WebGLRenderingContextBase::texImage2D(GLenum target,
@@ -4799,6 +4817,8 @@ void WebGLRenderingContextBase::texImageCanvasByGPU(HTMLCanvasElement* canvas,
if (!canvas->is3D()) {
ImageBuffer* buffer = canvas->buffer();
if (buffer &&
+ extensionsUtil()->canUseCopyTextureCHROMIUM(
+ targetTexture, targetInternalformat, targetType, targetLevel) &&
!buffer->copyToPlatformTexture(
contextGL(), targetTexture, targetInternalformat, targetType,
targetLevel, m_unpackPremultiplyAlpha, m_unpackFlipY)) {
@@ -4839,7 +4859,7 @@ void WebGLRenderingContextBase::texImageByGPU(TexImageByGPUType functionType,
GLint targetLevel = level;
bool possibleDirectCopy = false;
if (functionType == TexImage2DByGPU) {
- possibleDirectCopy = Extensions3DUtil::canUseCopyTextureCHROMIUM(
+ possibleDirectCopy = extensionsUtil()->canUseCopyTextureCHROMIUM(
target, internalformat, type, level);
}
@@ -5017,45 +5037,58 @@ void WebGLRenderingContextBase::texImageHelperHTMLVideoElement(
yoffset, zoffset))
return;
- if (functionID == TexImage2D) {
+ if (functionID == TexImage2D && GL_TEXTURE_2D == target &&
+ extensionsUtil()->canUseCopyTextureCHROMIUM(target, internalformat, type,
+ level)) {
// Go through the fast path doing a GPU-GPU textures copy without a readback
- // to system memory if possible. Otherwise, it will fall back to the normal
- // SW path.
- if (GL_TEXTURE_2D == target) {
- if (Extensions3DUtil::canUseCopyTextureCHROMIUM(target, internalformat,
- type, level) &&
- video->copyVideoTextureToPlatformTexture(
- contextGL(), texture->object(), internalformat, type,
- m_unpackPremultiplyAlpha, m_unpackFlipY)) {
- return;
- }
+ // to system memory if possible.
+ // Otherwise, it will fall back to the normal SW path.
+ if (video->copyVideoTextureToPlatformTexture(
+ contextGL(), texture->object(), internalformat, type,
+ m_unpackPremultiplyAlpha, m_unpackFlipY)) {
+ return;
+ }
+ }
- // Try using an accelerated image buffer, this allows YUV conversion to be
- // done on the GPU.
- std::unique_ptr<ImageBufferSurface> surface =
- wrapUnique(new AcceleratedImageBufferSurface(
- IntSize(video->videoWidth(), video->videoHeight())));
- if (surface->isValid()) {
- std::unique_ptr<ImageBuffer> imageBuffer(
- ImageBuffer::create(std::move(surface)));
- if (imageBuffer) {
- // The video element paints an RGBA frame into our surface here. By
- // using an AcceleratedImageBufferSurface, we enable the
- // WebMediaPlayer implementation to do any necessary color space
- // conversion on the GPU (though it
- // may still do a CPU conversion and upload the results).
- video->paintCurrentFrame(
- imageBuffer->canvas(),
- IntRect(0, 0, video->videoWidth(), video->videoHeight()),
- nullptr);
-
- // This is a straight GPU-GPU copy, any necessary color space
- // conversion was handled in the paintCurrentFrameInContext() call.
- if (imageBuffer->copyToPlatformTexture(
- contextGL(), texture->object(), internalformat, type, level,
- m_unpackPremultiplyAlpha, m_unpackFlipY)) {
- return;
- }
+ {
+ // Try using optimized CPU-GPU path for some formats: e.g. Y16 and Y8. It
+ // leaves early for other formats or if frame is stored on GPU.
+ ScopedUnpackParametersResetRestore(
+ this, m_unpackFlipY || m_unpackPremultiplyAlpha);
+ if (video->texImageImpl(getTexImageFunctionName(functionID), target,
+ contextGL(), level, internalformat, format, type,
+ xoffset, yoffset, zoffset, m_unpackFlipY,
+ m_unpackPremultiplyAlpha &&
+ m_unpackColorspaceConversion == GL_NONE))
+ return;
+ }
+
+ if (functionID == TexImage2D && GL_TEXTURE_2D == target &&
+ extensionsUtil()->canUseCopyTextureCHROMIUM(target, internalformat, type,
+ level)) {
+ // Try using an accelerated image buffer, this allows YUV conversion to be
+ // done on the GPU.
+ std::unique_ptr<ImageBufferSurface> surface =
+ wrapUnique(new AcceleratedImageBufferSurface(
+ IntSize(video->videoWidth(), video->videoHeight())));
+ if (surface->isValid()) {
+ std::unique_ptr<ImageBuffer> imageBuffer(
+ ImageBuffer::create(std::move(surface)));
+ if (imageBuffer) {
+ // The video element paints an RGBA frame into our surface here. By
+ // using an AcceleratedImageBufferSurface, we enable the WebMediaPlayer
+ // implementation to do any necessary color space conversion on the GPU
+ // (though it may still do a CPU conversion and upload the results).
+ video->paintCurrentFrame(
+ imageBuffer->canvas(),
+ IntRect(0, 0, video->videoWidth(), video->videoHeight()), nullptr);
+
+ // This is a straight GPU-GPU copy, any necessary color space conversion
+ // was handled in the paintCurrentFrameInContext() call.
+ if (imageBuffer->copyToPlatformTexture(
+ contextGL(), texture->object(), internalformat, type, level,
+ m_unpackPremultiplyAlpha, m_unpackFlipY)) {
+ return;
}
}
}
@@ -5180,7 +5213,7 @@ void WebGLRenderingContextBase::texImageHelperImageBitmap(
return;
}
}
- resetUnpackParameters();
+ ScopedUnpackParametersResetRestore temporaryResetUnpack(this);
if (functionID == TexImage2D) {
texImage2DBase(target, level, internalformat, bitmap->width(),
bitmap->height(), 0, format, type,
@@ -5196,7 +5229,6 @@ void WebGLRenderingContextBase::texImageHelperImageBitmap(
type,
needConversion ? data.data() : pixelDataPtr);
}
- restoreUnpackParameters();
}
void WebGLRenderingContextBase::texImage2D(GLenum target,

Powered by Google App Engine
This is Rietveld 408576698