Index: content/common/gpu/client/gl_helper.cc |
diff --git a/content/common/gpu/client/gl_helper.cc b/content/common/gpu/client/gl_helper.cc |
index 017ed977eda441116657751ee6d9524ad03f3b0d..3b7e23abb4731302787f644b1312965e43222943 100644 |
--- a/content/common/gpu/client/gl_helper.cc |
+++ b/content/common/gpu/client/gl_helper.cc |
@@ -127,14 +127,12 @@ |
const gfx::Rect& src_subrect, |
const gfx::Size& dst_size, |
unsigned char* out, |
- bool readback_config_rgb565, |
const base::Callback<void(bool)>& callback, |
GLHelper::ScalerQuality quality); |
void ReadbackTextureSync(GLuint texture, |
const gfx::Rect& src_rect, |
- unsigned char* out, |
- SkBitmap::Config format); |
+ unsigned char* out); |
// Reads back bytes from the currently bound frame buffer. |
// Note that dst_size is specified in bytes, not pixels. |
@@ -142,7 +140,6 @@ |
int32 bytes_per_row, // generally dst_size.width() * 4 |
int32 row_stride_bytes, // generally dst_size.width() * 4 |
unsigned char* out, |
- bool readback_config_rgb565, |
const base::Callback<void(bool)>& callback); |
void ReadbackPlane(TextureFrameBufferPair* source, |
@@ -288,11 +285,10 @@ |
const gfx::Size& dst_size, |
bool vertically_flip_texture, |
bool swizzle, |
- bool readback_config_rgb565, |
GLHelper::ScalerQuality quality); |
static void nullcallback(bool success) {} |
- void ReadbackDone(Request *request, int bytes_per_pixel); |
+ void ReadbackDone(Request* request); |
void FinishRequest(Request* request, bool result); |
void CancelRequests(); |
@@ -334,7 +330,6 @@ |
const gfx::Size& dst_size, |
bool vertically_flip_texture, |
bool swizzle, |
- bool readback_config_rgb565, |
GLHelper::ScalerQuality quality) { |
scoped_ptr<ScalerInterface> scaler( |
helper_->CreateScaler(quality, |
@@ -348,18 +343,14 @@ |
gl_->GenTextures(1, &dst_texture); |
{ |
ScopedTextureBinder<GL_TEXTURE_2D> texture_binder(gl_, dst_texture); |
- GLenum format = readback_config_rgb565 ? GL_RGB : GL_RGBA; |
- GLenum type = readback_config_rgb565 ? |
- GL_UNSIGNED_SHORT_5_6_5 : |
- GL_UNSIGNED_BYTE; |
gl_->TexImage2D(GL_TEXTURE_2D, |
0, |
- format, |
+ GL_RGBA, |
dst_size.width(), |
dst_size.height(), |
0, |
- format, |
- type, |
+ GL_RGBA, |
+ GL_UNSIGNED_BYTE, |
NULL); |
} |
scaler->Scale(src_texture, dst_texture); |
@@ -371,48 +362,41 @@ |
int32 bytes_per_row, |
int32 row_stride_bytes, |
unsigned char* out, |
- bool readback_config_rgb565, |
const base::Callback<void(bool)>& callback) { |
Request* request = |
new Request(dst_size, bytes_per_row, row_stride_bytes, out, callback); |
request_queue_.push(request); |
request->buffer = 0u; |
- int bytes_per_pixel = readback_config_rgb565 ? 2 : 4; |
gl_->GenBuffers(1, &request->buffer); |
gl_->BindBuffer(GL_PIXEL_PACK_TRANSFER_BUFFER_CHROMIUM, request->buffer); |
gl_->BufferData(GL_PIXEL_PACK_TRANSFER_BUFFER_CHROMIUM, |
- bytes_per_pixel * dst_size.GetArea(), |
+ 4 * dst_size.GetArea(), |
NULL, |
GL_STREAM_READ); |
request->query = 0u; |
gl_->GenQueriesEXT(1, &request->query); |
gl_->BeginQueryEXT(GL_ASYNC_PIXEL_PACK_COMPLETED_CHROMIUM, request->query); |
- GLenum format = readback_config_rgb565 ? GL_RGB : GL_RGBA; |
- GLenum type = readback_config_rgb565 ? |
- GL_UNSIGNED_SHORT_5_6_5 : |
- GL_UNSIGNED_BYTE; |
gl_->ReadPixels(0, |
0, |
dst_size.width(), |
dst_size.height(), |
- format, |
- type, |
+ GL_RGBA, |
+ GL_UNSIGNED_BYTE, |
NULL); |
gl_->EndQueryEXT(GL_ASYNC_PIXEL_PACK_COMPLETED_CHROMIUM); |
gl_->BindBuffer(GL_PIXEL_PACK_TRANSFER_BUFFER_CHROMIUM, 0); |
context_support_->SignalQuery( |
request->query, |
- base::Bind(&CopyTextureToImpl::ReadbackDone, AsWeakPtr(), |
- request, bytes_per_pixel)); |
-} |
+ base::Bind(&CopyTextureToImpl::ReadbackDone, AsWeakPtr(), request)); |
+} |
+ |
void GLHelper::CopyTextureToImpl::CropScaleReadbackAndCleanTexture( |
GLuint src_texture, |
const gfx::Size& src_size, |
const gfx::Rect& src_subrect, |
const gfx::Size& dst_size, |
unsigned char* out, |
- bool readback_config_rgb565, |
const base::Callback<void(bool)>& callback, |
GLHelper::ScalerQuality quality) { |
GLuint texture = ScaleTexture(src_texture, |
@@ -425,51 +409,33 @@ |
#else |
false, |
#endif |
- readback_config_rgb565, |
quality); |
- ScopedFramebuffer dst_framebuffer(gl_); |
- ScopedFramebufferBinder<GL_FRAMEBUFFER> framebuffer_binder(gl_, |
- dst_framebuffer); |
- ScopedTextureBinder<GL_TEXTURE_2D> texture_binder(gl_, texture); |
- gl_->FramebufferTexture2D(GL_FRAMEBUFFER, |
- GL_COLOR_ATTACHMENT0, |
- GL_TEXTURE_2D, |
- texture, |
- 0); |
- int bytes_per_pixel = readback_config_rgb565 ? 2 : 4; |
- ReadbackAsync(dst_size, |
- dst_size.width() * bytes_per_pixel, |
- dst_size.width() * bytes_per_pixel, |
- out, |
- readback_config_rgb565, |
- callback); |
- gl_->DeleteTextures(1, &texture); |
-} |
- |
-void GLHelper::CopyTextureToImpl::ReadbackTextureSync(GLuint texture, |
- const gfx::Rect& src_rect, |
- unsigned char* out, |
- SkBitmap::Config config) { |
ScopedFramebuffer dst_framebuffer(gl_); |
ScopedFramebufferBinder<GL_FRAMEBUFFER> framebuffer_binder(gl_, |
dst_framebuffer); |
ScopedTextureBinder<GL_TEXTURE_2D> texture_binder(gl_, texture); |
gl_->FramebufferTexture2D( |
GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, GL_TEXTURE_2D, texture, 0); |
- DCHECK((config == SkBitmap::kRGB_565_Config) || |
- (config == SkBitmap::kARGB_8888_Config)); |
- GLenum format = (config == SkBitmap::kRGB_565_Config) ? |
- GL_RGB : |
- GL_RGBA; |
- GLenum type = (config == SkBitmap::kRGB_565_Config) ? |
- GL_UNSIGNED_SHORT_5_6_5 : |
- GL_UNSIGNED_BYTE; |
+ ReadbackAsync( |
+ dst_size, dst_size.width() * 4, dst_size.width() * 4, out, callback); |
+ gl_->DeleteTextures(1, &texture); |
+} |
+ |
+void GLHelper::CopyTextureToImpl::ReadbackTextureSync(GLuint texture, |
+ const gfx::Rect& src_rect, |
+ unsigned char* out) { |
+ ScopedFramebuffer dst_framebuffer(gl_); |
+ ScopedFramebufferBinder<GL_FRAMEBUFFER> framebuffer_binder(gl_, |
+ dst_framebuffer); |
+ ScopedTextureBinder<GL_TEXTURE_2D> texture_binder(gl_, texture); |
+ gl_->FramebufferTexture2D( |
+ GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, GL_TEXTURE_2D, texture, 0); |
gl_->ReadPixels(src_rect.x(), |
src_rect.y(), |
src_rect.width(), |
src_rect.height(), |
- format, |
- type, |
+ GL_RGBA, |
+ GL_UNSIGNED_BYTE, |
out); |
} |
@@ -485,12 +451,10 @@ |
dst_size, |
vertically_flip_texture, |
false, |
- false, |
quality); |
} |
-void GLHelper::CopyTextureToImpl::ReadbackDone(Request* finished_request, |
- int bytes_per_pixel) { |
+void GLHelper::CopyTextureToImpl::ReadbackDone(Request* finished_request) { |
TRACE_EVENT0("mirror", |
"GLHelper::CopyTextureToImpl::CheckReadbackFramebufferComplete"); |
finished_request->done = true; |
@@ -510,22 +474,22 @@ |
GL_PIXEL_PACK_TRANSFER_BUFFER_CHROMIUM, GL_READ_ONLY)); |
if (data) { |
result = true; |
- if (request->bytes_per_row == request->size.width() * bytes_per_pixel && |
+ if (request->bytes_per_row == request->size.width() * 4 && |
request->bytes_per_row == request->row_stride_bytes) { |
- memcpy(request->pixels, data, |
- request->size.GetArea() * bytes_per_pixel); |
+ memcpy(request->pixels, data, request->size.GetArea() * 4); |
} else { |
unsigned char* out = request->pixels; |
for (int y = 0; y < request->size.height(); y++) { |
memcpy(out, data, request->bytes_per_row); |
out += request->row_stride_bytes; |
- data += request->size.width() * bytes_per_pixel; |
+ data += request->size.width() * 4; |
} |
} |
gl_->UnmapBufferCHROMIUM(GL_PIXEL_PACK_TRANSFER_BUFFER_CHROMIUM); |
} |
gl_->BindBuffer(GL_PIXEL_PACK_TRANSFER_BUFFER_CHROMIUM, 0); |
} |
+ |
FinishRequest(request, result); |
} |
} |
@@ -555,10 +519,7 @@ |
} |
GLHelper::GLHelper(GLES2Interface* gl, gpu::ContextSupport* context_support) |
- : gl_(gl), |
- context_support_(context_support), |
- initialized_565_format_check_(false), |
- support_565_format_(false) {} |
+ : gl_(gl), context_support_(context_support) {} |
GLHelper::~GLHelper() {} |
@@ -568,7 +529,6 @@ |
const gfx::Rect& src_subrect, |
const gfx::Size& dst_size, |
unsigned char* out, |
- bool readback_config_rgb565, |
const base::Callback<void(bool)>& callback) { |
InitCopyTextToImpl(); |
copy_texture_to_impl_->CropScaleReadbackAndCleanTexture( |
@@ -577,7 +537,6 @@ |
src_subrect, |
dst_size, |
out, |
- readback_config_rgb565, |
callback, |
GLHelper::SCALER_QUALITY_FAST); |
} |
@@ -589,22 +548,18 @@ |
const gfx::Rect& src_subrect, |
const gfx::Size& dst_size, |
unsigned char* out, |
- bool readback_config_rgb565, |
const base::Callback<void(bool)>& callback) { |
GLuint mailbox_texture = ConsumeMailboxToTexture(src_mailbox, sync_point); |
CropScaleReadbackAndCleanTexture( |
- mailbox_texture, src_size, src_subrect, dst_size, out, |
- readback_config_rgb565, |
- callback); |
+ mailbox_texture, src_size, src_subrect, dst_size, out, callback); |
gl_->DeleteTextures(1, &mailbox_texture); |
} |
void GLHelper::ReadbackTextureSync(GLuint texture, |
const gfx::Rect& src_rect, |
- unsigned char* out, |
- SkBitmap::Config format) { |
+ unsigned char* out) { |
InitCopyTextToImpl(); |
- copy_texture_to_impl_->ReadbackTextureSync(texture, src_rect, out, format); |
+ copy_texture_to_impl_->ReadbackTextureSync(texture, src_rect, out); |
} |
GLuint GLHelper::CopyTexture(GLuint texture, const gfx::Size& size) { |
@@ -772,46 +727,6 @@ |
GL_TEXTURE_2D, 0, GL_RGB, 0, 0, size.width(), size.height(), 0); |
} |
-bool GLHelper::CanUseRgb565Readback() { |
- if(initialized_565_format_check_){ |
- return support_565_format_; |
- } |
- const int kTestSize = 64; |
- GLuint dst_texture = 0u; |
- gl_->GenTextures(1, &dst_texture); |
- ScopedTextureBinder<GL_TEXTURE_2D> texture_binder(gl_, dst_texture); |
- gl_->TexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR); |
- gl_->TexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR); |
- gl_->TexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE); |
- gl_->TexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE); |
- gl_->TexImage2D(GL_TEXTURE_2D, |
- 0, |
- GL_RGB, |
- kTestSize, |
- kTestSize, |
- 0, |
- GL_RGB, |
- GL_UNSIGNED_SHORT_5_6_5, |
- NULL); |
- ScopedFramebuffer dst_framebuffer(gl_); |
- ScopedFramebufferBinder<GL_FRAMEBUFFER> framebuffer_binder(gl_, |
- dst_framebuffer); |
- gl_->FramebufferTexture2D(GL_FRAMEBUFFER, |
- GL_COLOR_ATTACHMENT0, |
- GL_TEXTURE_2D, |
- dst_texture, |
- 0); |
- int ext_format, ext_type; |
- gl_->GetIntegerv(GL_IMPLEMENTATION_COLOR_READ_FORMAT, &ext_format); |
- gl_->GetIntegerv(GL_IMPLEMENTATION_COLOR_READ_TYPE, &ext_type); |
- gl_->DeleteTextures(1, &dst_texture); |
- if ((ext_format == GL_RGB) && (ext_type == GL_UNSIGNED_SHORT_5_6_5)) { |
- support_565_format_ = true; |
- } |
- initialized_565_format_check_ = true; |
- return support_565_format_; |
-} |
- |
void GLHelper::CopyTextureToImpl::ReadbackPlane( |
TextureFrameBufferPair* source, |
const scoped_refptr<media::VideoFrame>& target, |
@@ -821,12 +736,11 @@ |
const base::Callback<void(bool)>& callback) { |
gl_->BindFramebuffer(GL_FRAMEBUFFER, source->framebuffer()); |
size_t offset = target->stride(plane) * (dst_subrect.y() >> size_shift) + |
- (dst_subrect.x() >> size_shift); |
+ (dst_subrect.x() >> size_shift); |
ReadbackAsync(source->size(), |
dst_subrect.width() >> size_shift, |
target->stride(plane), |
target->data(plane) + offset, |
- false, |
callback); |
} |