Fix neteq_speed_test.cc

Fix neteq_speed_test.cc

After https://codereview.webrtc.org/2340773002,
the path from webrtc::test::ResourcePath in
webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.cc is wrong.

It is
/path/to/repos/resources/audio_coding/testfile32kHz.pcm

It should be
/path/to/repos/webrtc-temp/src/resources/audio_coding/testfile32kHz.pcm.

The middle part is missing.

The reason this target is affected is because
webrtc::test::SetExecutablePath(argv[0]);
was not called.

That call is necessary for us to know that the test is being run from src/
and not from out/Default (as is assumed, when that function is not called.)

BUG=chromium:497757
R=kjellander@webrtc.org, henrik.lundin@webrtc.org

Review-Url: https://codereview.webrtc.org/2698743002
Cr-Commit-Position: refs/heads/master@{#16641}
Committed: https://chromium.googlesource.com/external/webrtc/+/454c1d6a2305b7f0c8395b8e76b9a8aaaf8ee258

[ehmaldonado_webrtc, 2017-02-15 20:51:52]

Description was changed from

==========
Fix neteq_speed_test.cc

After https://codereview.webrtc.org/2340773002,
the path from webrtc::test::ResourcePath in
webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.cc is wrong.

It is
/usr/local/google/home/hlundin/repos/resources/audio_coding/testfile32kHz.pcm

It should be
/usr/local/google/home/hlundin/repos/webrtc-temp/src/resources/audio_coding/testfile32kHz.pcm.

The middle part is missing.

The reason this target is affected is because
webrtc::test::SetExecutablePath(argv[0]);
was not called.

That call is necessary for us to know that the test is being run from src/
and not from out/Default (as is assumed, when that function is not called.)

BUG=chromium:497757
R=kjellander@webrtc.org, hlundin@webrtc.org
==========

to

==========
Fix neteq_speed_test.cc

After https://codereview.webrtc.org/2340773002,
the path from webrtc::test::ResourcePath in
webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.cc is wrong.

It is
/usr/local/google/home/hlundin/repos/resources/audio_coding/testfile32kHz.pcm

It should be
/usr/local/google/home/hlundin/repos/webrtc-temp/src/resources/audio_coding/testfile32kHz.pcm.

The middle part is missing.

The reason this target is affected is because
webrtc::test::SetExecutablePath(argv[0]);
was not called.

That call is necessary for us to know that the test is being run from src/
and not from out/Default (as is assumed, when that function is not called.)

BUG=chromium:497757
R=kjellander@webrtc.org, henrik.lundin@webrtc.org
==========

[ehmaldonado_webrtc, 2017-02-15 20:51:52]

ehmaldonado@webrtc.org changed reviewers:
+ henrik.lundin@webrtc.org
- hlundin@webrtc.org

[kjellander_webrtc, 2017-02-16 07:37:09]

lgtm

Ideally we'd fix fileutils so webrtc::test::SetExecutablePath() is never needed,
but obviously that's more work. Most test binaries use test_support_main anyway.

[hlundin-webrtc, 2017-02-16 08:28:52]

lgtm

But I think you should "anonymize" the example paths in the CL description a
bit. Instead of 
/usr/local/google/home/hlundin/repos/...
is suggest
/path/to/repos/...

[ehmaldonado_webrtc, 2017-02-16 11:09:13]

Description was changed from

==========
Fix neteq_speed_test.cc

After https://codereview.webrtc.org/2340773002,
the path from webrtc::test::ResourcePath in
webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.cc is wrong.

It is
/usr/local/google/home/hlundin/repos/resources/audio_coding/testfile32kHz.pcm

It should be
/usr/local/google/home/hlundin/repos/webrtc-temp/src/resources/audio_coding/testfile32kHz.pcm.

The middle part is missing.

The reason this target is affected is because
webrtc::test::SetExecutablePath(argv[0]);
was not called.

That call is necessary for us to know that the test is being run from src/
and not from out/Default (as is assumed, when that function is not called.)

BUG=chromium:497757
R=kjellander@webrtc.org, henrik.lundin@webrtc.org
==========

to

==========
Fix neteq_speed_test.cc

After https://codereview.webrtc.org/2340773002,
the path from webrtc::test::ResourcePath in
webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.cc is wrong.

It is
/path/to/repos/resources/audio_coding/testfile32kHz.pcm

It should be
/path/to/repos/webrtc-temp/src/resources/audio_coding/testfile32kHz.pcm.

The middle part is missing.

The reason this target is affected is because
webrtc::test::SetExecutablePath(argv[0]);
was not called.

That call is necessary for us to know that the test is being run from src/
and not from out/Default (as is assumed, when that function is not called.)

BUG=chromium:497757
R=kjellander@webrtc.org, henrik.lundin@webrtc.org
==========

[ehmaldonado_webrtc, 2017-02-16 11:10:23]

On 2017/02/16 08:28:52, hlundin-webrtc wrote:
> lgtm
> 
> But I think you should "anonymize" the example paths in the CL description a
> bit. Instead of 
> /usr/local/google/home/hlundin/repos/...
> is suggest
> /path/to/repos/...

Done :)

[ehmaldonado_webrtc, 2017-02-16 11:10:25]

The CQ bit was checked by ehmaldonado@webrtc.org

[commit-bot: I haz the power, 2017-02-16 11:10:32]

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-02-16 11:54:50]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487243425790160, "parent_rev":
"2b1020d35fdcc7a7692baaccdf85d0b8534a984f", "commit_rev":
"454c1d6a2305b7f0c8395b8e76b9a8aaaf8ee258"}

[commit-bot: I haz the power, 2017-02-16 11:54:53]

Description was changed from

==========
Fix neteq_speed_test.cc

After https://codereview.webrtc.org/2340773002,
the path from webrtc::test::ResourcePath in
webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.cc is wrong.

It is
/path/to/repos/resources/audio_coding/testfile32kHz.pcm

It should be
/path/to/repos/webrtc-temp/src/resources/audio_coding/testfile32kHz.pcm.

The middle part is missing.

The reason this target is affected is because
webrtc::test::SetExecutablePath(argv[0]);
was not called.

That call is necessary for us to know that the test is being run from src/
and not from out/Default (as is assumed, when that function is not called.)

BUG=chromium:497757
R=kjellander@webrtc.org, henrik.lundin@webrtc.org
==========

to

==========
Fix neteq_speed_test.cc

After https://codereview.webrtc.org/2340773002,
the path from webrtc::test::ResourcePath in
webrtc/modules/audio_coding/neteq/tools/neteq_performance_test.cc is wrong.

It is
/path/to/repos/resources/audio_coding/testfile32kHz.pcm

It should be
/path/to/repos/webrtc-temp/src/resources/audio_coding/testfile32kHz.pcm.

The middle part is missing.

The reason this target is affected is because
webrtc::test::SetExecutablePath(argv[0]);
was not called.

That call is necessary for us to know that the test is being run from src/
and not from out/Default (as is assumed, when that function is not called.)

BUG=chromium:497757
R=kjellander@webrtc.org, henrik.lundin@webrtc.org

Review-Url: https://codereview.webrtc.org/2698743002
Cr-Commit-Position: refs/heads/master@{#16641}
Committed:
https://chromium.googlesource.com/external/webrtc/+/454c1d6a2305b7f0c8395b8e7...
==========

[commit-bot: I haz the power, 2017-02-16 11:54:55]

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