Mojo C++ bindings: disallow copy and assign for generated classes when necessary.

Mojo C++ bindings: disallow copy and assign for generated classes when necessary.

On windows, if a generated class contains non-copyable members and we export it without explicitly disallowing copy and assign, compilation will fail in some cases. (An example could be found in test_export.mojom.)

BUG=None

Committed: https://crrev.com/9e060aa1a208f152f7a022879c95d3810281a75a
Cr-Commit-Position: refs/heads/master@{#412051}

[yzshen1, 2016-08-15 19:29:01]

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

[commit-bot: I haz the power, 2016-08-15 19:29:41]

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

[yzshen1, 2016-08-15 19:58:02]

Description was changed from

==========
Mojo C++ bindings: disallow copy and assign for generated classes when
necessary.

On windows, if a generated class contains non-copyable members, and we export it
without explicitly disallowing copy and assign operators, compilation will fail.

BUG=None
==========

to

==========
Mojo C++ bindings: disallow copy and assign for generated classes when
necessary.

On windows, if a generated class contains non-copyable members and we export it
without explicitly disallowing copy and assign, compilation will fail in some
cases. (An example could be found in test_export.mojom.)

BUG=None
==========

[yzshen1, 2016-08-15 19:58:13]

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

[commit-bot: I haz the power, 2016-08-15 19:58:39]

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

[yzshen1, 2016-08-15 20:00:44]

yzshen@chromium.org changed reviewers:
+ darin@chromium.org

[yzshen1, 2016-08-15 20:00:45]

Hi, Darin.

Would you please take a look? Thanks!

[darin (slow to review), 2016-08-15 21:00:41]

LGTM

[commit-bot: I haz the power, 2016-08-15 21:02:15]

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

[commit-bot: I haz the power, 2016-08-15 21:02:16]

Dry run: This issue passed the CQ dry run.

[yzshen1, 2016-08-15 21:11:01]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-08-15 21:11: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-08-15 21:15:01]

Description was changed from

==========
Mojo C++ bindings: disallow copy and assign for generated classes when
necessary.

On windows, if a generated class contains non-copyable members and we export it
without explicitly disallowing copy and assign, compilation will fail in some
cases. (An example could be found in test_export.mojom.)

BUG=None
==========

to

==========
Mojo C++ bindings: disallow copy and assign for generated classes when
necessary.

On windows, if a generated class contains non-copyable members and we export it
without explicitly disallowing copy and assign, compilation will fail in some
cases. (An example could be found in test_export.mojom.)

BUG=None
==========

[commit-bot: I haz the power, 2016-08-15 21:15:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-15 21:23:10]

Description was changed from

==========
Mojo C++ bindings: disallow copy and assign for generated classes when
necessary.

On windows, if a generated class contains non-copyable members and we export it
without explicitly disallowing copy and assign, compilation will fail in some
cases. (An example could be found in test_export.mojom.)

BUG=None
==========

to

==========
Mojo C++ bindings: disallow copy and assign for generated classes when
necessary.

On windows, if a generated class contains non-copyable members and we export it
without explicitly disallowing copy and assign, compilation will fail in some
cases. (An example could be found in test_export.mojom.)

BUG=None

Committed: https://crrev.com/9e060aa1a208f152f7a022879c95d3810281a75a
Cr-Commit-Position: refs/heads/master@{#412051}
==========

[commit-bot: I haz the power, 2016-08-15 21:23:11]

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