GitLab

The calculations for bookmark bar icons should only impact the start y location of the icons and not the height.

Regression mentioned in comment #5 in the bug is due to removing kBookmarkBarBottomMargin in the previous CL for the bug.
The underlying reason for this is that GetPreferredHeight() was being used for both the bookmark height as well as the button heights.

Bug: 865555
Change-Id: Ief5328a2a3a8c23ebfbaa9a393f1c3af4c7a6920
Reviewed-on: https://chromium-review.googlesource.com/1170226
Commit-Queue: Charlene Yan <cyan@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584470}