EventLogParser: use std::vector to reduce stack usage

EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.

NOTRY=true

Committed: https://crrev.com/1afdf99f9c00155b0c250e9a358de0184933b27c
Cr-Commit-Position: refs/heads/master@{#14055}

[skvlad, 2016-08-31 19:42:32]

Description was changed from

==========
EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.
==========

to

==========
EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.
==========

[skvlad, 2016-08-31 19:42:32]

skvlad@webrtc.org changed reviewers:
+ mflodman@webrtc.org

[mflodman, 2016-09-02 08:40:34]

lgtm

[skvlad, 2016-09-02 17:53:16]

The CQ bit was checked by skvlad@webrtc.org

[commit-bot: I haz the power, 2016-09-02 17:53:27]

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, 2016-09-02 19:53:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-02 19:53:52]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[skvlad, 2016-09-02 20:13:58]

Description was changed from

==========
EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.
==========

to

==========
EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.

NOTRY=true
==========

[skvlad, 2016-09-02 20:14:05]

The CQ bit was checked by skvlad@webrtc.org

[commit-bot: I haz the power, 2016-09-02 20:14:10]

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, 2016-09-02 20:22:28]

Description was changed from

==========
EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.

NOTRY=true
==========

to

==========
EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.

NOTRY=true
==========

[commit-bot: I haz the power, 2016-09-02 20:22:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-02 20:22:36]

Description was changed from

==========
EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.

NOTRY=true
==========

to

==========
EventLogParser: use std::vector to reduce stack usage

ParsedRtcEventLog::ParseStream was using a stack-allocated 64kb array
for a temporary buffer. This was causing problems in build environments
with restrictions on stack size.

This change replaces it with an std::vector.

NOTRY=true

Committed: https://crrev.com/1afdf99f9c00155b0c250e9a358de0184933b27c
Cr-Commit-Position: refs/heads/master@{#14055}
==========

[commit-bot: I haz the power, 2016-09-02 20:22:37]

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