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

Unified Diff: components/translate/ios/browser/js_translate_manager.mm

Issue 2645603002: [ObjC ARC] Converts components/translate/ios/browser:browser to ARC. (Closed)
Patch Set: Removed include Created 3 years, 11 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: components/translate/ios/browser/js_translate_manager.mm
diff --git a/components/translate/ios/browser/js_translate_manager.mm b/components/translate/ios/browser/js_translate_manager.mm
index 47f2a4e3e0c21cbb365b312a2fc86efcf8fcadd9..aa5df28283bf7a66e3b954cb978f9be608483c78 100644
--- a/components/translate/ios/browser/js_translate_manager.mm
+++ b/components/translate/ios/browser/js_translate_manager.mm
@@ -10,14 +10,18 @@
#include "base/logging.h"
#include "base/mac/bundle_locations.h"
-#import "base/mac/scoped_nsobject.h"
+#import "ios/web/public/web_state/js/crw_js_injection_receiver.h"
+
+#if !defined(__has_feature) || !__has_feature(objc_arc)
+#error "This file requires ARC support."
+#endif
@implementation JsTranslateManager {
- base::scoped_nsobject<NSString> _translationScript;
+ NSString* _translationScript;
}
- (NSString*)script {
- return _translationScript.get();
+ return _translationScript;
}
- (void)setScript:(NSString*)script {
@@ -43,7 +47,7 @@
error:&error];
DCHECK(!error && [content length]);
script = [script stringByAppendingString:content];
- _translationScript.reset([script copy]);
+ _translationScript = [script copy];
}
- (void)injectWaitUntilTranslateReadyScript {
@@ -75,7 +79,9 @@
- (NSString*)injectionContent {
DCHECK(_translationScript);
- return _translationScript.autorelease();
+ NSString* translationScript = _translationScript;
+ _translationScript = nil;
+ return translationScript;
}
@end

Powered by Google App Engine
This is Rietveld 408576698