Add scoped_ptr to Environment object.

Add scoped_ptr to Environment object.

BUG=None
TEST=Compiles.

Committed: https://crrev.com/a2b139ffaa11e31423ffd7007071c11cfa6d2ba7
Cr-Commit-Position: refs/heads/master@{#304085}

[Jorge Lucangeli Obes, 2014-11-13 01:39:45]

jorgelo@chromium.org changed reviewers:
+ agl@chromium.org, wtc@chromium.org

[Jorge Lucangeli Obes, 2014-11-13 01:39:45]

Aren't we leaking the Environment object here?

[agl, 2014-11-13 19:32:25]

lgtm

[Jorge Lucangeli Obes, 2014-11-13 19:34:55]

The CQ bit was checked by jorgelo@chromium.org

[commit-bot: I haz the power, 2014-11-13 19:37:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/722513005/1

[wtc, 2014-11-13 21:00:26]

Patch set 1 LGTM. Thanks.

https://chromiumcodereview.appspot.com/722513005/diff/1/crypto/nss_util.cc
File crypto/nss_util.cc (right):

https://chromiumcodereview.appspot.com/722513005/diff/1/crypto/nss_util.cc#ne...
crypto/nss_util.cc:838: scoped_ptr<base::Environment>
env(base::Environment::Create());

The object that base::Environment::Create() returns has no (apparent) data
members. Perhaps all the base::Environment methods should just be static
methods.

[commit-bot: I haz the power, 2014-11-13 21:30:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-11-13 21:31:36]

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