Mojo C++ bindings: switch WebKit mojom targets to use STL/WTF types.

Mojo C++ bindings: switch WebKit mojom targets to use STL/WTF types.

BUG=624136
Committed: https://crrev.com/6b5ad7f20127c0be5ebbc168b8e4cebb38446e26
Cr-Commit-Position: refs/heads/master@{#433988}

[yzshen1, 2016-11-21 19:02:47]

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

[commit-bot: I haz the power, 2016-11-21 19:03:44]

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

[yzshen1, 2016-11-21 19:13:45]

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

[commit-bot: I haz the power, 2016-11-21 19:13:53]

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-11-21 19:22:55]

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

[commit-bot: I haz the power, 2016-11-21 19:22:56]

Dry run: Try jobs failed on following builders:
  blimp_linux_dbg on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/blimp_linux_dbg...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[yzshen1, 2016-11-21 20:33:30]

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

[commit-bot: I haz the power, 2016-11-21 20:34:21]

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

[yzshen1, 2016-11-21 20:34:36]

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

[yzshen1, 2016-11-21 20:34:37]

Hi, Jochen.

Would you please take a look? Thanks!

[commit-bot: I haz the power, 2016-11-21 21:07:29]

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

[commit-bot: I haz the power, 2016-11-21 21:07:30]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[yzshen1, 2016-11-21 22:02:28]

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

[commit-bot: I haz the power, 2016-11-21 22:03:24]

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-11-21 22:42:17]

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

[commit-bot: I haz the power, 2016-11-21 22:42:18]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[yzshen1, 2016-11-21 23:09:23]

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

[commit-bot: I haz the power, 2016-11-21 23:09:58]

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

[yzshen1, 2016-11-21 23:10:40]

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

[commit-bot: I haz the power, 2016-11-21 23:10:59]

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-11-21 23:29:56]

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

[commit-bot: I haz the power, 2016-11-21 23:29:57]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yzshen1, 2016-11-21 23:41:57]

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

[commit-bot: I haz the power, 2016-11-21 23:42:40]

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-11-22 00:09:00]

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

[commit-bot: I haz the power, 2016-11-22 00:09:01]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[haraken, 2016-11-22 00:50:29]

WebKit/ LGTM

[yzshen1, 2016-11-22 17:43:48]

Description was changed from

==========
Mojo C++ bindings: switch WebKit mojom targets to use STL/WTF types.

BUG=624136
==========

to

==========
Mojo C++ bindings: switch WebKit mojom targets to use STL/WTF types.

BUG=624136
==========

[yzshen1, 2016-11-22 17:43:48]

yzshen@chromium.org changed reviewers:
+ ben.yan@msn.com

[yzshen1, 2016-11-22 17:44:09]

yzshen@chromium.org changed reviewers:
+ ben@chromium.org
- ben.yan@msn.com

[yzshen1, 2016-11-22 17:47:35]

On 2016/11/22 00:50:29, haraken wrote:
> WebKit/ LGTM

+Ben for content/ owner review. Thanks!


(Jochen seems to be busy according to Calendar.)

[Ben Goodger (Google), 2016-11-22 18:00:44]

content lgtm

[yzshen1, 2016-11-22 18:02:57]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-11-22 18:03:19]

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-11-22 18:07:39]

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

[commit-bot: I haz the power, 2016-11-22 18:07:40]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yzshen1, 2016-11-22 18:53:21]

The CQ bit was checked by yzshen@chromium.org

[yzshen1, 2016-11-22 18:53:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, ben@chromium.org
Link to the patchset: https://codereview.chromium.org/2522553003/#ps140001
(title: ".")

[commit-bot: I haz the power, 2016-11-22 18:53:41]

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-11-22 19:28:19]

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

[commit-bot: I haz the power, 2016-11-22 19:28:20]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[yzshen1, 2016-11-22 19:34:37]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-11-22 19:34:57]

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-11-22 21:38:24]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1479843277438390,
"parent_rev": "73c4ef90a175d01e0a6584047be930b4356d27ca", "commit_rev":
"4fadcb6754afcd0feb66193d416e71c60219e4ae"}

[commit-bot: I haz the power, 2016-11-22 21:38:50]

Description was changed from

==========
Mojo C++ bindings: switch WebKit mojom targets to use STL/WTF types.

BUG=624136
==========

to

==========
Mojo C++ bindings: switch WebKit mojom targets to use STL/WTF types.

BUG=624136
==========

[commit-bot: I haz the power, 2016-11-22 21:38:52]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2016-11-22 21:41:51]

Description was changed from

==========
Mojo C++ bindings: switch WebKit mojom targets to use STL/WTF types.

BUG=624136
==========

to

==========
Mojo C++ bindings: switch WebKit mojom targets to use STL/WTF types.

BUG=624136

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

[commit-bot: I haz the power, 2016-11-22 21:41:52]

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

[kolos1, 2016-11-24 12:11:50]

A revert of this CL (patchset #8 id:140001) has been created in
https://codereview.chromium.org/2525243003/ by kolos@chromium.org.

The reason for reverting is: This CL causes mojo failures on ChromeOS. See the
bug for details.

BUG=668420.