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

Unified Diff: ui/compositor/layer_owner.cc

Issue 2458833003: Revert of [M55] Generalize layer mirroring for phantom windows (Closed)
Patch Set: Created 4 years, 2 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 | « ui/compositor/layer_owner.h ('k') | ui/compositor/layer_owner_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/compositor/layer_owner.cc
diff --git a/ui/compositor/layer_owner.cc b/ui/compositor/layer_owner.cc
index 04a053cbd318a6eb9374101091574bc311111c27..d94ff2cc8239813e500d1296b780ae5cd16cff9d 100644
--- a/ui/compositor/layer_owner.cc
+++ b/ui/compositor/layer_owner.cc
@@ -17,10 +17,10 @@
LayerOwner::~LayerOwner() {
}
-void LayerOwner::SetLayer(std::unique_ptr<Layer> layer) {
+void LayerOwner::SetLayer(Layer* layer) {
DCHECK(!OwnsLayer());
- layer_owner_ = std::move(layer);
- layer_ = layer_owner_.get();
+ layer_owner_.reset(layer);
+ layer_ = layer;
layer_->owner_ = this;
}
@@ -38,16 +38,33 @@
LayerDelegate* old_delegate = old_layer->delegate();
old_layer->set_delegate(NULL);
- SetLayer(old_layer->Clone());
+ const gfx::Rect layer_bounds(old_layer->bounds());
+ Layer* new_layer = new ui::Layer(old_layer->type());
+ SetLayer(new_layer);
+ new_layer->SetVisible(old_layer->GetTargetVisibility());
+ new_layer->SetOpacity(old_layer->GetTargetOpacity());
+ new_layer->SetBounds(layer_bounds);
+ new_layer->SetMasksToBounds(old_layer->GetMasksToBounds());
+ new_layer->set_name(old_layer->name());
+ new_layer->SetFillsBoundsOpaquely(old_layer->fills_bounds_opaquely());
+ new_layer->SetFillsBoundsCompletely(old_layer->FillsBoundsCompletely());
+ new_layer->SetSubpixelPositionOffset(old_layer->subpixel_position_offset());
+ new_layer->SetLayerInverted(old_layer->layer_inverted());
+ new_layer->SetTransform(old_layer->GetTargetTransform());
+ if (old_layer->type() == LAYER_SOLID_COLOR)
+ new_layer->SetColor(old_layer->GetTargetColor());
+ SkRegion* alpha_shape = old_layer->alpha_shape();
+ if (alpha_shape)
+ new_layer->SetAlphaShape(base::MakeUnique<SkRegion>(*alpha_shape));
if (old_layer->parent()) {
// Install new layer as a sibling of the old layer, stacked below it.
- old_layer->parent()->Add(layer_);
- old_layer->parent()->StackBelow(layer_, old_layer.get());
+ old_layer->parent()->Add(new_layer);
+ old_layer->parent()->StackBelow(new_layer, old_layer.get());
} else if (old_layer->GetCompositor()) {
// If old_layer was the layer tree root then we need to move the Compositor
// over to the new root.
- old_layer->GetCompositor()->SetRootLayer(layer_);
+ old_layer->GetCompositor()->SetRootLayer(new_layer);
}
// Migrate all the child layers over to the new layer. Copy the list because
@@ -57,15 +74,15 @@
it != children_copy.end();
++it) {
ui::Layer* child = *it;
- layer_->Add(child);
+ new_layer->Add(child);
}
// Install the delegate last so that the delegate isn't notified as we copy
// state to the new layer.
- layer_->set_delegate(old_delegate);
+ new_layer->set_delegate(old_delegate);
if (layer_owner_delegate_)
- layer_owner_delegate_->OnLayerRecreated(old_layer.get(), layer_);
+ layer_owner_delegate_->OnLayerRecreated(old_layer.get(), new_layer);
return old_layer;
}
« no previous file with comments | « ui/compositor/layer_owner.h ('k') | ui/compositor/layer_owner_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698