cc::OutputSurfaceClient::InitializeForGL

cc/trees/layer_tree_host_impl.cc

 // Copyright 2011 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/trees/layer_tree_host_impl.h"
 
 #include <algorithm>
 
 #include "base/basictypes.h"
 #include "base/json/json_writer.h"
@@ skipping 1418 matching lines @@
   actual.priority_cutoff_when_visible =
       ManagedMemoryPolicy::CUTOFF_ALLOW_REQUIRED_ONLY;
   return actual;
 }
 
 bool LayerTreeHostImpl::InitializeRenderer(
     scoped_ptr<OutputSurface> output_surface) {
   // Since we will create a new resource provider, we cannot continue to use
   // the old resources (i.e. render_surfaces and texture IDs). Clear them
   // before we destroy the old resource provider.
+  DropLayerResources();
+
+  // Note: order is important here.
+  renderer_.reset();
+  tile_manager_.reset();
+  resource_provider_.reset();
+  output_surface_.reset();
+
+  if (!output_surface->BindToClient(this))
+    return false;
+
+  return DoInitializeRenderer(output_surface.Pass(),
+                              false /* is_deferred_init */);
+}
+
+void LayerTreeHostImpl::DropLayerResources() {
+  // TODO(boliu): Rename LoseOutputSurface to DrawLayerResources.
   if (active_tree_->root_layer())
     ClearRenderSurfaces();
   if (active_tree_->root_layer())
     SendDidLoseOutputSurfaceRecursive(active_tree_->root_layer());
   if (pending_tree_ && pending_tree_->root_layer())
     SendDidLoseOutputSurfaceRecursive(pending_tree_->root_layer());
   if (recycle_tree_ && recycle_tree_->root_layer())
     SendDidLoseOutputSurfaceRecursive(recycle_tree_->root_layer());
   if (resource_provider_)
     resource_provider_->DidLoseOutputSurface();

[piman, 2013/06/05 00:39:37]

You don't want to do that in the re-initialization case, it will cause the RP
think the context is lost and won't properly delete resources

[boliu, 2013/06/05 01:01:54]

Actually this needs more work as I remembered Dana pointing out that layers are
just dropping resources instead of properly releasing them.

Is it correct to not change layers code, but destroy the current
ResourceProvider in DeferredInitialize. I think destroying the ResourceProvider
cleans up all the resources?

+}
 
-  // Note: order is important here.
-  renderer_.reset();
-  tile_manager_.reset();
-  resource_provider_.reset();
-  output_surface_.reset();
-
-  if (!output_surface->BindToClient(this))
-    return false;
-
-  if (output_surface->capabilities().deferred_gl_initialization) {
+bool LayerTreeHostImpl::DoInitializeRenderer(
+    scoped_ptr<OutputSurface> output_surface,
+    bool is_deferred_init) {
+  if (output_surface->capabilities().deferred_gl_initialization &&
+      !is_deferred_init) {
     // TODO(joth): Defer creating the Renderer too, until GL is initialized.
     // See http://crbug.com/230197
     renderer_ = SoftwareRenderer::Create(this, output_surface.get(), NULL);
   } else {
     scoped_ptr<ResourceProvider> resource_provider = ResourceProvider::Create(
         output_surface.get(), settings_.highp_threshold_min);
     if (!resource_provider)
       return false;
 
     if (output_surface->capabilities().has_parent_compositor) {
@@ skipping 36 matching lines @@
 
   // See note in LayerTreeImpl::UpdateDrawProperties.  Renderer needs
   // to be initialized to get max texture size.
   active_tree_->set_needs_update_draw_properties();
   if (pending_tree_)
     pending_tree_->set_needs_update_draw_properties();
 
   return true;
 }
 
+bool LayerTreeHostImpl::DeferredInitialize(
+    scoped_refptr<ContextProvider> offscreen_context_provider) {
+  DCHECK(output_surface_->capabilities().deferred_gl_initialization);
+  DCHECK(output_surface_->context3d());
+
+  DropLayerResources();
+  bool success =
+      DoInitializeRenderer(output_surface_.Pass(), true /* is_deferred_init */);
+  client_->DidTryInitializeRendererOnImplThread(success,
+                                                offscreen_context_provider);
+  return success;
+}
+
 void LayerTreeHostImpl::SetViewportSize(gfx::Size device_viewport_size) {
   if (device_viewport_size == device_viewport_size_)
     return;
 
   if (pending_tree_ && device_viewport_size_ != device_viewport_size)
     active_tree_->SetViewportSizeInvalid();
 
   device_viewport_size_ = device_viewport_size;
 
   UpdateMaxScrollOffset();
@@ skipping 746 matching lines @@
 }
 
 void LayerTreeHostImpl::SetDebugState(const LayerTreeDebugState& debug_state) {
   if (debug_state_.continuous_painting != debug_state.continuous_painting)
     paint_time_counter_->ClearHistory();
 
   debug_state_ = debug_state;
 }
 
 }  // namespace cc