Add a layout test for iframe flicker due to overflow scroll promotion

Add a layout test for iframe flicker due to overflow scroll promotion

This test checks that the composited layer tree is updated immediately
after the inner iframe becomes composited due to an accelerated
overflow scrolling layer.

I have confirmed that this test fails without Ojan's change to give
all RenderParts render layers (this CL marks composited iframes as self
painting immediately rather than after a style update allowing the
GraphicsLayerUpdater to stitch in the subframe's graphics layer tree
right away) -- https://codereview.chromium.org/302083002

R=ojan@chromium.org
BUG=379097
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175817

[ojan, 2014-06-09 17:42:38]

lgtm

Please fix the rAF thing before committing.

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
File LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html
(right):

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html:2:
<iframe id="frame" srcdoc="<style>input { width: 100px; }</style><input
type='text' id='input' value='add stuff here'></input>"></iframe>
Nit: type=text is the default.

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html:2:
<iframe id="frame" srcdoc="<style>input { width: 100px; }</style><input
type='text' id='input' value='add stuff here'></input>"></iframe>
Nit: type=text is the default.

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html:5:
testRunner.dumpAsText();
Nit: 4 space indent?

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html:24:
requestAnimationFrame(function() {
I don't think we need the requestAnimationFrame since you read both layer trees
before. Similarly, this test doesn't need to be async.

[Ian Vollick, 2014-06-09 18:35:57]

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
File LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html
(right):

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html:2:
<iframe id="frame" srcdoc="<style>input { width: 100px; }</style><input
type='text' id='input' value='add stuff here'></input>"></iframe>
On 2014/06/09 17:42:38, ojan wrote:
> Nit: type=text is the default.

Done.

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html:5:
testRunner.dumpAsText();
On 2014/06/09 17:42:38, ojan wrote:
> Nit: 4 space indent?

Done.

https://codereview.chromium.org/324733002/diff/20001/LayoutTests/compositing/...
LayoutTests/compositing/iframes/composited-iframe-transition-flicker.html:24:
requestAnimationFrame(function() {
On 2014/06/09 17:42:37, ojan wrote:
> I don't think we need the requestAnimationFrame since you read both layer
trees
> before. Similarly, this test doesn't need to be async.

Done.

[Ian Vollick, 2014-06-09 18:36:02]

The CQ bit was checked by vollick@chromium.org

[commit-bot: I haz the power, 2014-06-09 18:36:24]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vollick@chromium.org/324733002/40001

[Ian Vollick, 2014-06-09 18:36:28]

The CQ bit was unchecked by vollick@chromium.org

[Ian Vollick, 2014-06-09 18:40:25]

The CQ bit was checked by vollick@chromium.org

[commit-bot: I haz the power, 2014-06-09 18:40:43]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vollick@chromium.org/324733002/60001

[commit-bot: I haz the power, 2014-06-09 19:44:30]

Change committed as 175817