Chromium Code Reviews| Index: ios/chrome/browser/ui/omnibox/page_info_view_controller.mm |
| diff --git a/ios/chrome/browser/ui/omnibox/page_info_view_controller.mm b/ios/chrome/browser/ui/omnibox/page_info_view_controller.mm |
| index e977b05e787d5a11c536b7c0ffb6325747d7e77e..43817cca90f14ad158ca29b3aed8ff6cadb45a44 100644 |
| --- a/ios/chrome/browser/ui/omnibox/page_info_view_controller.mm |
| +++ b/ios/chrome/browser/ui/omnibox/page_info_view_controller.mm |
| @@ -508,6 +508,9 @@ void PageInfoModelBubbleBridge::OnPageInfoModelChanged() { |
| CGSize sizeWithFont = |
| [[[button titleLabel] text] cr_pixelAlignedSizeWithFont:font]; |
| frame.size = sizeWithFont; |
| + // According to iOS Human Interface Guidelines, minimal size of UIButton |
| + // should be 44x44. |
| + frame.size.height = MAX(44, frame.size.height); |
|
pkl (ping after 24h if needed)
2017/03/02 14:34:22
Do we normally prefer std::max over MAX ?
Olivier
2017/03/03 08:27:38
$ find . -name "*.mm" -exec grep "std::max(" {} \;
|
| [button setFrame:frame]; |