Revert of Don't change layout size due to top control show/hide

Revert of Don't change layout size due to top control show/hide (patchset #13 id:240001 of https://codereview.chromium.org/1573283002/ )

Reason for revert:
speculative reverts for https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/51570

see crbug/579666

Original issue's description:
> Don't change layout size due to top control show/hide.
>
> This patch changes Chrome's top control (URL bar) behavior on Android
> to match Safari's in how it affects layout size. To summarize, the
> layout size or, equivalently, the initial containing block (ICB), is no
> longer dynamically adjusted when the top controls are shown/hidden.
> Instead the ICB is statically sized to be the size of the viewport when
> top controls are showing. position: fixed elements are an exception,
> they continue to work as they do today, resizing to mach the full
> viewport size in response to top controls.
>
> Additionally, vh units were changed in a similar way; they are now also
> statically sized but to the size of the viewport with top controls
> hidden.
>
> See https://github.com/bokand/URLBarSizing for full details.
>
> Intent to ship:
> https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/BK0oHURgmJ4
>
> BUG=428132
>
> Committed: https://crrev.com/7ea490e3e6a79cc1e7fd4d952bcc36b2ad38cf40
> Cr-Commit-Position: refs/heads/master@{#370425}

TBR=jbroman@chromium.org,aelias@chromium.org,bokan@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=428132
Committed: https://crrev.com/f656eeb17cf8de94f6c6fc10671f9a06b6c98de4
Cr-Commit-Position: refs/heads/master@{#370486}

[sgurun-gerrit only, 2016-01-20 20:59:05]

Created Revert of Don't change layout size due to top control show/hide

[commit-bot: I haz the power, 2016-01-20 20:59:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1611693002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1611693002/1

[commit-bot: I haz the power, 2016-01-20 21:01:47]

Description was changed from

==========
Revert of Don't change layout size due to top control show/hide (patchset #13
id:240001 of https://codereview.chromium.org/1573283002/ )

Reason for revert:
speculative reverts for
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...

see crbug/579666




Original issue's description:
> Don't change layout size due to top control show/hide.
> 
> This patch changes Chrome's top control (URL bar) behavior on Android
> to match Safari's in how it affects layout size. To summarize, the
> layout size or, equivalently, the initial containing block (ICB), is no
> longer dynamically adjusted when the top controls are shown/hidden.
> Instead the ICB is statically sized to be the size of the viewport when
> top controls are showing. position: fixed elements are an exception,
> they continue to work as they do today, resizing to mach the full
> viewport size in response to top controls.
> 
> Additionally, vh units were changed in a similar way; they are now also
> statically sized but to the size of the viewport with top controls
> hidden.
> 
> See https://github.com/bokand/URLBarSizing for full details.
> 
> Intent to ship:
> https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/BK0oHURgmJ4
> 
> BUG=428132
> 
> Committed: https://crrev.com/7ea490e3e6a79cc1e7fd4d952bcc36b2ad38cf40
> Cr-Commit-Position: refs/heads/master@{#370425}

TBR=jbroman@chromium.org,aelias@chromium.org,bokan@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=428132
==========

to

==========
Revert of Don't change layout size due to top control show/hide (patchset #13
id:240001 of https://codereview.chromium.org/1573283002/ )

Reason for revert:
speculative reverts for
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...

see crbug/579666




Original issue's description:
> Don't change layout size due to top control show/hide.
> 
> This patch changes Chrome's top control (URL bar) behavior on Android
> to match Safari's in how it affects layout size. To summarize, the
> layout size or, equivalently, the initial containing block (ICB), is no
> longer dynamically adjusted when the top controls are shown/hidden.
> Instead the ICB is statically sized to be the size of the viewport when
> top controls are showing. position: fixed elements are an exception,
> they continue to work as they do today, resizing to mach the full
> viewport size in response to top controls.
> 
> Additionally, vh units were changed in a similar way; they are now also
> statically sized but to the size of the viewport with top controls
> hidden.
> 
> See https://github.com/bokand/URLBarSizing for full details.
> 
> Intent to ship:
> https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/BK0oHURgmJ4
> 
> BUG=428132
> 
> Committed: https://crrev.com/7ea490e3e6a79cc1e7fd4d952bcc36b2ad38cf40
> Cr-Commit-Position: refs/heads/master@{#370425}

TBR=jbroman@chromium.org,aelias@chromium.org,bokan@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=428132
==========

[commit-bot: I haz the power, 2016-01-20 21:01:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-20 21:02:57]

Description was changed from

==========
Revert of Don't change layout size due to top control show/hide (patchset #13
id:240001 of https://codereview.chromium.org/1573283002/ )

Reason for revert:
speculative reverts for
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...

see crbug/579666




Original issue's description:
> Don't change layout size due to top control show/hide.
> 
> This patch changes Chrome's top control (URL bar) behavior on Android
> to match Safari's in how it affects layout size. To summarize, the
> layout size or, equivalently, the initial containing block (ICB), is no
> longer dynamically adjusted when the top controls are shown/hidden.
> Instead the ICB is statically sized to be the size of the viewport when
> top controls are showing. position: fixed elements are an exception,
> they continue to work as they do today, resizing to mach the full
> viewport size in response to top controls.
> 
> Additionally, vh units were changed in a similar way; they are now also
> statically sized but to the size of the viewport with top controls
> hidden.
> 
> See https://github.com/bokand/URLBarSizing for full details.
> 
> Intent to ship:
> https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/BK0oHURgmJ4
> 
> BUG=428132
> 
> Committed: https://crrev.com/7ea490e3e6a79cc1e7fd4d952bcc36b2ad38cf40
> Cr-Commit-Position: refs/heads/master@{#370425}

TBR=jbroman@chromium.org,aelias@chromium.org,bokan@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=428132
==========

to

==========
Revert of Don't change layout size due to top control show/hide (patchset #13
id:240001 of https://codereview.chromium.org/1573283002/ )

Reason for revert:
speculative reverts for
https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%2...

see crbug/579666

Original issue's description:
> Don't change layout size due to top control show/hide.
>
> This patch changes Chrome's top control (URL bar) behavior on Android
> to match Safari's in how it affects layout size. To summarize, the
> layout size or, equivalently, the initial containing block (ICB), is no
> longer dynamically adjusted when the top controls are shown/hidden.
> Instead the ICB is statically sized to be the size of the viewport when
> top controls are showing. position: fixed elements are an exception,
> they continue to work as they do today, resizing to mach the full
> viewport size in response to top controls.
>
> Additionally, vh units were changed in a similar way; they are now also
> statically sized but to the size of the viewport with top controls
> hidden.
>
> See https://github.com/bokand/URLBarSizing for full details.
>
> Intent to ship:
> https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/BK0oHURgmJ4
>
> BUG=428132
>
> Committed: https://crrev.com/7ea490e3e6a79cc1e7fd4d952bcc36b2ad38cf40
> Cr-Commit-Position: refs/heads/master@{#370425}

TBR=jbroman@chromium.org,aelias@chromium.org,bokan@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=428132

Committed: https://crrev.com/f656eeb17cf8de94f6c6fc10671f9a06b6c98de4
Cr-Commit-Position: refs/heads/master@{#370486}
==========

[commit-bot: I haz the power, 2016-01-20 21:03:00]

Patchset 1 (id:??) landed as
https://crrev.com/f656eeb17cf8de94f6c6fc10671f9a06b6c98de4
Cr-Commit-Position: refs/heads/master@{#370486}

[sgurun-gerrit only, 2016-01-20 22:52:12]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1612683002/ by sgurun@chromium.org.

The reason for reverting is: does not look like related to crbug/57966, probably
the test is flaky

relanding.