Make CreateRtcLogStreamConfig() return a std::unique_ptr

Make CreateRtcLogStreamConfig() return a std::unique_ptr

This is done for two reasons:
1. This will be useful in an upcoming CL, when all logging will be done with one function that accepts a std::unique_ptr<RtcEvent>.
2. Ensure no unnecessary copies when the compiler fails to optimize.

BUG=webrtc:8111
TBR=stefan@webrtc.org

Review-Url: https://codereview.webrtc.org/3010273002
Cr-Commit-Position: refs/heads/master@{#19746}
Committed: https://chromium.googlesource.com/external/webrtc/+/8ec568a9ff9b30dd7e8881a0c1bdbc418dc05ff4

[eladalon, 2017-09-08 12:01:13]

eladalon@webrtc.org changed reviewers:
+ stefan@webrtc.org, terelius@webrtc.org

[eladalon, 2017-09-08 12:01:14]

PTAL

[terelius, 2017-09-08 12:28:27]

lgtm

[eladalon, 2017-09-08 12:44:13]

Description was changed from

==========
Make CreateRtcLogStreamConfig() return a std::unique_ptr

This is done for two reasons:
1. This will be useful in an upcoming CL, when all logging will be done with one
function that accepts a std::unique_ptr<RtcEvent>.
2. Ensure no unnecessary copies when the compiler fails to optimize.

BUG=webrtc:8111
==========

to

==========
Make CreateRtcLogStreamConfig() return a std::unique_ptr

This is done for two reasons:
1. This will be useful in an upcoming CL, when all logging will be done with one
function that accepts a std::unique_ptr<RtcEvent>.
2. Ensure no unnecessary copies when the compiler fails to optimize.

BUG=webrtc:8111
TBR:stefan@webrtc.org
==========

[eladalon, 2017-09-08 12:44:29]

Description was changed from

==========
Make CreateRtcLogStreamConfig() return a std::unique_ptr

This is done for two reasons:
1. This will be useful in an upcoming CL, when all logging will be done with one
function that accepts a std::unique_ptr<RtcEvent>.
2. Ensure no unnecessary copies when the compiler fails to optimize.

BUG=webrtc:8111
TBR:stefan@webrtc.org
==========

to

==========
Make CreateRtcLogStreamConfig() return a std::unique_ptr

This is done for two reasons:
1. This will be useful in an upcoming CL, when all logging will be done with one
function that accepts a std::unique_ptr<RtcEvent>.
2. Ensure no unnecessary copies when the compiler fails to optimize.

BUG=webrtc:8111
TBR=stefan@webrtc.org
==========

[eladalon, 2017-09-08 12:44:36]

The CQ bit was checked by eladalon@webrtc.org

[commit-bot: I haz the power, 2017-09-08 12:44:52]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-09-08 13:15:52]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1504874676806670, "parent_rev":
"f901661f30cd0bd1416ff95b76a4da9df1451aa6", "commit_rev":
"8ec568a9ff9b30dd7e8881a0c1bdbc418dc05ff4"}

[commit-bot: I haz the power, 2017-09-08 13:15:56]

Description was changed from

==========
Make CreateRtcLogStreamConfig() return a std::unique_ptr

This is done for two reasons:
1. This will be useful in an upcoming CL, when all logging will be done with one
function that accepts a std::unique_ptr<RtcEvent>.
2. Ensure no unnecessary copies when the compiler fails to optimize.

BUG=webrtc:8111
TBR=stefan@webrtc.org
==========

to

==========
Make CreateRtcLogStreamConfig() return a std::unique_ptr

This is done for two reasons:
1. This will be useful in an upcoming CL, when all logging will be done with one
function that accepts a std::unique_ptr<RtcEvent>.
2. Ensure no unnecessary copies when the compiler fails to optimize.

BUG=webrtc:8111
TBR=stefan@webrtc.org

Review-Url: https://codereview.webrtc.org/3010273002
Cr-Commit-Position: refs/heads/master@{#19746}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8ec568a9ff9b30dd7e8881a0c...
==========

[commit-bot: I haz the power, 2017-09-08 13:15:57]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/8ec568a9ff9b30dd7e8881a0c...