Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1080)

Unified Diff: cc/layers/ui_resource_layer.cc

Issue 2322943003: cc: Move UI Resource management out of LayerTreeHost. (Closed)
Patch Set: virtual dtor Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/layers/scrollbar_layer_unittest.cc ('k') | cc/layers/ui_resource_layer_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/ui_resource_layer.cc
diff --git a/cc/layers/ui_resource_layer.cc b/cc/layers/ui_resource_layer.cc
index de86098ad71110c8458b3404e7d517bc9783aebc..a887c0f0514ac91e64eaf38469fe24b3a9aeeb84 100644
--- a/cc/layers/ui_resource_layer.cc
+++ b/cc/layers/ui_resource_layer.cc
@@ -9,6 +9,7 @@
#include "cc/layers/ui_resource_layer_impl.h"
#include "cc/resources/scoped_ui_resource.h"
#include "cc/resources/ui_resource_bitmap.h"
+#include "cc/resources/ui_resource_manager.h"
#include "cc/trees/layer_tree_host.h"
namespace cc {
@@ -19,15 +20,18 @@ namespace {
class ScopedUIResourceHolder : public UIResourceLayer::UIResourceHolder {
public:
static std::unique_ptr<ScopedUIResourceHolder> Create(
- LayerTreeHost* host,
+ UIResourceManager* ui_resource_manager,
const SkBitmap& skbitmap) {
- return base::WrapUnique(new ScopedUIResourceHolder(host, skbitmap));
+ return base::WrapUnique(
+ new ScopedUIResourceHolder(ui_resource_manager, skbitmap));
}
UIResourceId id() override { return resource_->id(); }
private:
- ScopedUIResourceHolder(LayerTreeHost* host, const SkBitmap& skbitmap) {
- resource_ = ScopedUIResource::Create(host, UIResourceBitmap(skbitmap));
+ ScopedUIResourceHolder(UIResourceManager* ui_resource_manager,
+ const SkBitmap& skbitmap) {
+ resource_ = ScopedUIResource::Create(ui_resource_manager,
+ UIResourceBitmap(skbitmap));
}
std::unique_ptr<ScopedUIResource> resource_;
@@ -118,9 +122,9 @@ void UIResourceLayer::RecreateUIResourceHolder() {
void UIResourceLayer::SetBitmap(const SkBitmap& skbitmap) {
bitmap_ = skbitmap;
- if (layer_tree_host() && !bitmap_.empty()) {
- ui_resource_holder_ =
- ScopedUIResourceHolder::Create(layer_tree_host(), bitmap_);
+ if (GetLayerTree() && !bitmap_.empty()) {
+ ui_resource_holder_ = ScopedUIResourceHolder::Create(
+ GetLayerTree()->GetUIResourceManager(), bitmap_);
} else {
ui_resource_holder_ = nullptr;
}
@@ -157,10 +161,11 @@ void UIResourceLayer::PushPropertiesTo(LayerImpl* layer) {
if (!ui_resource_holder_) {
layer_impl->SetUIResourceId(0);
} else {
- DCHECK(layer_tree_host());
+ DCHECK(GetLayerTree());
gfx::Size image_size =
- layer_tree_host()->GetUIResourceSize(ui_resource_holder_->id());
+ GetLayerTree()->GetUIResourceManager()->GetUIResourceSize(
+ ui_resource_holder_->id());
layer_impl->SetUIResourceId(ui_resource_holder_->id());
layer_impl->SetImageBounds(image_size);
layer_impl->SetUV(uv_top_left_, uv_bottom_right_);
« no previous file with comments | « cc/layers/scrollbar_layer_unittest.cc ('k') | cc/layers/ui_resource_layer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698