Stop painting when receiving delegated frame

ui/aura/window.cc

Index: ui/aura/window.cc
diff --git a/ui/aura/window.cc b/ui/aura/window.cc
index 992642ed059430e692535edddb94498b63cf7eed..4eb30f2e4b18f144c1f40bd3c7cf3647462d955e 100644
--- a/ui/aura/window.cc
+++ b/ui/aura/window.cc
@@ -497,6 +497,20 @@ void Window::SchedulePaintInRect(const gfx::Rect& rect) {
   }
 }
 
+void Window::NotifyDamage(const gfx::Rect& rect) {

[enne (OOO), 2014/08/11 23:43:23]

This code is an unfortunate duplicate of Window::SchedulePaintInRect, but I'm
not sure how to encapsulate this logic any better.

[sky, 2014/08/13 20:01:01]

Is this a notification that a paint has been scheduled, or that a layer painted?
My initial reaction is that we shouldn't have this and instead the video
detector should be listening to the compositor. But it's entirely likely I'm
missing something.

[enne (OOO), 2014/08/13 23:55:45]

This is a notification that part of the screen has changed (or will, once the
delegated frame has been drawn).  It's more in the sense of "a layer painted"
than "a paint has been scheduled", but I'm not sure I'd use the word "paint" to
describe it.

I'm not sure what you mean by the detector listening to the compositor, as that
seems to me to be what it's doing.  The NotifyDamage call in DelegatedFrameHost
is essentially plumbing through the damage from the delegated frame to the
window to the detector.  How would you suggest doing this more simply?

+  if (!layer() && parent_) {
+    gfx::Rect parent_rect(bounds().size());
+    parent_rect.Intersect(rect);
+    if (!parent_rect.IsEmpty()) {
+      parent_rect.Offset(bounds().origin().OffsetFromOrigin());
+      parent_->NotifyDamage(parent_rect);
+    }
+  } else if (layer()) {
+    FOR_EACH_OBSERVER(
+        WindowObserver, observers_, OnWindowPaintScheduled(this, rect));
+  }
+}
+
 void Window::StackChildAtTop(Window* child) {
   if (children_.size() <= 1 || child == children_.back())
     return;  // In the front already.