Bug 142145 – LayoutTestRelay: App environment variables not set for --guard-malloc or --leaks

• Component: Tools / Tests
• Status: RESOLVED FIXED

[David Kilzer (:ddkilzer)]

When running iOS Simulator tests with --guard-malloc or --leaks, LayoutTestRelay is not passing through the appropriate environment variables, so DumpRenderTree.app and WebKitTestRunner.app are not running with these environment variables set.

[David Kilzer (:ddkilzer)]

Created attachment 247630 [details]
Patch v1

[Alexey Proskuryakov]

Comment on attachment 247630 [details]
Patch v1

View in context: https://bugs.webkit.org/attachment.cgi?id=247630&action=review

> Tools/LayoutTestRelay/LayoutTestRelay/LTRelayController.m:211
> +            @"__XPC_DYLD_FRAMEWORK_PATH": productDirectory,

Adding __XPC_ variants of DYLD_FRAMEWORK_PATH and DYLD_LIBRARY_PATH doesn't seem like part of this fix. What does it fix?

[WebKit Commit Bot]

Comment on attachment 247630 [details]
Patch v1

Clearing flags on attachment: 247630

Committed r180864: <http://trac.webkit.org/changeset/180864>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Darin Adler]

Comment on attachment 247630 [details]
Patch v1

View in context: https://bugs.webkit.org/attachment.cgi?id=247630&action=review

> Tools/LayoutTestRelay/LayoutTestRelay/LTRelayController.m:225
> +        environmentVariables = [dictionary copy];

Is this tool compiled with ARC? If not, we need a [dictionary release].

[David Kilzer (:ddkilzer)]

(In reply to comment #2)
> Comment on attachment 247630 [details]
> Patch v1
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=247630&action=review
> 
> > Tools/LayoutTestRelay/LayoutTestRelay/LTRelayController.m:211
> > +            @"__XPC_DYLD_FRAMEWORK_PATH": productDirectory,
> 
> Adding __XPC_ variants of DYLD_FRAMEWORK_PATH and DYLD_LIBRARY_PATH doesn't
> seem like part of this fix. What does it fix?

It doesn't fix any known issue.  It makes the environment for any XPC-launched process consistent with its parent.  Is that bad?

[David Kilzer (:ddkilzer)]

(In reply to comment #5)
> Comment on attachment 247630 [details]
> Patch v1
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=247630&action=review
> 
> > Tools/LayoutTestRelay/LayoutTestRelay/LTRelayController.m:225
> > +        environmentVariables = [dictionary copy];
> 
> Is this tool compiled with ARC? If not, we need a [dictionary release].

Yes, it's compiled with ARC.

[David Kilzer (:ddkilzer)]

(In reply to comment #3)
> Comment on attachment 247630 [details]
> Patch v1
> 
> Clearing flags on attachment: 247630
> 
> Committed r180864: <http://trac.webkit.org/changeset/180864>

Fix think-o:  Committed r180866: <http://trac.webkit.org/changeset/180866>

[Alexey Proskuryakov]

> It doesn't fix any known issue.  It makes the environment for any XPC-launched process consistent with its parent.  Is that bad?

To me, it seems wrong to have code that does strange things with weird underscores and doesn't actually achieve anything. The functionality of __XPC environment variables is deeply suspect in general, and I expect that people will be looking at these lines in the future, trying to figure out if they can be removed or reimplemented.