Commit b743337e authored by Chris Blume's avatar Chris Blume Committed by Commit Bot

Remove accidental debug logging

In a previous CL [1] some debug logging was added which shouldn't have
been committed.

This CL removes those accidental debug logs.

[1] Add SharedImage thread safe flag
https://chromium-review.googlesource.com/c/chromium/src/+/1504894

Bug: 938594
Change-Id: I938a8fd021823bde0b5017c3ac758ea78fccbf8e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572602Reviewed-by: default avatarEric Karl <ericrk@chromium.org>
Commit-Queue: Chris Blume <cblume@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652195}
parent af2b425d
...@@ -412,26 +412,18 @@ GlLegacySharedImage::GlLegacySharedImage( ...@@ -412,26 +412,18 @@ GlLegacySharedImage::GlLegacySharedImage(
mailbox_, format, size_, color_space, usage, is_thread_safe); mailbox_, format, size_, color_space, usage, is_thread_safe);
EXPECT_TRUE(backing_); EXPECT_TRUE(backing_);
LOG(ERROR) << "cblume 1";
// Check clearing. // Check clearing.
if (!backing_->IsCleared()) { if (!backing_->IsCleared()) {
backing_->SetCleared(); backing_->SetCleared();
EXPECT_TRUE(backing_->IsCleared()); EXPECT_TRUE(backing_->IsCleared());
} }
LOG(ERROR) << "cblume 2";
// First, validate via a legacy mailbox. // First, validate via a legacy mailbox.
GLenum expected_target = GL_TEXTURE_2D; GLenum expected_target = GL_TEXTURE_2D;
EXPECT_TRUE(backing_->ProduceLegacyMailbox(mailbox_manager_)); EXPECT_TRUE(backing_->ProduceLegacyMailbox(mailbox_manager_));
LOG(ERROR) << "cblume 3";
TextureBase* texture_base = mailbox_manager_->ConsumeTexture(mailbox_); TextureBase* texture_base = mailbox_manager_->ConsumeTexture(mailbox_);
LOG(ERROR) << "cblume 4";
// Currently there is no support for passthrough texture on android and hence // Currently there is no support for passthrough texture on android and hence
// in AHB backing. So the TextureBase* should be pointing to a Texture object. // in AHB backing. So the TextureBase* should be pointing to a Texture object.
auto* texture = gles2::Texture::CheckedCast(texture_base); auto* texture = gles2::Texture::CheckedCast(texture_base);
...@@ -445,18 +437,12 @@ GlLegacySharedImage::GlLegacySharedImage( ...@@ -445,18 +437,12 @@ GlLegacySharedImage::GlLegacySharedImage(
EXPECT_EQ(width, size_.width()); EXPECT_EQ(width, size_.width());
EXPECT_EQ(height, size_.height()); EXPECT_EQ(height, size_.height());
LOG(ERROR) << "cblume 5";
shared_image_ = shared_image_ =
shared_image_manager->Register(std::move(backing_), memory_type_tracker); shared_image_manager->Register(std::move(backing_), memory_type_tracker);
LOG(ERROR) << "cblume 6";
auto gl_representation = auto gl_representation =
shared_image_representation_factory->ProduceGLTexture(mailbox_); shared_image_representation_factory->ProduceGLTexture(mailbox_);
LOG(ERROR) << "cblume 7";
EXPECT_TRUE(gl_representation); EXPECT_TRUE(gl_representation);
EXPECT_TRUE(gl_representation->GetTexture()->service_id()); EXPECT_TRUE(gl_representation->GetTexture()->service_id());
EXPECT_EQ(expected_target, gl_representation->GetTexture()->target()); EXPECT_EQ(expected_target, gl_representation->GetTexture()->target());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment