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

Unified Diff: ios/chrome/browser/geolocation/test_location_manager.mm

Issue 2510133003: [ObjC ARC] Converts ios/chrome/browser/geolocation:test_support to ARC.Automatically generated AR… (Closed)
Patch Set: Created 4 years, 1 month 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/geolocation/test_location_manager.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/geolocation/test_location_manager.mm
diff --git a/ios/chrome/browser/geolocation/test_location_manager.mm b/ios/chrome/browser/geolocation/test_location_manager.mm
index eb6633eaf84d124313a21a4fa8bbb16f557887da..402f28c66194d107a3a8e32e2eabc8a11133a6e3 100644
--- a/ios/chrome/browser/geolocation/test_location_manager.mm
+++ b/ios/chrome/browser/geolocation/test_location_manager.mm
@@ -6,6 +6,10 @@
#include "base/mac/scoped_nsobject.h"
+#if !defined(__has_feature) || !__has_feature(objc_arc)
+#error "This file requires ARC support."
+#endif
+
@interface TestLocationManager () {
CLAuthorizationStatus _authorizationStatus;
sdefresne 2016/11/25 10:13:24 nit: remove
base::scoped_nsobject<CLLocation> _currentLocation;
@@ -43,7 +47,7 @@
}
- (void)setCurrentLocation:(CLLocation*)currentLocation {
- _currentLocation.reset([currentLocation retain]);
+ _currentLocation.reset(currentLocation);
}
- (void)reset {
« no previous file with comments | « ios/chrome/browser/geolocation/test_location_manager.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698