Reinstated sctputils_unittest.cc

Reinstated sctputils_unittest.cc

As I was preparing to move some files from the api/ folder, I noticed
that this file was not included in the BUILD.gn file. I've added it back
in and updated it to compile and run successfully again.

BUG=webrtc:5883
Committed: https://crrev.com/d4d2f6009e664cb6833bb1b1a3e8820c1bc44245
Cr-Commit-Position: refs/heads/master@{#14965}

[ossu, 2016-11-07 13:19:18]

Description was changed from

==========
Reinstated sctputils_unittest.cc

As I was preparing to move some files from the api/ folder, I noticed
that this file was not included in the BUILD.gn file. I've added it back
in and updated it to compile and run successfully again.

BUG=
==========

to

==========
Reinstated sctputils_unittest.cc

As I was preparing to move some files from the api/ folder, I noticed
that this file was not included in the BUILD.gn file. I've added it back
in and updated it to compile and run successfully again.

BUG=webrtc:5883
==========

[ossu, 2016-11-07 13:19:30]

The CQ bit was checked by ossu@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-07 13:19:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[ossu, 2016-11-07 13:21:13]

ossu@webrtc.org changed reviewers:
+ deadbeef@webrtc.org

[ossu, 2016-11-07 13:21:13]

The file's been dead for a while but you seem to be one of the last to change
it, and as an added bonus, you're an owner as well.

[commit-bot: I haz the power, 2016-11-07 14:43:13]

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

[commit-bot: I haz the power, 2016-11-07 14:43:13]

Dry run: Try jobs failed on following builders:
  win_x64_clang_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_clang_dbg/build...)

[Taylor Brandstetter, 2016-11-07 20:06:20]

lgtm

https://codereview.webrtc.org/2485603002/diff/1/webrtc/api/sctputils_unittest.cc
File webrtc/api/sctputils_unittest.cc (left):

https://codereview.webrtc.org/2485603002/diff/1/webrtc/api/sctputils_unittest...
webrtc/api/sctputils_unittest.cc:153: EXPECT_FALSE(webrtc::IsOpenMessage(open));
Wait, how was this ever passing?

[ossu, 2016-11-08 09:55:48]

The CQ bit was checked by ossu@webrtc.org

[commit-bot: I haz the power, 2016-11-08 09:55:51]

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

[ossu, 2016-11-08 09:55:56]

https://codereview.webrtc.org/2485603002/diff/1/webrtc/api/sctputils_unittest.cc
File webrtc/api/sctputils_unittest.cc (left):

https://codereview.webrtc.org/2485603002/diff/1/webrtc/api/sctputils_unittest...
webrtc/api/sctputils_unittest.cc:153: EXPECT_FALSE(webrtc::IsOpenMessage(open));
On 2016/11/07 20:06:20, Taylor Brandstetter wrote:
> Wait, how was this ever passing?

Yeah, I'm not sure this was ever run, at least not after these changes.

[commit-bot: I haz the power, 2016-11-08 10:05:34]

Description was changed from

==========
Reinstated sctputils_unittest.cc

As I was preparing to move some files from the api/ folder, I noticed
that this file was not included in the BUILD.gn file. I've added it back
in and updated it to compile and run successfully again.

BUG=webrtc:5883
==========

to

==========
Reinstated sctputils_unittest.cc

As I was preparing to move some files from the api/ folder, I noticed
that this file was not included in the BUILD.gn file. I've added it back
in and updated it to compile and run successfully again.

BUG=webrtc:5883
==========

[commit-bot: I haz the power, 2016-11-08 10:05:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-08 10:05:38]

Description was changed from

==========
Reinstated sctputils_unittest.cc

As I was preparing to move some files from the api/ folder, I noticed
that this file was not included in the BUILD.gn file. I've added it back
in and updated it to compile and run successfully again.

BUG=webrtc:5883
==========

to

==========
Reinstated sctputils_unittest.cc

As I was preparing to move some files from the api/ folder, I noticed
that this file was not included in the BUILD.gn file. I've added it back
in and updated it to compile and run successfully again.

BUG=webrtc:5883

Committed: https://crrev.com/d4d2f6009e664cb6833bb1b1a3e8820c1bc44245
Cr-Commit-Position: refs/heads/master@{#14965}
==========

[commit-bot: I haz the power, 2016-11-08 10:05:39]

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