content/public/common should depend on the mojo_bindings target

content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
causes multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
TBR=jam@chromium.org,rockot@chromium.org

Committed: https://crrev.com/7279505c6d3be5eb212e81a3b769d88e76d413e7
Cr-Commit-Position: refs/heads/master@{#401230}

[Mostyn Bramley-Moore, 2016-06-22 07:52:32]

The CQ bit was checked by mostynb@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-22 07:52:41]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2088163002/1

[Mostyn Bramley-Moore, 2016-06-22 07:53:52]

mostynb@opera.com changed reviewers:
+ jam@chromium.org, rockot@chromium.org

[Mostyn Bramley-Moore, 2016-06-22 07:53:52]

@jam, rockot: please take a look at this fixup.  Without this, gyp component
builds fail to link libcontent due to duplicate symbols.  I have yet to
investigate the GN side.

[Mostyn Bramley-Moore, 2016-06-22 08:01:37]

The blink side looks OK, since it uses a header target.

[Mostyn Bramley-Moore, 2016-06-22 08:06:24]

Description was changed from

==========
content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
caused multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
==========

to

==========
content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
causes multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
==========

[commit-bot: I haz the power, 2016-06-22 08:30:56]

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

[commit-bot: I haz the power, 2016-06-22 08:30:57]

Dry run: This issue passed the CQ dry run.

[Mostyn Bramley-Moore, 2016-06-22 08:31:57]

Description was changed from

==========
content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
causes multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
==========

to

==========
content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
causes multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
TBR=jam@chromium.org,rockot@chromium.org
==========

[Mostyn Bramley-Moore, 2016-06-22 08:32:22]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2016-06-22 08:32:42]

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

[commit-bot: I haz the power, 2016-06-22 08:36:56]

Description was changed from

==========
content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
causes multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
TBR=jam@chromium.org,rockot@chromium.org
==========

to

==========
content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
causes multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
TBR=jam@chromium.org,rockot@chromium.org
==========

[commit-bot: I haz the power, 2016-06-22 08:36:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-22 08:38:55]

Description was changed from

==========
content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
causes multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
TBR=jam@chromium.org,rockot@chromium.org
==========

to

==========
content/public/common should depend on the mojo_bindings target

Rather than the mojo_bindings_mojom target, which exports sources, and
causes multiple definition errors in component builds.

Followup to https://codereview.chromium.org/2089823002

BUG=622076
TBR=jam@chromium.org,rockot@chromium.org

Committed: https://crrev.com/7279505c6d3be5eb212e81a3b769d88e76d413e7
Cr-Commit-Position: refs/heads/master@{#401230}
==========

[commit-bot: I haz the power, 2016-06-22 08:38:55]

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