cc: Update Main RendererCapabilities on DeferredInitialize

cc/layers/texture_layer.cc

 // Copyright 2010 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/layers/texture_layer.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/location.h"
 #include "base/synchronization/lock.h"
@@ skipping 258 matching lines @@
       release_callback = holder->GetCallbackForImplThread();
     }
     texture_layer->SetTextureMailbox(texture_mailbox, release_callback.Pass());
     needs_set_mailbox_ = false;
   } else {
     texture_layer->SetTextureId(texture_id_);
     content_committed_ = DrawsContent();
   }
 }
 
+void TextureLayer::OnOutputSurfaceCreated() {
+  if (uses_mailbox_) {
+    holder_ref_.reset();

[boliu, 2014/02/13 20:14:39]

I removed the DCHECK for now. This broke context_lost test on the gpu bots.

The issue here is that in a chrome context loss, it recreates another renderer
with the same RendererCapabilities. So there is nothing stopping a new commit
going through, even if the context is lost.

I still think it's correct here to drop the ref here. But to make the DCHECK
work, we have to pass a bool param saying whether the renderer caps actually
changed with the new OS, and only assert single ref if it did change. Thought
I'd get your opinion first before implementing this though.

[danakj, 2014/02/14 17:59:11]

Maybe we can do this without adding more LayerAPI.. would it work to just
SetNeedsUpdate on the tree when renderer caps change, and then in
Layer::Update() it could decide to drop things as needed. This layer does not
even use the RendererCapabilities, so maybe I'm not sure why it cares that they
change even?

[boliu, 2014/02/14 18:44:42]

RendererCapabilities is used in the
layer_tree_host()->UsingSharedMemoryResources(). I think checking if that
changed in Update works as well. Trying to add a test for it now.

+  }
+}
+
 Region TextureLayer::VisibleContentOpaqueRegion() const {
   if (contents_opaque())
     return visible_content_rect();
 
   if (blend_background_color_ && (SkColorGetA(background_color()) == 0xFF))
     return visible_content_rect();
 
   return Region();
 }
 
@@ skipping 62 matching lines @@
 
 void TextureLayer::TextureMailboxHolder::ReturnAndReleaseOnImplThread(
     uint32 sync_point,
     bool is_lost) {
   Return(sync_point, is_lost);
   message_loop_->PostTask(
       FROM_HERE, base::Bind(&TextureMailboxHolder::InternalRelease, this));
 }
 
 }  // namespace cc