Revert of Reland "Implement FullScreen using top layer."

Revert of Reland "Implement FullScreen using top layer." (patchset #13 id:260001 of https://codereview.chromium.org/2107233002/ )

Reason for revert:
Multiple regressions around the fullscreen transition becoming visibly
worse, as well as a suspected leak.

BUG=628554, 628605, 628662, 630191
Original issue's description:
> Reland "Implement FullScreen using top layer."
>
> This reverts commit 2460a95281959d1b4dd941fed10998adf8dcb63c.
>
> Notable changes on top of previous attempt:
>
> full-screen-iframe-ua-style.html is changed to use overflow property
> instead, as position is no longer changed on ancestors.
>
> :root:-webkit-full-screen-ancestor { overflow: hidden !important; } is
> retained, as the previous attempt regressed on Vimeo (crbug.com/543946)
> and it's not clear yet how to handle this in the spec:
> https://github.com/whatwg/fullscreen/issues/19
>
> Make ::backdrop work for replaced content (video, iframe, etc.) by
> changing StyleResolver::createPseudoElementIfNeeded to allow creating
> the ::backdrop pseudo-element even if the parent doesn't support
> children. With this change, full-screen-stacking-context.html was
> simplified and full-screen-video-has-backdrop.html was
> simplified+renamed.
>
> The will-change override is dropped, as full-screen-will-change.html
> passes without it.
>
> BUG=240576, 543946
>
> Committed: https://crrev.com/d666f503ae854fac360cc70da1f5971a6724546a
> Cr-Commit-Position: refs/heads/master@{#405478}

TBR=dsinclair@chromium.org,esprehn@chromium.org,eae@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=240576, 543946
Committed: https://crrev.com/87365b3f6c6293bac8a2513cef257d8e2cb0898a
Cr-Commit-Position: refs/heads/master@{#407089}

[foolip, 2016-07-22 03:35:13]

The CQ bit was checked by foolip@chromium.org

[foolip, 2016-07-22 03:35:14]

Created Revert of Reland "Implement FullScreen using top layer."

[foolip, 2016-07-22 03:36:07]

Description was changed from

==========
Revert of Reland "Implement FullScreen using top layer." (patchset #13 id:260001
of https://codereview.chromium.org/2107233002/ )

Reason for revert:
Multiple regressions around the fullscreen transition becoming visibly worse, as
well as a suspected leak.

BUG=628554,628605,628662,630191

Original issue's description:
> Reland "Implement FullScreen using top layer."
> 
> This reverts commit 2460a95281959d1b4dd941fed10998adf8dcb63c.
> 
> Notable changes on top of previous attempt:
> 
> full-screen-iframe-ua-style.html is changed to use overflow property
> instead, as position is no longer changed on ancestors.
> 
> :root:-webkit-full-screen-ancestor { overflow: hidden !important; } is
> retained, as the previous attempt regressed on Vimeo (crbug.com/543946)
> and it's not clear yet how to handle this in the spec:
> https://github.com/whatwg/fullscreen/issues/19
> 
> Make ::backdrop work for replaced content (video, iframe, etc.) by
> changing StyleResolver::createPseudoElementIfNeeded to allow creating
> the ::backdrop pseudo-element even if the parent doesn't support
> children. With this change, full-screen-stacking-context.html was
> simplified and full-screen-video-has-backdrop.html was
> simplified+renamed.
> 
> The will-change override is dropped, as full-screen-will-change.html
> passes without it.
> 
> BUG=240576, 543946
> 
> Committed: https://crrev.com/d666f503ae854fac360cc70da1f5971a6724546a
> Cr-Commit-Position: refs/heads/master@{#405478}

TBR=dsinclair@chromium.org,esprehn@chromium.org,eae@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=240576, 543946
==========

to

==========
Revert of Reland "Implement FullScreen using top layer." (patchset #13 id:260001
of https://codereview.chromium.org/2107233002/ )

Reason for revert:
Multiple regressions around the fullscreen transition becoming visibly
worse, as well as a suspected leak.

BUG=628554,628605,628662,630191

Original issue's description:
> Reland "Implement FullScreen using top layer."
> 
> This reverts commit 2460a95281959d1b4dd941fed10998adf8dcb63c.
> 
> Notable changes on top of previous attempt:
> 
> full-screen-iframe-ua-style.html is changed to use overflow property
> instead, as position is no longer changed on ancestors.
> 
> :root:-webkit-full-screen-ancestor { overflow: hidden !important; } is
> retained, as the previous attempt regressed on Vimeo (crbug.com/543946)
> and it's not clear yet how to handle this in the spec:
> https://github.com/whatwg/fullscreen/issues/19
> 
> Make ::backdrop work for replaced content (video, iframe, etc.) by
> changing StyleResolver::createPseudoElementIfNeeded to allow creating
> the ::backdrop pseudo-element even if the parent doesn't support
> children. With this change, full-screen-stacking-context.html was
> simplified and full-screen-video-has-backdrop.html was
> simplified+renamed.
> 
> The will-change override is dropped, as full-screen-will-change.html
> passes without it.
> 
> BUG=240576, 543946
> 
> Committed: https://crrev.com/d666f503ae854fac360cc70da1f5971a6724546a
> Cr-Commit-Position: refs/heads/master@{#405478}

TBR=dsinclair@chromium.org,esprehn@chromium.org,eae@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=240576, 543946
==========

[commit-bot: I haz the power, 2016-07-22 03:36:12]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-22 03:39:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-22 03:39:24]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[foolip, 2016-07-22 04:04:38]

The CQ bit was checked by foolip@chromium.org

[commit-bot: I haz the power, 2016-07-22 04:04:55]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[esprehn, 2016-07-22 05:53:26]

lgtm

[commit-bot: I haz the power, 2016-07-22 06:55:20]

Description was changed from

==========
Revert of Reland "Implement FullScreen using top layer." (patchset #13 id:260001
of https://codereview.chromium.org/2107233002/ )

Reason for revert:
Multiple regressions around the fullscreen transition becoming visibly
worse, as well as a suspected leak.

BUG=628554,628605,628662,630191

Original issue's description:
> Reland "Implement FullScreen using top layer."
> 
> This reverts commit 2460a95281959d1b4dd941fed10998adf8dcb63c.
> 
> Notable changes on top of previous attempt:
> 
> full-screen-iframe-ua-style.html is changed to use overflow property
> instead, as position is no longer changed on ancestors.
> 
> :root:-webkit-full-screen-ancestor { overflow: hidden !important; } is
> retained, as the previous attempt regressed on Vimeo (crbug.com/543946)
> and it's not clear yet how to handle this in the spec:
> https://github.com/whatwg/fullscreen/issues/19
> 
> Make ::backdrop work for replaced content (video, iframe, etc.) by
> changing StyleResolver::createPseudoElementIfNeeded to allow creating
> the ::backdrop pseudo-element even if the parent doesn't support
> children. With this change, full-screen-stacking-context.html was
> simplified and full-screen-video-has-backdrop.html was
> simplified+renamed.
> 
> The will-change override is dropped, as full-screen-will-change.html
> passes without it.
> 
> BUG=240576, 543946
> 
> Committed: https://crrev.com/d666f503ae854fac360cc70da1f5971a6724546a
> Cr-Commit-Position: refs/heads/master@{#405478}

TBR=dsinclair@chromium.org,esprehn@chromium.org,eae@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=240576, 543946
==========

to

==========
Revert of Reland "Implement FullScreen using top layer." (patchset #13 id:260001
of https://codereview.chromium.org/2107233002/ )

Reason for revert:
Multiple regressions around the fullscreen transition becoming visibly
worse, as well as a suspected leak.

BUG=628554,628605,628662,630191

Original issue's description:
> Reland "Implement FullScreen using top layer."
> 
> This reverts commit 2460a95281959d1b4dd941fed10998adf8dcb63c.
> 
> Notable changes on top of previous attempt:
> 
> full-screen-iframe-ua-style.html is changed to use overflow property
> instead, as position is no longer changed on ancestors.
> 
> :root:-webkit-full-screen-ancestor { overflow: hidden !important; } is
> retained, as the previous attempt regressed on Vimeo (crbug.com/543946)
> and it's not clear yet how to handle this in the spec:
> https://github.com/whatwg/fullscreen/issues/19
> 
> Make ::backdrop work for replaced content (video, iframe, etc.) by
> changing StyleResolver::createPseudoElementIfNeeded to allow creating
> the ::backdrop pseudo-element even if the parent doesn't support
> children. With this change, full-screen-stacking-context.html was
> simplified and full-screen-video-has-backdrop.html was
> simplified+renamed.
> 
> The will-change override is dropped, as full-screen-will-change.html
> passes without it.
> 
> BUG=240576, 543946
> 
> Committed: https://crrev.com/d666f503ae854fac360cc70da1f5971a6724546a
> Cr-Commit-Position: refs/heads/master@{#405478}

TBR=dsinclair@chromium.org,esprehn@chromium.org,eae@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=240576, 543946
==========

[commit-bot: I haz the power, 2016-07-22 06:55:21]

Committed patchset #2 (id:300001)

[commit-bot: I haz the power, 2016-07-22 06:57:05]

Description was changed from

==========
Revert of Reland "Implement FullScreen using top layer." (patchset #13 id:260001
of https://codereview.chromium.org/2107233002/ )

Reason for revert:
Multiple regressions around the fullscreen transition becoming visibly
worse, as well as a suspected leak.

BUG=628554,628605,628662,630191

Original issue's description:
> Reland "Implement FullScreen using top layer."
> 
> This reverts commit 2460a95281959d1b4dd941fed10998adf8dcb63c.
> 
> Notable changes on top of previous attempt:
> 
> full-screen-iframe-ua-style.html is changed to use overflow property
> instead, as position is no longer changed on ancestors.
> 
> :root:-webkit-full-screen-ancestor { overflow: hidden !important; } is
> retained, as the previous attempt regressed on Vimeo (crbug.com/543946)
> and it's not clear yet how to handle this in the spec:
> https://github.com/whatwg/fullscreen/issues/19
> 
> Make ::backdrop work for replaced content (video, iframe, etc.) by
> changing StyleResolver::createPseudoElementIfNeeded to allow creating
> the ::backdrop pseudo-element even if the parent doesn't support
> children. With this change, full-screen-stacking-context.html was
> simplified and full-screen-video-has-backdrop.html was
> simplified+renamed.
> 
> The will-change override is dropped, as full-screen-will-change.html
> passes without it.
> 
> BUG=240576, 543946
> 
> Committed: https://crrev.com/d666f503ae854fac360cc70da1f5971a6724546a
> Cr-Commit-Position: refs/heads/master@{#405478}

TBR=dsinclair@chromium.org,esprehn@chromium.org,eae@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=240576, 543946
==========

to

==========
Revert of Reland "Implement FullScreen using top layer." (patchset #13 id:260001
of https://codereview.chromium.org/2107233002/ )

Reason for revert:
Multiple regressions around the fullscreen transition becoming visibly
worse, as well as a suspected leak.

BUG=628554,628605,628662,630191

Original issue's description:
> Reland "Implement FullScreen using top layer."
>
> This reverts commit 2460a95281959d1b4dd941fed10998adf8dcb63c.
>
> Notable changes on top of previous attempt:
>
> full-screen-iframe-ua-style.html is changed to use overflow property
> instead, as position is no longer changed on ancestors.
>
> :root:-webkit-full-screen-ancestor { overflow: hidden !important; } is
> retained, as the previous attempt regressed on Vimeo (crbug.com/543946)
> and it's not clear yet how to handle this in the spec:
> https://github.com/whatwg/fullscreen/issues/19
>
> Make ::backdrop work for replaced content (video, iframe, etc.) by
> changing StyleResolver::createPseudoElementIfNeeded to allow creating
> the ::backdrop pseudo-element even if the parent doesn't support
> children. With this change, full-screen-stacking-context.html was
> simplified and full-screen-video-has-backdrop.html was
> simplified+renamed.
>
> The will-change override is dropped, as full-screen-will-change.html
> passes without it.
>
> BUG=240576, 543946
>
> Committed: https://crrev.com/d666f503ae854fac360cc70da1f5971a6724546a
> Cr-Commit-Position: refs/heads/master@{#405478}

TBR=dsinclair@chromium.org,esprehn@chromium.org,eae@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=240576, 543946

Committed: https://crrev.com/87365b3f6c6293bac8a2513cef257d8e2cb0898a
Cr-Commit-Position: refs/heads/master@{#407089}
==========

[commit-bot: I haz the power, 2016-07-22 06:57:07]

Patchset 2 (id:??) landed as
https://crrev.com/87365b3f6c6293bac8a2513cef257d8e2cb0898a
Cr-Commit-Position: refs/heads/master@{#407089}

[eae, 2016-07-22 17:34:31]

LGTM :(