Add DBUS_SESSION_BUS_ADDRESS to allowed slave env vars.

Add DBUS_SESSION_BUS_ADDRESS to allowed slave env vars.

R=cmp@chromium.org, jln@chromium.org, kbr@chromium.org
BUG=309093
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=230897

[Ken Russell (switch to Gerrit), 2013-10-25 01:11:38]

Thanks. LGTM

[commit-bot: I haz the power, 2013-10-25 01:12:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/agable@chromium.org/42843002/1

[commit-bot: I haz the power, 2013-10-25 01:12:48]

Presubmit check for 42843002-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    build/slave/run_slave.py

Presubmit checks took 44.5s to calculate.

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[ghost stip (do not use), 2013-10-25 01:14:24]

lgtm

[commit-bot: I haz the power, 2013-10-25 01:14:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/agable@chromium.org/42843002/1

[commit-bot: I haz the power, 2013-10-25 01:15:28]

Presubmit check for 42843002-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    build/slave/run_slave.py

Presubmit checks took 44.2s to calculate.

[agable, 2013-10-25 01:17:16]

On 2013/10/25 01:15:28, I haz the power (commit-bot) wrote:
> Presubmit check for 42843002-1 failed and returned exit status 1.
> 
> Running presubmit commit checks ...
> 
> ** Presubmit ERRORS **
> Missing LGTM from an OWNER for these files:
>     build/slave/run_slave.py
> 
> Presubmit checks took 44.2s to calculate.

Dammit I always forget that run_slave.py has different OWNERS from everything
else.

[iannucci, 2013-10-25 01:19:04]

lgtm

[commit-bot: I haz the power, 2013-10-25 01:19:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/agable@chromium.org/42843002/1

[iannucci, 2013-10-25 01:19:42]

On 2013/10/25 01:19:04, iannucci wrote:
> lgtm

Do we also need DBUS_SESSION_BUS_PID?

[commit-bot: I haz the power, 2013-10-25 01:20:23]

Change committed as 230897

[Ken Russell (switch to Gerrit), 2013-10-25 01:59:55]

Not sure whether we need DBUS_SESSION_BUS_PID. It isn't set on my desktop Linux
machine. I actually doubt either environment variable is set in the parent
environment of the slave scripts. We'll find out in one of the next runs.