cc: Rework overlays to not use the ResourceProvider and pass texture size

cc/blink/web_external_texture_layer_impl.cc

 // Copyright 2014 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/blink/web_external_texture_layer_impl.h"
 
 #include "cc/blink/web_external_bitmap_impl.h"
 #include "cc/blink/web_layer_impl.h"
 #include "cc/layers/texture_layer.h"
 #include "cc/resources/resource_update_queue.h"
@@ skipping 69 matching lines @@
   if (!client_->prepareMailbox(&client_mailbox, bitmap)) {
     if (bitmap)
       free_bitmaps_.push_back(bitmap);
     return false;
   }
   gpu::Mailbox name;
   name.SetName(client_mailbox.name);
   if (bitmap) {
     *mailbox = cc::TextureMailbox(bitmap->shared_bitmap(), bitmap->size());
   } else {
-    *mailbox =
-        cc::TextureMailbox(name, GL_TEXTURE_2D, client_mailbox.syncPoint);
+    *mailbox = cc::TextureMailbox(name, GL_TEXTURE_2D, client_mailbox.syncPoint,
+                                  gfx::Size(), client_mailbox.allowOverlay);

[danakj, 2015/05/29 20:04:14]

TODO to pass a valid size and DCHECK allowOverlay => non-empty size?

[achaulk, 2015/06/01 15:43:10]

Done.

   }
-  mailbox->set_allow_overlay(client_mailbox.allowOverlay);
   mailbox->set_nearest_neighbor(client_mailbox.nearestNeighbor);
 
   if (mailbox->IsValid()) {
     *release_callback = cc::SingleReleaseCallback::Create(
         base::Bind(&WebExternalTextureLayerImpl::DidReleaseMailbox,
                    this->AsWeakPtr(),
                    client_mailbox,
                    bitmap));
   }
 
@@ skipping 19 matching lines @@
   DCHECK(layer);
   blink::WebExternalTextureMailbox available_mailbox;
   memcpy(available_mailbox.name, mailbox.name, sizeof(available_mailbox.name));
   available_mailbox.syncPoint = sync_point;
   if (bitmap)
     layer->free_bitmaps_.push_back(bitmap);
   layer->client_->mailboxReleased(available_mailbox, lost_resource);
 }
 
 }  // namespace cc_blink