Commit beb48c98 authored by Mark Cogan's avatar Mark Cogan Committed by Commit Bot

[iOS] Handle nil in GridTransitionLayout setSelectedItem:

In some cases, there may be no visible items in a grid, so the selected
item will be nil, which is never a member of any collection. This CL
updates the DCHECK condition in [GridTransitionLayout setSelectedItem:]
to not trigger in this case.

Bug: 822912
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I845355feee5f4af1d3443ff3d362aae45c05d9c9
Reviewed-on: https://chromium-review.googlesource.com/973121Reviewed-by: default avatarGauthier Ambard <gambard@chromium.org>
Commit-Queue: Mark Cogan <marq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544687}
parent 6c783dbf
...@@ -29,7 +29,7 @@ ...@@ -29,7 +29,7 @@
} }
- (void)setSelectedItem:(GridTransitionLayoutItem*)selectedItem { - (void)setSelectedItem:(GridTransitionLayoutItem*)selectedItem {
DCHECK([self.items containsObject:selectedItem]); DCHECK(!selectedItem || [self.items containsObject:selectedItem]);
_selectedItem = selectedItem; _selectedItem = selectedItem;
} }
......
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