Refactor AcceleratedStaticBitmapImage

third_party/WebKit/Source/platform/graphics/MailboxTextureHolder.cpp

Index: third_party/WebKit/Source/platform/graphics/MailboxTextureHolder.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/MailboxTextureHolder.cpp b/third_party/WebKit/Source/platform/graphics/MailboxTextureHolder.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..3af6d55eb69035e4d29d5d7cc9f2e9efd64eadbf
--- /dev/null
+++ b/third_party/WebKit/Source/platform/graphics/MailboxTextureHolder.cpp
@@ -0,0 +1,86 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "platform/graphics/MailboxTextureHolder.h"
+
+#include "gpu/command_buffer/client/gles2_interface.h"
+#include "platform/graphics/SkiaTextureHolder.h"
+#include "platform/graphics/gpu/DrawingBuffer.h"
+#include "platform/graphics/gpu/SharedGpuContext.h"
+#include "public/platform/Platform.h"
+#include "skia/ext/texture_handle.h"
+#include "third_party/skia/include/gpu/GrContext.h"
+
+namespace blink {
+
+MailboxTextureHolder::MailboxTextureHolder(
+    const gpu::Mailbox& mailbox,
+    const gpu::SyncToken& syncToken,
+    unsigned textureIdToDeleteAfterMailboxConsumed,
+    WeakPtr<DrawingBuffer> drawingBuffer,
+    IntSize mailboxSize)
+    : m_mailbox(mailbox),
+      m_syncToken(syncToken),
+      m_textureId(textureIdToDeleteAfterMailboxConsumed),
+      m_drawingBuffer(drawingBuffer),
+      m_size(mailboxSize),
+      m_isConvertedFromSkiaTexture(false) {}
+
+MailboxTextureHolder::MailboxTextureHolder(
+    std::unique_ptr<TextureHolder> textureHolder) {
+  DCHECK(textureHolder->isSkiaTextureHolder());
+  sk_sp<SkImage> image = textureHolder->skImage();
+  DCHECK(image);
+
+  gpu::gles2::GLES2Interface* sharedGL = SharedGpuContext::gl();
+  GrContext* sharedGrContext = SharedGpuContext::gr();
+  if (!sharedGrContext) {
+    // Can happen if the context is lost. The SkImage won't be any good now
+    // anyway.
+    return;
+  }
+  GLuint imageTextureId =
+      skia::GrBackendObjectToGrGLTextureInfo(image->getTextureHandle(true))
+          ->fID;
+  sharedGL->BindTexture(GL_TEXTURE_2D, imageTextureId);
+
+  sharedGL->GenMailboxCHROMIUM(m_mailbox.name);
+  sharedGL->ProduceTextureCHROMIUM(GL_TEXTURE_2D, m_mailbox.name);
+  const GLuint64 fenceSync = sharedGL->InsertFenceSyncCHROMIUM();
+  sharedGL->Flush();
+  sharedGL->GenSyncTokenCHROMIUM(fenceSync, m_syncToken.GetData());
+
+  sharedGL->BindTexture(GL_TEXTURE_2D, 0);
+  // We changed bound textures in this function, so reset the GrContext.
+  sharedGrContext->resetContext(kTextureBinding_GrGLBackendState);
+  m_size = IntSize(image->width(), image->height());
+  m_textureId = imageTextureId;
+  m_isConvertedFromSkiaTexture = true;
+}
+
+MailboxTextureHolder::~MailboxTextureHolder() {
+  // Avoid leaking mailboxes in cases where the texture gets recycled by skia.
+  if (SharedGpuContext::isValid()) {
+    SharedGpuContext::gl()->ProduceTextureDirectCHROMIUM(0, GL_TEXTURE_2D,
+                                                         m_mailbox.name);
+  }
+  if (m_drawingBuffer && m_textureId && !m_isConvertedFromSkiaTexture) {
+    m_drawingBuffer->contextGL()->WaitSyncTokenCHROMIUM(m_syncToken.GetData());
+    m_drawingBuffer->contextGL()->DeleteTextures(1, &m_textureId);
+  } else if (m_isConvertedFromSkiaTexture && m_textureId) {
+    /*gpu::gles2::GLES2Interface* sharedGL = SharedGpuContext::gl();
+    sharedGL->WaitSyncTokenCHROMIUM(m_syncToken.GetData());
+    sharedGL->DeleteTextures(1, &m_textureId);*/

[xidachen, 2016/11/02 01:28:08]

If I un-comment this three lines of code, a lot of layout tests (~10) fails. But
when I run those failed layout tests one by one, they all pass...I don't know
how to debug and why these three lines causes failure.

+  }
+}
+
+WebThread* MailboxTextureHolder::imageThread() {
+  return Platform::current()->currentThread();
+}
+
+unsigned MailboxTextureHolder::sharedContextId() {
+  return SharedGpuContext::kNoSharedContext;
+}
+
+}  // namespace blink