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

Remove the unseen state in UI Refresh

This CL removes the unseen state (in the tools menu button in the
toolbar) when the UI Refresh flag is enabled.

Bug: 828367, 831134
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I4446d5b4f2565f256e4fedb829a77e93295b0d00
Reviewed-on: https://chromium-review.googlesource.com/1014092Reviewed-by: default avatarOlivier Robin <olivierrobin@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551302}
parent f506af59
...@@ -423,8 +423,7 @@ PopupMenuToolsItem* CreateTableViewItem(int titleID, ...@@ -423,8 +423,7 @@ PopupMenuToolsItem* CreateTableViewItem(int titleID,
} }
- (void)unseenStateChanged:(BOOL)unseenItemsExist { - (void)unseenStateChanged:(BOOL)unseenItemsExist {
// TODO(crbug.com/828367): Implement this once the "unseen items effect" is // TODO(crbug.com/800266): Remove this method.
// defined.
} }
#pragma mark - Popup updates (Private) #pragma mark - Popup updates (Private)
......
...@@ -121,7 +121,11 @@ const CGFloat kStrokeEndAtApogee = 1; ...@@ -121,7 +121,11 @@ const CGFloat kStrokeEndAtApogee = 1;
} }
- (void)triggerAnimation { - (void)triggerAnimation {
[self animateToColor:toolbar::HighlighButtonTint(style_)]; if (IsUIRefreshPhase1Enabled()) {
[self animateToColor:self.tintColor];
} else {
[self animateToColor:toolbar::HighlighButtonTint(style_)];
}
} }
#pragma mark - Private #pragma mark - Private
......
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