Fix nacl_integration script since it needs to be updated after test_env.py updates.

Fix nacl_integration script since it needs to be updated after test_env.py updates.

TBR=raymes

Committed: https://chromium.googlesource.com/chromium/src/+/7e31938ec36279c2f1a64640aa8e448488cd6b19

[jam, 2014-10-15 21:11:15]

jam@chromium.org changed reviewers:
+ yzshen@chromium.org

[jam, 2014-10-15 21:12:09]

jam@chromium.org changed reviewers:
+ raymes@chromium.org
- yzshen@chromium.org

[commit-bot: I haz the power, 2014-10-15 21:15:35]

Patchset 1 (id:??) landed as
https://crrev.com/7e31938ec36279c2f1a64640aa8e448488cd6b19
Cr-Commit-Position: refs/heads/master@{#299764}

[jam, 2014-10-15 21:15:42]

Committed patchset #1 (id:1) manually as
7e31938ec36279c2f1a64640aa8e448488cd6b19 (presubmit successful).

[raymes, 2014-10-15 21:38:25]

raymes@chromium.org changed reviewers:
+ mseaborn@chromium.org

[raymes, 2014-10-15 21:38:25]

rubberstamp lgtm

[Mark Seaborn, 2014-10-15 22:03:32]

https://codereview.chromium.org/659853002/diff/1/ppapi/native_client/tools/br...
File ppapi/native_client/tools/browser_tester/browser_tester.py (right):

https://codereview.chromium.org/659853002/diff/1/ppapi/native_client/tools/br...
ppapi/native_client/tools/browser_tester/browser_tester.py:214:
env.update(test_env.get_sandbox_env(sys.argv, os.environ))
This doesn't look right.  Where is "env" defined?  Doesn't pyflakes give a
warning about this?

What is the test_env change you're changing this to match?

[raymes, 2014-10-15 22:05:25]

See the followup in https://codereview.chromium.org/662503002/

On Wed Oct 15 2014 at 3:03:33 PM <mseaborn@chromium.org> wrote:

>
> https://codereview.chromium.org/659853002/diff/1/ppapi/
> native_client/tools/browser_tester/browser_tester.py
> File ppapi/native_client/tools/browser_tester/browser_tester.py (right):
>
> https://codereview.chromium.org/659853002/diff/1/ppapi/
> native_client/tools/browser_tester/browser_tester.py#newcode214
> ppapi/native_client/tools/browser_tester/browser_tester.py:214
>
<https://codereview.chromium.org/659853002/diff/1/ppapi/native_client/tools/br...>
> :
> env.update(test_env.get_sandbox_env(sys.argv, os.environ))
> This doesn't look right.  Where is "env" defined?  Doesn't pyflakes give
> a warning about this?
>
> What is the test_env change you're changing this to match?
>
> https://codereview.chromium.org/659853002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.