Re-Reland: Make media/capture gn and gyps produce components

Re-Reland: Make media/capture gn and gyps produce components

The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ).
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.

Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/compile/logs/stdio

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a
rebase :)

Committed: https://crrev.com/a0f0c4ba5a7fd23448342a879d0a306b441af216
Cr-Commit-Position: refs/heads/master@{#400708}

[mcasas, 2016-06-20 14:58:49]

Description was changed from

==========
[WIP] Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). Still figuring
this one out.

Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
==========

to

==========
[WIP] Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
==========

[mcasas, 2016-06-20 15:01:58]

Description was changed from

==========
[WIP] Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
==========

to

==========
[WIP] Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-disabling that was lost in a rebase :)
==========

[mcasas, 2016-06-20 15:02:17]

Description was changed from

==========
[WIP] Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-disabling that was lost in a rebase :)
==========

to

==========
[WIP] Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a 
rebase :)
==========

[mcasas, 2016-06-20 16:01:39]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-06-20 16:02:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2068763002/20001

[Ken Rockot(use gerrit already), 2016-06-20 16:28:14]

rockot@chromium.org changed reviewers:
+ rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-06-20 16:28:15]

Could you get rid of the "WIP" before landing this?

[mcasas, 2016-06-20 16:33:21]

Description was changed from

==========
[WIP] Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a 
rebase :)
==========

to

==========
Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a 
rebase :)
==========

[mcasas, 2016-06-20 16:33:45]

On 2016/06/20 16:28:15, Ken Rockot wrote:
> Could you get rid of the "WIP" before landing this?

Done :)

[mcasas, 2016-06-20 16:34:52]

Description was changed from

==========
Re-Reland: Make media/capture gn and gyps produce components


[WIP]. The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a 
rebase :)
==========

to

==========
Re-Reland: Make media/capture gn and gyps produce components


The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a 
rebase :)
==========

[commit-bot: I haz the power, 2016-06-20 17:28:50]

Description was changed from

==========
Re-Reland: Make media/capture gn and gyps produce components


The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a 
rebase :)
==========

to

==========
Re-Reland: Make media/capture gn and gyps produce components


The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a 
rebase :)
==========

[commit-bot: I haz the power, 2016-06-20 17:28:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-20 17:31:05]

Description was changed from

==========
Re-Reland: Make media/capture gn and gyps produce components


The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ). 
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.



Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a 
rebase :)
==========

to

==========
Re-Reland: Make media/capture gn and gyps produce components

The CL is ready but causes a Warning C4267 (implicit size_t to int
conversion) in some Windows Waterfall bots (not in CQ).
After some land-archaeology, it seems that capture.gyp did not
have the warning explicitly disable, I believe it was lost in some
rebase. So, re-enabling it and relanding.

Original re-reland description -----------------------------------------
Re-reland: Make media/capture gn and gyps produce components

Reland got reverted due to a WinX64 typical issue:
warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of
data

This CL adds explicitly some buid/config/compiler/no_size_t_to_int_warning
causing C4267 in
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1472/steps/co...

Original reland description -------------------------------------------
Reland: Make media/capture gn and gyps produce components

Which got reverted due to a Mac build breakage (see
https://codereview.chromium.org/2059493002/) and
a Win one  (https://crbug.com/618854).

Original description ---------------------------------------------------
Make media/capture gn and gyps produce components
This CL separates media/capture into a component,
to allow, in a subsequent CL, linking with mojo for using
ImageCapture types.

This basically reproduces the media/cast and media/midi
structure.

Windows import/export issues made necessary to add
capture_export and associated CAPTURE_EXPORT macro,
again, replicating media/midi.

gn gen ... --check uncovered some transitive dependencies
during the "analyze" phase of the bots and I had to make
them explicit (e.g. media:gpu).

BUG=518807
-----------------------------------------------------------------------
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel,mac_blink_rel,win_blink_rel

TBR=avi@chromium.org, xhwang@chromium.org, miu@chromium.org
rationale:
reland: since the PS2/3 are trivial, just making explicit libs that are
already elsewhere in media/ build files- I wonder why it didn't fail in
the bots :( .
re-reland: I'm just adding a MSVS-warning disable that was lost in a
rebase :)

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

[commit-bot: I haz the power, 2016-06-20 17:31:06]

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

[emircan, 2016-06-21 04:54:35]

emircan@chromium.org changed reviewers:
+ emircan@chromium.org

[emircan, 2016-06-21 04:54:36]

https://codereview.chromium.org/2068763002/diff/20001/media/capture/video/win...
File media/capture/video/win/filter_base_win.cc (right):

https://codereview.chromium.org/2068763002/diff/20001/media/capture/video/win...
media/capture/video/win/filter_base_win.cc:42: IPin* pin =
filter_->GetPin(static_cast<int>(index_));
This would pre-increment and cause a different behavior than before.

[emircan, 2016-06-21 05:06:15]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2081653004/ by emircan@chromium.org.

The reason for reverting is: Speculative revert: Appears to cause capture issues
in Win7 tester using actual cameras. See below for more detail.

https://bugs.chromium.org/p/chromium/issues/detail?id=621783.