Chromium Code Reviews
Description[merge-m57] Anchor the first run bubble off the location bar icon's ImageView.
Currently anchors off the whole icon, which might be the centre of a
rectangular security chip.
Use LocationBarView::GetSecurityBubbleAnchorView(), same as other
bubbles. The FirstRunBubble constructor already offsets the anchor by
LOCATION_BAR_BUBBLE_ANCHOR_VERTICAL_INSET, like other bubbles anchoring
off the ImageView.
BUG=690361
Review-Url: https://codereview.chromium.org/2705323004
Cr-Commit-Position: refs/heads/master@{#452728}
(cherry picked from commit 96752f23762f0ad4320bb4aa0737857b5fe582f3)
Review-Url: https://codereview.chromium.org/2716173002 .
Cr-Commit-Position: refs/branch-heads/2987@{#690}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed: https://chromium.googlesource.com/chromium/src/+/fbe3e139935ab2dc813669ae8ac2d2de8ed3715f
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
|||||||||||||||||||