ui: Pass the correct container size to DelegatedFrameHostAndroid.

ui: Pass the correct container size to DelegatedFrameHostAndroid.

Currently we pass the VisibleViewportSize from
RenderWidgetHostViewAndroid to the DelegatedFrameHostAndroid.
ContentViewCore eliminates the size of the top controls from this size.

So use GetViewportSizeDip instead. Also, the surface size for the
SurfaceLayer is the size of the viewport itself, so it can directly be
compared with the container size with no adjustments for the top
controls.

Committed: https://crrev.com/4c584d37669a3b2a4e0b80130cb61bad93b0e32e
Cr-Commit-Position: refs/heads/master@{#413639}

[Khushal, 2016-08-17 23:02:49]

khushalsagar@chromium.org changed reviewers:
+ dtrainor@chromium.org, sievers@chromium.org

[David Trainor- moved to gerrit, 2016-08-18 16:42:30]

lgtm

[Khushal, 2016-08-18 17:49:04]

On 2016/08/18 16:42:30, David Trainor wrote:
> lgtm

Thanks!
ping to sievers@

[Khushal, 2016-08-22 17:11:56]

On 2016/08/18 17:49:04, Khushal wrote:
> On 2016/08/18 16:42:30, David Trainor wrote:
> > lgtm
> 
> Thanks!
> ping to sievers@

ping! :)

[no sievers, 2016-08-22 21:47:46]

lgtm, thanks!

https://codereview.chromium.org/2253313002/diff/1/ui/android/delegated_frame_...
File ui/android/delegated_frame_host_android.cc (right):

https://codereview.chromium.org/2253313002/diff/1/ui/android/delegated_frame_...
ui/android/delegated_frame_host_android.cc:113: gfx::Size surface_size =
root_pass->output_rect.size();
Ok, I didn't expect this to include the top controls offset either.

[Khushal, 2016-08-22 21:52:01]

https://codereview.chromium.org/2253313002/diff/1/ui/android/delegated_frame_...
File ui/android/delegated_frame_host_android.cc (right):

https://codereview.chromium.org/2253313002/diff/1/ui/android/delegated_frame_...
ui/android/delegated_frame_host_android.cc:113: gfx::Size surface_size =
root_pass->output_rect.size();
On 2016/08/22 21:47:46, sievers wrote:
> Ok, I didn't expect this to include the top controls offset either.

Yeah, Me neither...

[Khushal, 2016-08-23 01:39:06]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2016-08-23 01:39:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org, sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/2253313002/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2016-08-23 01:39:34]

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-08-23 02:33:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-23 02:35:13]

Description was changed from

==========
ui: Pass the correct container size to DelegatedFrameHostAndroid.

Currently we pass the VisibleViewportSize from
RenderWidgetHostViewAndroid to the DelegatedFrameHostAndroid.
ContentViewCore eliminates the size of the top controls from this size.

So use GetViewportSizeDip instead. Also, the surface size for the
SurfaceLayer is the size of the viewport itself, so it can directly be
compared with the container size with no adjustments for the top
controls.
==========

to

==========
ui: Pass the correct container size to DelegatedFrameHostAndroid.

Currently we pass the VisibleViewportSize from
RenderWidgetHostViewAndroid to the DelegatedFrameHostAndroid.
ContentViewCore eliminates the size of the top controls from this size.

So use GetViewportSizeDip instead. Also, the surface size for the
SurfaceLayer is the size of the viewport itself, so it can directly be
compared with the container size with no adjustments for the top
controls.

Committed: https://crrev.com/4c584d37669a3b2a4e0b80130cb61bad93b0e32e
Cr-Commit-Position: refs/heads/master@{#413639}
==========

[commit-bot: I haz the power, 2016-08-23 02:35:15]

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