Commit 594a5f31 authored by Gauthier Ambard's avatar Gauthier Ambard Committed by Commit Bot

Display the toolbar when opening new tab on iPad

When the user long press a link and open it in a new tab, the toolbar
should stay in its state on iPhone. On iPad, the toolbar should be
shown.

Bug: 793830
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I34c345ba0687385a6e546181584230631ed61ad3
Reviewed-on: https://chromium-review.googlesource.com/823923Reviewed-by: default avatarMark Cogan <marq@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523738}
parent 5ec1b52d
...@@ -3295,7 +3295,8 @@ bubblePresenterForFeature:(const base::Feature&)feature ...@@ -3295,7 +3295,8 @@ bubblePresenterForFeature:(const base::Feature&)feature
// TODO(crbug.com/778822): This can be cleaned up when the new fullscreen // TODO(crbug.com/778822): This can be cleaned up when the new fullscreen
// is enabled. // is enabled.
if (IsSafeAreaCompatibleToolbarEnabled() && if (IsSafeAreaCompatibleToolbarEnabled() &&
header.view == _toolbarCoordinator.toolbarViewController.view) { header.view == _toolbarCoordinator.toolbarViewController.view &&
!IsIPadIdiom()) {
self.toolbarOffsetConstraint.constant = yOrigin; self.toolbarOffsetConstraint.constant = yOrigin;
} }
CGRect frame = [header.view frame]; CGRect frame = [header.view frame];
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment