Make PicturePile pile.

cc/picture_pile.cc

 // Copyright 2012 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 <algorithm>
 
 #include "cc/picture_pile.h"
 #include "cc/picture_pile_impl.h"
 
+namespace {
+const float kResetThreshold = 0.7f;

[enne (OOO), 2012/12/04 04:11:09]

Can you add a comment for what this constant is for?

+}
+
 namespace cc {
 
 PicturePile::PicturePile() {
 }
 
 PicturePile::~PicturePile() {
 }
 
-class OutOfBoundsPredicate {
-public:
-  OutOfBoundsPredicate(gfx::Size size) : layer_rect_(gfx::Point(), size) { }
-  bool operator()(const scoped_refptr<Picture>& picture) {
-    return !picture->LayerRect().Intersects(layer_rect_);
-  }
-  gfx::Rect layer_rect_;
-};
+void PicturePile::Resize(gfx::Size size) {
+  if (size_ == size)
+    return;
 
-void PicturePile::Resize(gfx::Size size) {
-  // Remove pictures that aren't in bounds anymore.
-  if (size.width() < size_.width() || size.height() < size_.height()) {
-    OutOfBoundsPredicate oob(size);
-    pile_.erase(std::remove_if(pile_.begin(), pile_.end(), oob), pile_.end());
-  }
-
+  pile_.clear();

[enne (OOO), 2012/12/04 04:11:09]

Ah, quite right. I guess if you're resizing and have to rerecord, then you might
as well just make a new pile.

   size_ = size;
 }
 
 void PicturePile::Update(
     ContentLayerClient* painter,
-    const Region&,
+    const Region& invalidation,
     RenderingStats& stats) {
-  // TODO(enne): Add things to the pile, consolidate if needed, etc...
-  // TODO(enne): Only re-record invalidated areas.
-  // TODO(enne): Also re-record areas that have been newly exposed by resize.
+  if (pile_.empty()) {
+    ResetPile(painter, stats);
+    return;
+  }
 
-  // Always re-record the entire layer into a single picture, just to get
-  // this class up and running.
+  for (Region::Iterator i(invalidation); i.has_rect(); i.next())
+    InvalidateRect(i.rect());
+
+  for (Pile::iterator i = pile_.begin(); i != pile_.end(); ++i) {
+    if (!(*i)->HasRecording())
+      (*i)->Record(painter, stats);
+  }
+}
+
+class FullyContainedPredicate {
+public:
+  FullyContainedPredicate(gfx::Rect rect) : layer_rect_(rect) { }
+  bool operator()(const scoped_refptr<Picture>& picture) {
+    return layer_rect_.Contains(picture->LayerRect());
+  }
+  gfx::Rect layer_rect_;
+};
+
+void PicturePile::InvalidateRect(gfx::Rect invalidation) {
+  if (invalidation.IsEmpty())
+    return;
+
+  std::vector<Pile::iterator> overlaps;
+  for (Pile::iterator i = pile_.begin(); i != pile_.end(); ++i) {
+    if ((*i)->LayerRect().Contains(invalidation) && !(*i)->HasRecording())
+      return;
+    if ((*i)->LayerRect().Intersects(invalidation) && i != pile_.begin())
+      overlaps.push_back(i);
+  }
+
+  gfx::Rect picture_rect = invalidation;
+  for (int j = 0; j < overlaps.size(); j++)
+    picture_rect = gfx::UnionRects(picture_rect, (*overlaps[j])->LayerRect());
+  if (picture_rect.size().GetArea() / static_cast<float>(size_.GetArea())
+      > kResetThreshold)

[enne (OOO), 2012/12/04 04:11:09]

style nit: I'm pretty sure operators go on the previous line in Chromium.

+    picture_rect = gfx::Rect(size_);
+
+  FullyContainedPredicate pred(picture_rect);
+  pile_.erase(std::remove_if(pile_.begin(), pile_.end(), pred), pile_.end());
+
+  pile_.push_back(Picture::Create(picture_rect));
+}
+
+
+void PicturePile::ResetPile(ContentLayerClient* painter,
+                            RenderingStats& stats) {
   pile_.clear();
-  pile_.push_back(Picture::Create());
-  pile_[0]->Record(painter, gfx::Rect(gfx::Point(), size_), stats);
+
+  scoped_refptr<Picture> base_picture = Picture::Create(gfx::Rect(size_));
+  base_picture->Record(painter, stats);
+  pile_.push_back(base_picture);
 }
 
 void PicturePile::PushPropertiesTo(PicturePileImpl* other) {
-  other->pile_.resize(pile_.size());
-  for (size_t i = 0; i < pile_.size(); ++i)
-    other->pile_[i] = pile_[i];
+  other->pile_ = pile_;
 }
 
 }  // namespace cc