[webkit-reviews] review granted: [Bug 130627] Don't flush compositing layers within FrameView::willPaintContents() : [Attachment 227527] Patch

[bugzilla-daemon at webkit.org]

Sam Weinig <sam at webkit.org> has granted Simon Fraser (smfr)
<simon.fraser at apple.com>'s request for review:
Bug 130627: Don't flush compositing layers within
FrameView::willPaintContents()
https://bugs.webkit.org/show_bug.cgi?id=130627

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

------- Additional Comments from Sam Weinig <sam at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=227527&action=review


> Source/WebKit/mac/ChangeLog:26
> +2014-03-21  Simon Fraser  <simon.fraser at apple.com>
> +
> +	   Don't flush compositing layers within FrameView::willPaintContents()

> +	   https://bugs.webkit.org/show_bug.cgi?id=130627
> +
> +	   Reviewed by NOBODY (OOPS!).
> +	   
> +	   Make _flushCompositingChanges private so DRT can call it;
> +	   moved it into the Private category.
> +
> +	   * WebView/WebView.mm:
> +	   (-[WebView _flushCompositingChanges]):
> +	   * WebView/WebViewPrivate.h:
> +
> +2014-03-21  Simon Fraser  <simon.fraser at apple.com>
> +
> +	   Remove duplicate line added during merging
> +	   https://bugs.webkit.org/show_bug.cgi?id=130623
> +
> +	   Reviewed by Beth Dakin.
> +	   
> +	   Remove a duplicate call to -_viewWillDrawInternal which
> +	   was added during merges.
> +
> +	   * WebView/WebView.mm:
> +	   (LayerFlushController::flushLayers):

Double ChangeLog!