GitLab

MoveContentBelowHeader() was not working correctly with the UI refresh
flag enabled because it only knew about the height delta, while we
actually needed to inset based on the expanded height.  This CL updates
the broadcaster API to send the collapsed and expanded toolbar heights
so that this information is available for these content adjustments.

This CL also updates CRWWebViewContentView to check against the old
value of self.contentInset rather than the |_contentInset| ivar.  This
fixes the bug where navigating back from a pdf wouldn't correctly
reset WKScrollView.contentInset since the value of |_contentInset| had
not changed from the previous value.

Bug: 849132, 851129
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I1e46e4fc69fc0ac7a7d114b7c463e1f6a90964f9
Reviewed-on: https://chromium-review.googlesource.com/1094480
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569304}