Chromium Code Reviews| Index: aura/window.cc |
| =================================================================== |
| --- aura/window.cc (revision 98234) |
| +++ aura/window.cc (working copy) |
| @@ -9,17 +9,15 @@ |
| #include "aura/desktop.h" |
| #include "aura/window_delegate.h" |
| #include "base/logging.h" |
| -#include "third_party/skia/include/core/SkCanvas.h" |
| +#include "ui/gfx/canvas_skia.h" |
| #include "ui/gfx/compositor/compositor.h" |
| #include "ui/gfx/compositor/layer.h" |
| namespace aura { |
| -// TODO: do we need to support child windows? |
| -Window::Window(Desktop* desktop) |
| - : delegate_(NULL), |
| +Window::Window(WindowDelegate* delegate) |
| + : delegate_(delegate), |
| visibility_(VISIBILITY_HIDDEN), |
| - layer_(new ui::Layer(desktop->compositor())), |
| needs_paint_all_(true), |
| parent_(NULL), |
| id_(-1) { |
| @@ -28,6 +26,10 @@ |
| Window::~Window() { |
| } |
| +void Window::Init() { |
| + layer_.reset(new ui::Layer(Desktop::GetInstance()->compositor())); |
| +} |
| + |
| void Window::SetVisibility(Visibility visibility) { |
| if (visibility_ == visibility) |
| return; |
| @@ -51,12 +53,19 @@ |
| void Window::SetCanvas(const SkCanvas& canvas, const gfx::Point& origin) { |
| // TODO: figure out how this is going to work when animating the layer. In |
| - // particular if we're animating the size then the underyling Texture is going |
| + // particular if we're animating the size then the underlying Texture is going |
| // to be unhappy if we try to set a texture on a size bigger than the size of |
| // the texture. |
| layer_->SetCanvas(canvas, origin); |
| } |
| +void Window::SetParent(Window* parent) { |
| + if (parent) |
|
sky
2011/08/25 20:48:46
Should this change the parent of the layer too?
Ben Goodger (Google)
2011/08/25 21:08:00
Already done by AddChild()
|
| + parent->AddChild(this); |
| + else |
| + Desktop::GetInstance()->window()->AddChild(this); |
| +} |
| + |
| void Window::DrawTree() { |
| UpdateLayerCanvas(); |
| Draw(); |
| @@ -96,8 +105,12 @@ |
| dirty_rect_.SetRect(0, 0, 0, 0); |
| if (dirty_rect.IsEmpty()) |
| return; |
| - if (delegate_) |
| - delegate_->OnPaint(dirty_rect); |
| + if (delegate_) { |
| + scoped_ptr<gfx::Canvas> canvas(gfx::Canvas::CreateCanvas( |
| + dirty_rect.width(), dirty_rect.height(), false)); |
| + delegate_->OnPaint(canvas.get()); |
|
sky
2011/08/25 20:48:46
Doesn't this need to translate too? See canvas_pai
|
| + SetCanvas(*canvas->AsCanvasSkia(), bounds().origin()); |
| + } |
| } |
| void Window::Draw() { |