Chromium Code Reviews
DescriptionRevert of Fix layout of BubbleFrameView when there's only a close button in the (patchset #5 id:40002 of https://codereview.chromium.org/2807243004/ )
Reason for revert:
Breaks DialogTest.AcceptAndCancel on a couple of bots (https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/59105 and https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%281%29/builds/24880).
Original issue's description:
> Fix layout of BubbleFrameView when there's only a close button in the
> title row.
>
> Push the client area down slightly so it doesn't overlap close. This
> is a bandaid until bug 702196 is fixed.
>
> BUG=709380, 702196
TBR=tapted@chromium.org,pkasting@chromium.org,bsep@chromium.org,estade@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=709380, 702196
Review-Url: https://codereview.chromium.org/2821463004
Cr-Commit-Position: refs/heads/master@{#464773}
Committed: https://chromium.googlesource.com/chromium/src/+/77227cc6631117209eb6cfbbc5c78bf0d3c8a4b4
Patch Set 1 #
Messages
Total messages: 7 (4 generated)
|
|||||||||||||||||||||||||||||||||||||