Bug 173483

Summary: Port setup_environ_for_server should not set environment variables related with the driver.
Product: WebKit Reporter: Carlos Alberto Lopez Perez <clopez>
Component: Tools / TestsAssignee: Carlos Alberto Lopez Perez <clopez>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, buildbot, cgarcia, clopez, dbates, glenn, jbedard, lforschler, rniwa, youennf, zan
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 173433    
Attachments:
Description Flags
Patch cgarcia: review+

Description Carlos Alberto Lopez Perez 2017-06-16 11:17:19 PDT
On bug 173433 I found that the port setup_environ_for_server() is setting some environment variables (like DISPLAY) that should not be setted there.
Its the job of the driver (on _setup_environ_for_test()) setting that variables.
Comment 1 Carlos Alberto Lopez Perez 2017-06-16 11:41:08 PDT
Created attachment 313101 [details]
Patch
Comment 2 Carlos Garcia Campos 2017-06-17 00:13:41 PDT
Comment on attachment 313101 [details]
Patch

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

> Tools/Scripts/webkitpy/port/base.py:871
> +        # Use the driver _setup_environ_for_test() method for that..

Double . at the end.
Comment 3 Carlos Alberto Lopez Perez 2017-06-19 03:39:32 PDT
Committed r218482: <http://trac.webkit.org/changeset/218482>
Comment 4 Carlos Alberto Lopez Perez 2017-06-19 06:01:28 PDT
Committed r218490: <http://trac.webkit.org/changeset/218490>