[webkit-reviews] review granted: [Bug 32447] Content in <iframe> doesn't appear when going back to http://webkit.org/blog/386/3d-transforms/ : [Attachment 71507] Patch

[bugzilla-daemon at webkit.org]

Adam Roben (aroben) <aroben at apple.com> has granted Chris Marrin
<cmarrin at apple.com>'s request for review:
Bug 32447: Content in <iframe> doesn't appear when going back to
http://webkit.org/blog/386/3d-transforms/
https://bugs.webkit.org/show_bug.cgi?id=32447

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

------- Additional Comments from Adam Roben (aroben) <aroben at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=71507&action=review

>
WebCore/manual-tests/compositing/show-composited-iframe-on-back-button.html:10
> +    <!--
> +	   This content was originally intended as a layout test, but the
sequencing performed by 
> +	   queueBackNavigation/queueForwardNavigation differs somehow from what
occurs when hitting 
> +	   the back and forward buttons, so the error does not occur. I've left
all the LayoutTestController 
> +	   logic in the test in case this is fixed in the future.
> +    -->

I think you should add a comment about DRT's back/forward cache being disabled.
Maybe these days we have a way to enable it for certain tests?