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

Unified Diff: ios/chrome/browser/ui/elements/activity_overlay_coordinator.mm

Issue 2881183002: Revert of Replace ObjCPropertyReleaser with ReleaseProperties() project-wide. (Closed)
Patch Set: Created 3 years, 7 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
Index: ios/chrome/browser/ui/elements/activity_overlay_coordinator.mm
diff --git a/ios/chrome/browser/ui/elements/activity_overlay_coordinator.mm b/ios/chrome/browser/ui/elements/activity_overlay_coordinator.mm
index 8ff7dc0975855fbae81ad1a5b419058ec6299b29..f56daeec40b8d4b5b762d423d1ca172b72d4caae 100644
--- a/ios/chrome/browser/ui/elements/activity_overlay_coordinator.mm
+++ b/ios/chrome/browser/ui/elements/activity_overlay_coordinator.mm
@@ -4,11 +4,13 @@
#import "ios/chrome/browser/ui/elements/activity_overlay_coordinator.h"
-#include "base/mac/objc_release_properties.h"
+#import "base/mac/objc_property_releaser.h"
#import "ios/chrome/browser/ui/elements/activity_overlay_view_controller.h"
#import "ios/chrome/browser/ui/uikit_ui_util.h"
-@interface ActivityOverlayCoordinator ()
+@interface ActivityOverlayCoordinator () {
+ base::mac::ObjCPropertyReleaser _propertyReleaser_ActivityOverlayCoordinator;
+}
// View controller that displays an activity indicator.
@property(nonatomic, retain) UIViewController* activityOverlayViewController;
@@ -18,9 +20,14 @@
@synthesize activityOverlayViewController = _activityOverlayViewController;
-- (void)dealloc {
- base::mac::ReleaseProperties(self);
- [super dealloc];
+- (nullable instancetype)initWithBaseViewController:
+ (UIViewController*)viewController {
+ self = [super initWithBaseViewController:viewController];
+ if (self) {
+ _propertyReleaser_ActivityOverlayCoordinator.Init(
+ self, [ActivityOverlayCoordinator class]);
+ }
+ return self;
}
- (void)start {
« no previous file with comments | « ios/chrome/browser/ui/authentication/authentication_flow.mm ('k') | ios/chrome/browser/ui/main/browser_view_wrangler.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698