Animated Rotation

views/view.h

Index: views/view.h
diff --git a/views/view.h b/views/view.h
index c188531537a91bf0919c72854bf614d7a504dafb..18deaf06f8cbacf0eb9a62c2c344ce11ed10730c 100644
--- a/views/view.h
+++ b/views/view.h
@@ -1104,6 +1104,7 @@ class VIEWS_EXPORT View : public AcceleratorTarget {
   friend class FocusManager;
   friend class ViewStorage;
   friend class Widget;
+  friend class PaintLock;
 
   // Used to track a drag. RootView passes this into
   // ProcessMousePressed/Dragged.
@@ -1229,6 +1230,9 @@ class VIEWS_EXPORT View : public AcceleratorTarget {
 
   // Accelerated painting ------------------------------------------------------
 
+  // Disables painting during time-critical operations. Used by PaintLock.
+  void set_painting_enabled(bool enabled) { painting_enabled_ = enabled; }

[Ben Goodger (Google), 2011/08/31 20:34:00]

It troubles me that we need this.

It seems like the widget should just not dispatch paints into the hierarchy when
a time critical operation is underway. The widget can know this because it owns
(or has a link to) the compositor.

Can you add a TODO() to revisit this.

+
   // Returns true if this view should paint to layer.
   bool ShouldPaintToLayer() const;
 
@@ -1353,6 +1357,9 @@ class VIEWS_EXPORT View : public AcceleratorTarget {
   // Whether this view is enabled.
   bool enabled_;
 
+  // Whether this view is painting.
+  bool painting_enabled_;
+
   // Whether or not RegisterViewForVisibleBoundsNotification on the RootView
   // has been invoked.
   bool registered_for_visible_bounds_notification_;