RESOLVED FIXED144884
Simplify shim path computation 
https://bugs.webkit.org/show_bug.cgi?id=144884
Summary Simplify shim path computation
Anders Carlsson
Reported 2015-05-11 15:23:44 PDT
Simplify shim path computation
Attachments
Patch (6.47 KB, patch)
2015-05-11 15:25 PDT, Anders Carlsson 		sam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2015-05-11 15:25:51 PDT
Created attachment 252896 [details] Patch
Anders Carlsson
Comment 2 2015-05-11 15:33:03 PDT
Committed r184125: <http://trac.webkit.org/changeset/184125>
Darin Adler
Comment 3 2015-05-11 18:11:19 PDT
Comment on attachment 252896 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=252896&action=review > Source/WebKit2/UIProcess/Launcher/mac/ProcessLauncherMac.mm:97 > +static RetainPtr<NSString> computeProcessShimPath(const ProcessLauncher::LaunchOptions& launchOptions, NSBundle *webKitBundle) Why to just NSString * for the return value? Seems like all the code paths return something autoreleased.
Note You need to log in before you can comment on or make changes to this bug.