Tracking mock_process_thread with a GN target

Tracking mock_process_thread with a GN target

include/mock/mock_process_thread.h was not tracked by GN.

This cl creates a target for it. The target is testonly because it
depends on "webrtc/test:rtp_test_utils".

This means that dependencies to this header cannot fly under the
GN radar anymore. :)

BUG=webrtc:7652
NOTRY=True

Review-Url: https://codereview.webrtc.org/2881343003
Cr-Commit-Position: refs/heads/master@{#19234}
Committed: https://chromium.googlesource.com/external/webrtc/+/5166e54a3dbcdad0da6fe726824fd50f688dd512

[mbonadei, 2017-05-16 12:45:08]

mbonadei@webrtc.org changed reviewers:
+ perkj@chromium.org

[perkj_chrome, 2017-07-06 07:44:57]

Wrong account?

[mbonadei, 2017-07-06 08:00:12]

On 2017/07/06 07:44:57, perkj_chrome wrote:
> Wrong account?

You are the owner of webrtc/modules/utility. :-)

This is a small change, just a FYI. Changes in the webrtc/call and webrtc/video
BUILD.gn files
are an explicit declaration of a dependency on the newly added target in
webrtc/modules/utility.

[perkj_webrtc, 2017-07-06 08:01:32]

Description was changed from

==========
Tracking mock_process_thread with a GN target

include/mock/mock_process_thread.h was not tracked by GN.

This cl creates a target for it. The target is testonly because it
depends on "webrtc/test:rtp_test_utils".

This means that dependencies to this header cannot fly under the
GN radar anymore. :)

BUG=webrtc:7652
==========

to

==========
Tracking mock_process_thread with a GN target

include/mock/mock_process_thread.h was not tracked by GN.

This cl creates a target for it. The target is testonly because it
depends on "webrtc/test:rtp_test_utils".

This means that dependencies to this header cannot fly under the
GN radar anymore. :)

BUG=webrtc:7652
==========

[perkj_webrtc, 2017-07-06 08:01:32]

perkj@webrtc.org changed reviewers:
+ perkj@webrtc.org
- perkj@chromium.org

[perkj_webrtc, 2017-07-06 08:01:57]

Changed account to webrtc.

lgtm

[mbonadei, 2017-07-06 08:27:48]

On 2017/07/06 08:01:57, perkj_webrtc wrote:
> Changed account to webrtc.
> 
> lgtm

Oh yes... I didn't get I selected your wrong account. :)

[mbonadei, 2017-07-06 13:18:33]

The CQ bit was checked by mbonadei@webrtc.org

[mbonadei, 2017-07-06 13:18:33]

The patchset sent to the CQ was uploaded after l-g-t-m from perkj@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2881343003/#ps20001 (title:
"Rebasing")

[commit-bot: I haz the power, 2017-07-06 13:18:38]

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-07-06 13:48:52]

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

[commit-bot: I haz the power, 2017-07-06 13:48:53]

Try jobs failed on following builders:
  linux_memcheck on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_memcheck/builds/9450)

[mbonadei, 2017-07-26 09:11:06]

Description was changed from

==========
Tracking mock_process_thread with a GN target

include/mock/mock_process_thread.h was not tracked by GN.

This cl creates a target for it. The target is testonly because it
depends on "webrtc/test:rtp_test_utils".

This means that dependencies to this header cannot fly under the
GN radar anymore. :)

BUG=webrtc:7652
==========

to

==========
Tracking mock_process_thread with a GN target

include/mock/mock_process_thread.h was not tracked by GN.

This cl creates a target for it. The target is testonly because it
depends on "webrtc/test:rtp_test_utils".

This means that dependencies to this header cannot fly under the
GN radar anymore. :)

BUG=webrtc:7652
NOTRY=True
==========

[mbonadei, 2017-08-03 12:54:26]

The CQ bit was checked by mbonadei@webrtc.org

[mbonadei, 2017-08-03 12:54:26]

The patchset sent to the CQ was uploaded after l-g-t-m from perkj@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2881343003/#ps80001 (title:
"Rebasing (again)")

[commit-bot: I haz the power, 2017-08-03 12:54:38]

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-08-03 12:57:12]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1501764866028860,
"parent_rev": "7b7e06fd23ac67d81f378b773bb631abb1d82116", "commit_rev":
"5166e54a3dbcdad0da6fe726824fd50f688dd512"}

[commit-bot: I haz the power, 2017-08-03 12:57:18]

Description was changed from

==========
Tracking mock_process_thread with a GN target

include/mock/mock_process_thread.h was not tracked by GN.

This cl creates a target for it. The target is testonly because it
depends on "webrtc/test:rtp_test_utils".

This means that dependencies to this header cannot fly under the
GN radar anymore. :)

BUG=webrtc:7652
NOTRY=True
==========

to

==========
Tracking mock_process_thread with a GN target

include/mock/mock_process_thread.h was not tracked by GN.

This cl creates a target for it. The target is testonly because it
depends on "webrtc/test:rtp_test_utils".

This means that dependencies to this header cannot fly under the
GN radar anymore. :)

BUG=webrtc:7652
NOTRY=True

Review-Url: https://codereview.webrtc.org/2881343003
Cr-Commit-Position: refs/heads/master@{#19234}
Committed:
https://chromium.googlesource.com/external/webrtc/+/5166e54a3dbcdad0da6fe7268...
==========

[commit-bot: I haz the power, 2017-08-03 12:57:19]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/external/webrtc/+/5166e54a3dbcdad0da6fe7268...