Commit 23532e5f authored by Ewann's avatar Ewann Committed by Commit Bot

[iOS] Disables Done Button when Close All Tabs Confirmation is displayed

This CL updates the behavior of the Done button according to the other
buttons on the top Toolbar.

Bug: 1140518
Change-Id: I9e34024c4826a47fb9547ba3d0966f50596fbd1a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2512917Reviewed-by: default avatarMark Cogan <marq@chromium.org>
Commit-Queue: Ewann Pellé <ewannpv@chromium.org>
Cr-Commit-Position: refs/heads/master@{#823975}
parent f27ad210
...@@ -946,7 +946,10 @@ NSUInteger GetPageIndexFromPage(TabGridPage page) { ...@@ -946,7 +946,10 @@ NSUInteger GetPageIndexFromPage(TabGridPage page) {
NOTREACHED() << "The done button should not be configured based on the " NOTREACHED() << "The done button should not be configured based on the "
"contents of the recent tabs page."; "contents of the recent tabs page.";
} }
self.doneButton.enabled = !gridViewController.gridEmpty; // The Done button should have the same behavior as the other buttons on the
// top Toolbar.
self.doneButton.enabled = !gridViewController.gridEmpty &&
self.topToolbar.pageControl.userInteractionEnabled;
} }
- (void)configureCloseAllButtonForCurrentPageAndUndoAvailability { - (void)configureCloseAllButtonForCurrentPageAndUndoAvailability {
......
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