Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(97)

Unified Diff: ios/chrome/browser/ui/stack_view/card_set.h

Issue 2944443003: [ObjC ARC] Converts ios/chrome/browser/ui/stack_view:stack_view to ARC. (Closed)
Patch Set: Add and use explicit disconnect in CardSet. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/chrome/browser/ui/stack_view/BUILD.gn ('k') | ios/chrome/browser/ui/stack_view/card_set.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/ui/stack_view/card_set.h
diff --git a/ios/chrome/browser/ui/stack_view/card_set.h b/ios/chrome/browser/ui/stack_view/card_set.h
index 030e6e358b898e3a0970204507fa0b1ebea920f5..8280ac79620b3f6b7d1182e2a138abfbf3a0c0b2 100644
--- a/ios/chrome/browser/ui/stack_view/card_set.h
+++ b/ios/chrome/browser/ui/stack_view/card_set.h
@@ -52,17 +52,17 @@ struct LayoutRect;
@property(nonatomic, readonly) NSArray* cards;
// The card corresponding to the currently selected tab in the tab model.
// Setting this property will change the selection in the tab model.
-@property(nonatomic, assign, readwrite) StackCard* currentCard;
+@property(nonatomic) StackCard* currentCard;
// Set to the card that is currently animating closed, if any.
-@property(nonatomic, assign, readwrite) StackCard* closingCard;
+@property(nonatomic) StackCard* closingCard;
// The view that cards should be displayed in. Changing the display view will
// remove cards from the previous view, but they will not be re-displayed in the
// new view without a call to updateCardVisibilities.
-@property(nonatomic, retain, readwrite) UIView* displayView;
+@property(nonatomic, strong, readwrite) UIView* displayView;
// The side on which the close button should be displayed.
@property(nonatomic, readonly) CardCloseButtonSide closeButtonSide;
// The object to be notified about addition and removal of cards.
-@property(nonatomic, assign, readwrite) id<CardSetObserver> observer;
+@property(nonatomic, weak, readwrite) id<CardSetObserver> observer;
// While YES, changes to the tab model will not affect the card stack. When
// this is changed back to NO, the card stack will be completely rebuilt, so
// this should be used very carefully.
@@ -88,6 +88,10 @@ struct LayoutRect;
// outlive the card set.
- (id)initWithModel:(TabModel*)model;
+// Tears down any observation or other state. After this method is called, the
+// receiver should be set to nil or otherwise marked for deallocation.
+- (void)disconnect;
+
// The tab model backing the card set.
// TODO(stuartmorgan): See if this can reasonably be internalized.
- (TabModel*)tabModel;
« no previous file with comments | « ios/chrome/browser/ui/stack_view/BUILD.gn ('k') | ios/chrome/browser/ui/stack_view/card_set.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698