Mailbox support for texture layers.

cc/resource_provider.cc

 // Copyright 2012 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 "cc/resource_provider.h"
 
 #include <limits.h>
 
 #include "base/debug/alias.h"
 #include "base/hash_tables.h"
@@ skipping 217 matching lines @@
     GLC(context3d, context3d->texParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE));
     GLC(context3d, context3d->texParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE));
 
     ResourceId id = m_nextId++;
     Resource resource(textureId, gfx::Size(), 0, GL_LINEAR);
     resource.external = true;
     m_resources[id] = resource;
     return id;
 }
 
+ResourceProvider::ResourceId ResourceProvider::createResourceFromTextureMailbox(const std::string& mailbox)
+{
+    DCHECK(m_threadChecker.CalledOnValidThread());
+
+    const int8* name = reinterpret_cast<const int8*>(mailbox.c_str());

[piman, 2012/12/20 02:14:21]

nit: .data() instead of c_str() to avoid forcing an extra \0.

[danakj, 2012/12/20 02:16:19]

how come you need reinterpret_cast to do this? what if you use .data() instead?

[alexst (slow to review), 2012/12/21 14:15:21]

Done.

[alexst (slow to review), 2012/12/21 14:15:21]

consumeTextureCHROMIUM expects signed data both, c_str() and data() are
unsigned, hence the cast.

On 2012/12/20 02:16:19, danakj wrote:

+    WebGraphicsContext3D* context3d = m_outputSurface->Context3D();
+    DCHECK(context3d);
+    unsigned textureId;
+    GLC(context3d, textureId = context3d->createTexture());
+    GLC(context3d, context3d->bindTexture(GL_TEXTURE_2D, textureId));
+    GLC(context3d, context3d->texParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR));
+    GLC(context3d, context3d->texParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR));
+    GLC(context3d, context3d->texParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE));
+    GLC(context3d, context3d->texParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE));

[piman, 2012/12/20 02:14:21]

nit: we shouldn't have to set the tex parameters, since they should already have
been set by the producer.

[alexst (slow to review), 2012/12/21 14:15:21]

Done.

+    GLC(context3d, context3d->consumeTextureCHROMIUM(GL_TEXTURE_2D, name));
+    GLC(context3d, context3d->flush());

[danakj, 2012/12/20 02:16:19]

how come the flush is needed?

[alexst (slow to review), 2012/12/21 14:15:21]

Backing texture for the mailbox is in a different context, we need to make sure
it latches to the newly generated texture id before we start using it.
On 2012/12/20 02:16:19, danakj wrote:

+
+    ResourceId id = m_nextId++;
+    Resource resource(textureId, gfx::Size(), 0, GL_LINEAR);

[piman, 2012/12/20 02:14:21]

We probably want to pass all the info so that these can have the proper values.

[alexst (slow to review), 2012/12/21 14:15:21]

I only know the size but not the format from swapBuffers. 

It's easy to plumb it through, but will it be used anywhere? External texture
id's follow the same pattern and since mailboxes are an alternate way to
represent them, I left this part the same.

On 2012/12/20 02:14:21, piman wrote:

+    resource.external = true;
+    resource.mailbox.setName(name);
+    m_resources[id] = resource;
+    return id;
+}
+
 void ResourceProvider::deleteResource(ResourceId id)
 {
     DCHECK(m_threadChecker.CalledOnValidThread());
     ResourceMap::iterator it = m_resources.find(id);
     CHECK(it != m_resources.end());
     Resource* resource = &it->second;
     DCHECK(!resource->lockedForWrite);
     DCHECK(!resource->lockForReadCount);
     DCHECK(!resource->markedForDeletion);
 
@@ skipping 15 matching lines @@
     if (resource->glUploadQueryId) {
         WebGraphicsContext3D* context3d = m_outputSurface->Context3D();
         DCHECK(context3d);
         GLC(context3d, context3d->deleteQueryEXT(resource->glUploadQueryId));
     }
     if (resource->glPixelBufferId) {
         WebGraphicsContext3D* context3d = m_outputSurface->Context3D();
         DCHECK(context3d);
         GLC(context3d, context3d->deleteBuffer(resource->glPixelBufferId));
     }
+    if (!resource->mailbox.isZero()) {

[piman, 2012/12/20 02:14:21]

This is not quite right, since we will have a mailbox set for
compositor-originated resources we sent to the parent, but we don't want to
produce before sending in that case.

[alexst (slow to review), 2012/12/21 14:15:21]

Done.

+        WebGraphicsContext3D* context3d = m_outputSurface->Context3D();
+        DCHECK(context3d);
+        GLC(context3d, context3d->bindTexture(GL_TEXTURE_2D, resource->glId));
+        GLC(context3d, context3d->produceTextureCHROMIUM(GL_TEXTURE_2D, resource->mailbox.name));
+        GLC(context3d, context3d->deleteTexture(resource->glId));
+    }
     if (resource->pixels)
         delete[] resource->pixels;
     if (resource->pixelBuffer)
         delete[] resource->pixelBuffer;
 
     m_resources.erase(it);
 }
 
 ResourceProvider::ResourceType ResourceProvider::resourceType(ResourceId id)
 {
@@ skipping 667 matching lines @@
             return false;
     }
 
     resource->pendingSetPixels = false;
     unlockForWrite(id);
 
     return true;
 }
 
 }  // namespace cc