Fix an ownership issue in ScopedAgentLogger

Fix an ownership issue in ScopedAgentLogger

The temporary directory used by the ScopedAgentLogger class shouldn't be
cleaned up on destruction as it's owned by the object that owns this
logger's instance.

This is a part of the integration_test's design as we need to process the
logger files after it has been destroyed (i.e. to analyze the minidump).

Review-Url: https://codereview.chromium.org/2993173002
Committed: https://github.com/google/syzygy/commit/3c81c059f6e414ce979c5654fd88699f8835e89b

[Sébastien Marchand, 2017-08-07 17:09:42]

sebmarchand@chromium.org changed reviewers:
+ chrisha@chromium.org

[Sébastien Marchand, 2017-08-07 17:09:43]

PTAL.

[Sébastien Marchand, 2017-08-08 20:33:02]

ping?

[chrisha, 2017-08-09 21:29:12]

The CQ bit was checked by chrisha@chromium.org

[chrisha, 2017-08-09 21:29:13]

lgtm

[commit-bot: I haz the power, 2017-08-09 21:29:28]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Sébastien Marchand, 2017-08-09 21:29:33]

The CQ bit was checked by sebmarchand@chromium.org

[commit-bot: I haz the power, 2017-08-09 21:29:46]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-08-09 22:29:02]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1502314173588400,
"parent_rev": "4c7cb860467e960396dc24e5146c57802ce3669b", "commit_rev":
"3c81c059f6e414ce979c5654fd88699f8835e89b"}

[commit-bot: I haz the power, 2017-08-09 22:29:07]

Description was changed from

==========
Fix an ownership issue in ScopedAgentLogger

The temporary directory used by the ScopedAgentLogger class shouldn't be
cleaned up on destruction as it's owned by the object that owns this
logger's instance.

This is a part of the integration_test's design as we need to process the
logger files after it has been destroyed (i.e. to analyze the minidump).
==========

to

==========
Fix an ownership issue in ScopedAgentLogger

The temporary directory used by the ScopedAgentLogger class shouldn't be
cleaned up on destruction as it's owned by the object that owns this
logger's instance.

This is a part of the integration_test's design as we need to process the
logger files after it has been destroyed (i.e. to analyze the minidump).

Review-Url: https://codereview.chromium.org/2993173002
Committed:
https://github.com/google/syzygy/commit/3c81c059f6e414ce979c5654fd88699f8835e89b
==========

[commit-bot: I haz the power, 2017-08-09 22:29:08]

Committed patchset #2 (id:20001) as
https://github.com/google/syzygy/commit/3c81c059f6e414ce979c5654fd88699f8835e89b