Add posciak and wuchengli to WebRTC HW decoder and encoder files

Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to content/renderer/media/gpu.

BUG=None
TEST=Build Chrome on ChromeOS

Committed: https://crrev.com/6f7ef8a5b3deacc6d2ab70f282ff93486947f5a1
Cr-Commit-Position: refs/heads/master@{#403168}

[wuchengli, 2016-06-27 14:16:42]

Description was changed from

==========
Add owners to rtc_video_decoder* and rtc_video_encoder*.

Add posciak@ and wuchengli@ as owners to rtc_video_decoder*
and rtc_video_encoder* files.

BUG=None
TEST=None
==========

to

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

BUG=None
TEST=None
==========

[wuchengli, 2016-06-27 14:17:22]

wuchengli@chromium.org changed reviewers:
+ mcasas@chromium.org, posciak@chromium.org

[mcasas, 2016-06-27 17:05:32]

https://codereview.chromium.org/2095393002/diff/1/content/renderer/media/OWNERS
File content/renderer/media/OWNERS (right):

https://codereview.chromium.org/2095393002/diff/1/content/renderer/media/OWNE...
content/renderer/media/OWNERS:18: per-file rtc_video_*=wuchengli@chromium.org
I'm not opposed to this change, but renderer/media
is getting a tad too large. I see that rtc_video_* files
are only included by themselves and by 
peer_connection_dependency_factory.cc, so
I'd propose moving the rtc_video_* files into a folder
on its own and creating an OWNERS file there.
(You can use tools/git/mass-rename.py which
should make this changes trivial). WDYT?

[wuchengli, 2016-06-28 03:54:58]

https://codereview.chromium.org/2095393002/diff/1/content/renderer/media/OWNERS
File content/renderer/media/OWNERS (right):

https://codereview.chromium.org/2095393002/diff/1/content/renderer/media/OWNE...
content/renderer/media/OWNERS:18: per-file rtc_video_*=wuchengli@chromium.org
On 2016/06/27 17:05:31, mcasas wrote:
> I'm not opposed to this change, but renderer/media
> is getting a tad too large. I see that rtc_video_* files
> are only included by themselves and by 
> peer_connection_dependency_factory.cc, so
> I'd propose moving the rtc_video_* files into a folder
> on its own and creating an OWNERS file there.
> (You can use tools/git/mass-rename.py which
> should make this changes trivial). WDYT?

Any suggestion for the folder path? How about content/renderer/media/gpu?

[mcasas, 2016-06-28 04:38:14]

On 2016/06/28 03:54:58, wuchengli wrote:
>
https://codereview.chromium.org/2095393002/diff/1/content/renderer/media/OWNERS
> File content/renderer/media/OWNERS (right):
> 
>
https://codereview.chromium.org/2095393002/diff/1/content/renderer/media/OWNE...
> content/renderer/media/OWNERS:18: per-file
mailto:rtc_video_*=wuchengli@chromium.org
> On 2016/06/27 17:05:31, mcasas wrote:
> > I'm not opposed to this change, but renderer/media
> > is getting a tad too large. I see that rtc_video_* files
> > are only included by themselves and by 
> > peer_connection_dependency_factory.cc, so
> > I'd propose moving the rtc_video_* files into a folder
> > on its own and creating an OWNERS file there.
> > (You can use tools/git/mass-rename.py which
> > should make this changes trivial). WDYT?
> 
> Any suggestion for the folder path? How about content/renderer/media/gpu?

Or .../rtc_video, whichever you prefer

[wuchengli, 2016-06-29 05:48:30]

Description was changed from

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

BUG=None
TEST=None
==========

to

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to
content/renderer/media/gpu.

BUG=None
TEST=None
==========

[wuchengli, 2016-06-29 05:48:45]

Description was changed from

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to
content/renderer/media/gpu.

BUG=None
TEST=None
==========

to

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to content/renderer/media/gpu.

BUG=None
TEST=None
==========

[wuchengli, 2016-06-29 05:49:05]

Description was changed from

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to content/renderer/media/gpu.

BUG=None
TEST=None
==========

to

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to content/renderer/media/gpu.

BUG=None
TEST=Build Chrome on ChromeOS
==========

[wuchengli, 2016-06-29 15:01:42]

The CQ bit was checked by wuchengli@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-29 15:02:26]

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

[wuchengli, 2016-06-29 15:05:24]

The CQ bit was unchecked by wuchengli@chromium.org

[wuchengli, 2016-06-29 15:06:03]

Patchset #2 (id:20001) has been deleted

[wuchengli, 2016-06-29 15:06:14]

The CQ bit was checked by wuchengli@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-29 15:06:46]

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

[wuchengli, 2016-06-29 15:06:54]

Moved to content/renderer/media/gpu. PTAL.

[mcasas, 2016-06-29 15:09:43]

RS LGTM % my comment.

https://codereview.chromium.org/2095393002/diff/40001/content/renderer/media/...
File content/renderer/media/gpu/OWNERS (right):

https://codereview.chromium.org/2095393002/diff/40001/content/renderer/media/...
content/renderer/media/gpu/OWNERS:3: miu@chromium.org
OWNERS are hierarchical and most of these are in 
content/renderer/media/OWNERS, so just put
here the additions.

[wuchengli, 2016-06-29 15:32:55]

https://codereview.chromium.org/2095393002/diff/40001/content/renderer/media/...
File content/renderer/media/gpu/OWNERS (right):

https://codereview.chromium.org/2095393002/diff/40001/content/renderer/media/...
content/renderer/media/gpu/OWNERS:3: miu@chromium.org
On 2016/06/29 15:09:43, mcasas wrote:
> OWNERS are hierarchical and most of these are in 
> content/renderer/media/OWNERS, so just put
> here the additions.
Oh right. Done. I also removed file://media/OWNERS.

[wuchengli, 2016-06-29 15:33:02]

The CQ bit was checked by wuchengli@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-29 15:33:31]

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

[commit-bot: I haz the power, 2016-06-29 16:21:53]

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

[commit-bot: I haz the power, 2016-06-29 16:21:55]

Dry run: This issue passed the CQ dry run.

[wuchengli, 2016-06-30 14:04:55]

The CQ bit was checked by wuchengli@chromium.org

[wuchengli, 2016-06-30 14:04:56]

The patchset sent to the CQ was uploaded after l-g-t-m from mcasas@chromium.org
Link to the patchset: https://codereview.chromium.org/2095393002/#ps60001
(title: "remove duplicated owners")

[commit-bot: I haz the power, 2016-06-30 14:05:16]

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

[commit-bot: I haz the power, 2016-06-30 14:09:34]

Description was changed from

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to content/renderer/media/gpu.

BUG=None
TEST=Build Chrome on ChromeOS
==========

to

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to content/renderer/media/gpu.

BUG=None
TEST=Build Chrome on ChromeOS
==========

[commit-bot: I haz the power, 2016-06-30 14:09:35]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-06-30 14:09:39]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 14:10:50]

Description was changed from

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to content/renderer/media/gpu.

BUG=None
TEST=Build Chrome on ChromeOS
==========

to

==========
Add posciak and wuchengli to WebRTC HW decoder and encoder files

Covers:
rtc_video_decoder.cc
rtc_video_decoder_factory.cc
rtc_video_decoder_factory.h
rtc_video_decoder.h
rtc_video_decoder_unittest.cc
rtc_video_encoder.cc
rtc_video_encoder_factory.cc
rtc_video_encoder_factory.h
rtc_video_encoder.h

The above files have been moved to content/renderer/media/gpu.

BUG=None
TEST=Build Chrome on ChromeOS

Committed: https://crrev.com/6f7ef8a5b3deacc6d2ab70f282ff93486947f5a1
Cr-Commit-Position: refs/heads/master@{#403168}
==========

[commit-bot: I haz the power, 2016-06-30 14:10:51]

Patchset 3 (id:??) landed as
https://crrev.com/6f7ef8a5b3deacc6d2ab70f282ff93486947f5a1
Cr-Commit-Position: refs/heads/master@{#403168}