[webkit-reviews] review requested: [Bug 97801] Support invalidation tracking for composited layers : [Attachment 171917] Patch

[bugzilla-daemon at webkit.org]

vollick at chromium.org has asked	for review:
Bug 97801: Support invalidation tracking for composited layers
https://bugs.webkit.org/show_bug.cgi?id=97801

Attachment 171917: Patch
https://bugs.webkit.org/attachment.cgi?id=171917&action=review

------- Additional Comments from vollick at chromium.org
(In reply to comment #42)
> (From update of attachment 171916 [details])
> View in context:
https://bugs.webkit.org/attachment.cgi?id=171916&action=review
>
> > Source/WebCore/page/FrameView.cpp:3650
> > +static void setTracksRepaintsRecursive(Frame* frame, bool trackRepaints)
> > +{
> > +	 if (!frame)
> > +	     return;
> > +
> > +	 if (RenderView* root = frame->contentRenderer())
> > +	     root->compositor()->setTracksRepaints(trackRepaints);
> > +
> > +	 for (unsigned i = 0; i < frame->tree()->childCount(); ++i)
> > +	     setTracksRepaintsRecursive(frame->tree()->child(i),
trackRepaints);
> > +}
>
> You can do this non-recursively; the FrameTree can enumerate all subframes
for you.

Done.