Commit ec8115eb authored by Gauthier Ambard's avatar Gauthier Ambard Committed by Commit Bot

[iOS] Do not check for user engagement for bottom tip

The bottom toolbar tip doesn't use the isUserEngaged variable.
This allows us to display the tip multiple time during the isUserEngaged
period, for example during test.

Bug: 881829
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: Ia4559353aaf831abeb34fd01bf4f4911d1a3267e
Reviewed-on: https://chromium-review.googlesource.com/1215208Reviewed-by: default avataredchin <edchin@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590250}
parent 63df278f
...@@ -170,8 +170,10 @@ const CGFloat kBubblePresentationDelay = 1; ...@@ -170,8 +170,10 @@ const CGFloat kBubblePresentationDelay = 1;
[self presentNewTabTipBubble]; [self presentNewTabTipBubble];
if (!self.incognitoTabTipBubblePresenter.isUserEngaged) if (!self.incognitoTabTipBubblePresenter.isUserEngaged)
[self presentNewIncognitoTabTipBubble]; [self presentNewIncognitoTabTipBubble];
if (!self.bottomToolbarTipBubblePresenter.isUserEngaged)
[self presentBottomToolbarTipBubble]; // The bottom toolbar doesn't use the isUserEngaged, so don't check if the
// user is engaged here.
[self presentBottomToolbarTipBubble];
} }
- (void)presentLongPressBubble { - (void)presentLongPressBubble {
......
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