RESOLVED FIXED 143943
Remove OwnPtr and PassOwnPtr use from WebKit/cf, WebKit/mac, and WebKit2
https://bugs.webkit.org/show_bug.cgi?id=143943
Summary Remove OwnPtr and PassOwnPtr use from WebKit/cf, WebKit/mac, and WebKit2
Darin Adler
Reported 2015-04-19 22:52:23 PDT
Remove OwnPtr and PassOwnPtr use from WebKit/cf, WebKit/mac, and WebKit2
Attachments
Patch (29.28 KB, patch)
2015-04-19 23:03 PDT, Darin Adler
no flags
Patch (29.28 KB, patch)
2015-04-19 23:27 PDT, Darin Adler
no flags
Patch (8.66 KB, patch)
2015-04-20 09:45 PDT, Darin Adler
no flags
Patch (29.99 KB, patch)
2015-04-21 08:55 PDT, Darin Adler
no flags
Patch (29.99 KB, patch)
2015-04-21 08:56 PDT, Darin Adler
andersca: review+
Patch (35.58 KB, patch)
2015-04-22 21:38 PDT, Darin Adler
no flags
Patch (35.67 KB, patch)
2015-04-22 22:23 PDT, Darin Adler
no flags
Patch (35.53 KB, patch)
2015-04-22 22:34 PDT, Darin Adler
no flags
Darin Adler
Comment 1 2015-04-19 23:03:18 PDT
Darin Adler
Comment 2 2015-04-19 23:27:05 PDT
Darin Adler
Comment 3 2015-04-20 09:45:45 PDT
Darin Adler
Comment 4 2015-04-20 10:14:13 PDT
Comment on attachment 251162 [details] Patch Need to fix iOS compilation.
Darin Adler
Comment 5 2015-04-21 08:55:38 PDT
Darin Adler
Comment 6 2015-04-21 08:56:14 PDT
Anders Carlsson
Comment 7 2015-04-22 08:40:31 PDT
Comment on attachment 251231 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=251231&action=review iOS build needs fixing, Editor::insertDictationPhrases needs to be updated. > Source/WebCore/platform/network/ios/QuickLook.mm:191 > +NSURLRequest *WebCore::registerQLPreviewConverterIfNeeded(NSURL *url, NSString *mimeType, NSData *data) I think this should return a RetainPtr instead.
Darin Adler
Comment 8 2015-04-22 09:04:42 PDT
Comment on attachment 251231 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=251231&action=review >> Source/WebCore/platform/network/ios/QuickLook.mm:191 >> +NSURLRequest *WebCore::registerQLPreviewConverterIfNeeded(NSURL *url, NSString *mimeType, NSData *data) > > I think this should return a RetainPtr instead. I see your point. I thought it would just return an autoreleased object, but I guess there’s no guarantee that the result of [converter previewRequest] will still be valid once converter is released.
Darin Adler
Comment 9 2015-04-22 21:38:09 PDT
Darin Adler
Comment 10 2015-04-22 22:23:11 PDT
Darin Adler
Comment 11 2015-04-22 22:34:44 PDT
Darin Adler
Comment 12 2015-04-22 23:03:22 PDT
Note You need to log in before you can comment on or make changes to this bug.