UI Resource Manager

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 22 matching lines @@
 #include "cc/output/copy_output_request.h"
 #include "cc/output/delegating_renderer.h"
 #include "cc/output/gl_renderer.h"
 #include "cc/output/software_renderer.h"
 #include "cc/quads/render_pass_draw_quad.h"
 #include "cc/quads/shared_quad_state.h"
 #include "cc/quads/solid_color_draw_quad.h"
 #include "cc/resources/memory_history.h"
 #include "cc/resources/picture_layer_tiling.h"
 #include "cc/resources/prioritized_resource_manager.h"
+#include "cc/resources/ui_resource_bitmap.h"
 #include "cc/scheduler/delay_based_time_source.h"
 #include "cc/scheduler/texture_uploader.h"
 #include "cc/trees/damage_tracker.h"
 #include "cc/trees/layer_tree_host.h"
 #include "cc/trees/layer_tree_host_common.h"
 #include "cc/trees/layer_tree_impl.h"
 #include "cc/trees/quad_culler.h"
 #include "cc/trees/single_thread_proxy.h"
 #include "cc/trees/tree_synchronizer.h"
+#include "gpu/GLES2/gl2extchromium.h"
 #include "ui/gfx/size_conversions.h"
 #include "ui/gfx/vector2d_conversions.h"
 
 namespace {
 
 void DidVisibilityChange(cc::LayerTreeHostImpl* id, bool visible) {
   if (visible) {
     TRACE_EVENT_ASYNC_BEGIN1("webkit",
                              "LayerTreeHostImpl::SetVisible",
                              id,
@@ skipping 1241 matching lines @@
       top_controls_manager_ ? top_controls_manager_->content_top_offset() : 0.f;
   return gfx::SizeF(dip_size.width(),
                     dip_size.height() - top_offset - overdraw_bottom_height_);
 }
 
 const LayerTreeSettings& LayerTreeHostImpl::Settings() const {
   return settings();
 }
 
 void LayerTreeHostImpl::DidLoseOutputSurface() {
+  // Remove all existing maps from UIResourceId to ResourceId.
+  ui_resource_map_.clear();
+  // Also remove any requests in the pending queue.  Reasons:
+  // 1. If the request was a creation, then the main thread will issue a special

[aelias_OOO_until_Jul13, 2013/07/26 23:57:31]

It won't reissue creations so this is incorrect.

I think you should never clear the request queue.  It's enough to have wiped the
LayerTreeHostImpl::ui_resource_map_.  Since at the moment of processing each
delete request, nothing will happen if it was already unmapped to due lost
context.

[powei, 2013/07/30 21:47:00]

The main thread issues a "lost resource" creation for all entries in the
ui_resource_client_map_.  If a request is in the impl-side request queue, then
it must have an entry in ui_resource_client_map_.  That's what I mean by the
comment.  

This impl-side request queue probably needs to be cleared because any creation
requests left in this queue will be duplicated by the main-thread "re-creation
requests", and there would be two creation requests for the same UIResourceId,
and would not pass the DCHECK on line 2439.

[aelias_OOO_until_Jul13, 2013/07/30 22:28:44]

But the main thread is avoiding re-creation requests for anything that has a
pending create or delete, so it seems like that shouldn't be a problem?

[powei, 2013/07/30 22:37:50]

For impl-side painting, the main thread actually copies the request queue to the
pending tree and clears its queue (layer_tree_host.cc:366).  I guess,
alternatively, we can think about keeping the main-thread queue around until
impl-side has finished the requests.

[aelias_OOO_until_Jul13, 2013/07/30 23:04:29]

Good point.  The main thread has a pointer to the pending tree so we don't need
to do "keep around" anything extra.  We could just exclude pending-tree
create/deletes as well when deciding what to send recreates requests for.

+  // "resource lost" re-creation.
+  // 2. If the request was a deletion, then the request is moot since the
+  // resource no longer exists.
+  if (pending_tree())
+    pending_tree()->set_ui_resource_request_queue(UIResourceRequestQueue());
+
   // TODO(jamesr): The renderer_ check is needed to make some of the
   // LayerTreeHostContextTest tests pass, but shouldn't be necessary (or
   // important) in production. We should adjust the test to not need this.
   if (renderer_)
     client_->DidLoseOutputSurfaceOnImplThread();
 }
 
 void LayerTreeHostImpl::Readback(void* pixels,
                                  gfx::Rect rect_in_device_viewport) {
   DCHECK(renderer_);
@@ skipping 91 matching lines @@
         TreeSynchronizer::SynchronizeTrees(pending_tree_->root_layer(),
                                            active_tree_->DetachLayerTree(),
                                            active_tree_.get()));
   }
   TreeSynchronizer::PushProperties(pending_tree_->root_layer(),
                                    active_tree_->root_layer());
   DCHECK(!recycle_tree_);
 
   pending_tree_->PushPropertiesTo(active_tree_.get());
 
+  // Process any requests in the UI resource queue.  The request queue is given
+  // in LayerTreeHost::FinishCommitOnImplThread.  This must take place before
+  // the swap.
+  pending_tree_->ProcessUIResourceRequestQueue();
+
   // Now that we've synced everything from the pending tree to the active
   // tree, rename the pending tree the recycle tree so we can reuse it on the
   // next sync.
   pending_tree_.swap(recycle_tree_);
 
   active_tree_->SetRootLayerScrollOffsetDelegate(
       root_layer_scroll_offset_delegate_);
   active_tree_->DidBecomeActive();
 
   // Reduce wasted memory now that unlinked resources are guaranteed not
@@ skipping 970 matching lines @@
     const LayerTreeDebugState& new_debug_state) {
   if (LayerTreeDebugState::Equal(debug_state_, new_debug_state))
     return;
   if (debug_state_.continuous_painting != new_debug_state.continuous_painting)
     paint_time_counter_->ClearHistory();
 
   debug_state_ = new_debug_state;
   SetFullRootLayerDamage();
 }
 
+void LayerTreeHostImpl::CreateUIResource(
+    UIResourceId uid,
+    scoped_refptr<UIResourceBitmap> bitmap) {
+  DCHECK(bitmap->GetFormat() == UIResourceBitmap::RGBA8);
+
+  ResourceProvider::ResourceId id = resource_provider_->CreateResource(
+      bitmap->GetSize(), GL_RGBA, ResourceProvider::TextureUsageAny);
+
+  DCHECK(ui_resource_map_.find(uid) == ui_resource_map_.end());
+  ui_resource_map_[uid] = id;
+  gfx::Size bitmap_size = bitmap->GetSize();
+  resource_provider_->SetPixels(id,
+                                reinterpret_cast<uint8_t*>(bitmap->GetPixels()),
+                                gfx::Rect(bitmap_size),
+                                gfx::Rect(bitmap_size),
+                                gfx::Vector2d(0, 0));
+}
+
+void LayerTreeHostImpl::DeleteUIResource(UIResourceId uid) {
+  ResourceProvider::ResourceId id = ResourceIdForUIResource(uid);
+  if (id) {
+    resource_provider_->DeleteResource(id);
+    ui_resource_map_.erase(uid);
+  }
+}
+
+ResourceProvider::ResourceId LayerTreeHostImpl::ResourceIdForUIResource(
+    UIResourceId uid) const {
+  UIResourceMap::const_iterator iter = ui_resource_map_.find(uid);
+  if (iter != ui_resource_map_.end())
+    return iter->second;
+  return 0;
+}
+
 }  // namespace cc