Move ViewHostMsg_CreateWidget to mojom

Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500
Committed: https://crrev.com/ae1560a11b938a67fe7ef11f23268617e0915d5e
Cr-Commit-Position: refs/heads/master@{#429139}

[Jay Civelli, 2016-10-27 22:46:16]

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

[commit-bot: I haz the power, 2016-10-27 22:47:25]

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

[Jay Civelli, 2016-10-27 22:47:33]

Description was changed from

==========
Move ViewHostMsg_CreateWidget to mojom

Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500
==========

to

==========
Move ViewHostMsg_CreateWidget to mojom

Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500
==========

[Jay Civelli, 2016-10-27 22:47:33]

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

[Jay Civelli, 2016-10-27 22:47:44]

Description was changed from

==========
Move ViewHostMsg_CreateWidget to mojom

Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500
==========

to

==========
Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500
==========

[commit-bot: I haz the power, 2016-10-27 22:53:37]

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

[commit-bot: I haz the power, 2016-10-27 22:53:37]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  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_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Jay Civelli, 2016-10-28 05:53:30]

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

[commit-bot: I haz the power, 2016-10-28 05:53:45]

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-10-28 07:08:59]

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

[commit-bot: I haz the power, 2016-10-28 07:08:59]

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...)

[Ken Rockot(use gerrit already), 2016-10-31 19:44:44]

lgtm

[Jay Civelli, 2016-10-31 20:06:11]

The CQ bit was checked by jcivelli@chromium.org

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

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-10-31 20:16:12]

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

[commit-bot: I haz the power, 2016-10-31 20:16:14]

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

[Jay Civelli, 2016-10-31 20:26:01]

jcivelli@chromium.org changed reviewers:
+ jam@chromium.org

[Jay Civelli, 2016-10-31 20:26:24]

Adding @jam for content/ OWNER review.

Thanks!

[jam, 2016-11-01 17:11:45]

lgtm

https://codereview.chromium.org/2450353004/diff/60001/content/browser/rendere...
File content/browser/renderer_host/render_message_filter.cc (right):

https://codereview.chromium.org/2450353004/diff/60001/content/browser/rendere...
content/browser/renderer_host/render_message_filter.cc:303: {
nit: bracket on previous line per style guide

https://codereview.chromium.org/2450353004/diff/60001/content/browser/rendere...
content/browser/renderer_host/render_message_filter.cc:304: int route_id = 0;
nit: MSG_ROUTING_NONE since 0 is a valid routing ID

[Jay Civelli, 2016-11-01 20:47:29]

https://codereview.chromium.org/2450353004/diff/60001/content/browser/rendere...
File content/browser/renderer_host/render_message_filter.cc (right):

https://codereview.chromium.org/2450353004/diff/60001/content/browser/rendere...
content/browser/renderer_host/render_message_filter.cc:303: {
On 2016/11/01 17:11:45, jam wrote:
> nit: bracket on previous line per style guide

Done.

https://codereview.chromium.org/2450353004/diff/60001/content/browser/rendere...
content/browser/renderer_host/render_message_filter.cc:304: int route_id = 0;
On 2016/11/01 17:11:45, jam wrote:
> nit: MSG_ROUTING_NONE since 0 is a valid routing ID

Done.

[Jay Civelli, 2016-11-01 20:47:34]

The CQ bit was checked by jcivelli@chromium.org

[Jay Civelli, 2016-11-01 20:47:34]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org,
jam@chromium.org
Link to the patchset: https://codereview.chromium.org/2450353004/#ps80001
(title: "Addressed jam's comments + sync")

[commit-bot: I haz the power, 2016-11-01 20:48:07]

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-01 20:56:40]

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

[commit-bot: I haz the power, 2016-11-01 20:56:41]

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

[Jay Civelli, 2016-11-01 21:01:50]

jcivelli@chromium.org changed reviewers:
+ tsepez@chromium.org

[Jay Civelli, 2016-11-01 21:03:03]

Adding @tsepez for SECURITY_OWNER review of content/common/*_messages.h and
content/common/*.mojom

Thanks.

[Tom Sepez, 2016-11-01 21:06:13]

lgtm

https://codereview.chromium.org/2450353004/diff/80001/content/common/native_t...
File content/common/native_types.typemap (right):

https://codereview.chromium.org/2450353004/diff/80001/content/common/native_t...
content/common/native_types.typemap:46:
"content.mojom.WebPopupType=blink::WebPopupType",
nit: alphabetize.

[Jay Civelli, 2016-11-01 21:08:41]

https://codereview.chromium.org/2450353004/diff/80001/content/common/native_t...
File content/common/native_types.typemap (right):

https://codereview.chromium.org/2450353004/diff/80001/content/common/native_t...
content/common/native_types.typemap:46:
"content.mojom.WebPopupType=blink::WebPopupType",
On 2016/11/01 21:06:13, Tom Sepez wrote:
> nit: alphabetize.

Done.

[Jay Civelli, 2016-11-01 21:08:48]

The CQ bit was checked by jcivelli@chromium.org

[Jay Civelli, 2016-11-01 21:08:48]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
rockot@chromium.org, jam@chromium.org
Link to the patchset: https://codereview.chromium.org/2450353004/#ps100001
(title: "Addressed @tsepez's comment")

[commit-bot: I haz the power, 2016-11-01 21:09:31]

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-01 22:40:50]

Description was changed from

==========
Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500
==========

to

==========
Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500
==========

[commit-bot: I haz the power, 2016-11-01 22:40:51]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-11-01 22:42:35]

Description was changed from

==========
Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500
==========

to

==========
Converts the ViewHostMsg_CreateWidget message to mojom.

BUG=612500

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

[commit-bot: I haz the power, 2016-11-01 22:42:36]

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