Mojofy GpuHostMsg_FieldTrialActivated

Mojofy GpuHostMsg_FieldTrialActivated

Also, a little bit of cleaning mojo interface registration in
gpu_process_host.

BUG=698284
Review-Url: https://codereview.chromium.org/2803743002
Cr-Commit-Position: refs/heads/master@{#462649}
Committed: https://chromium.googlesource.com/chromium/src/+/701fa21b5ccf60143b103799f83cae7c51e6b5ee

[chiniforooshan, 2017-04-05 18:26:42]

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

[commit-bot: I haz the power, 2017-04-05 18:27:35]

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, 2017-04-05 18:31:59]

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

[commit-bot: I haz the power, 2017-04-05 18:32:00]

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

[chiniforooshan, 2017-04-05 18:36:49]

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

[commit-bot: I haz the power, 2017-04-05 18:37:26]

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

[chiniforooshan, 2017-04-05 20:37:13]

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

[commit-bot: I haz the power, 2017-04-05 20:37:53]

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

[chiniforooshan, 2017-04-05 20:43:15]

chiniforooshan@chromium.org changed reviewers:
+ sadrul@chromium.org

[chiniforooshan, 2017-04-05 20:43:16]

PTAL

[commit-bot: I haz the power, 2017-04-05 20:43:44]

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

[commit-bot: I haz the power, 2017-04-05 20:43:44]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[chiniforooshan, 2017-04-05 23:13:40]

I think the ios-simulator failure is because the bot ran out of space.

[sadrul, 2017-04-06 04:34:23]

LGTM!

https://codereview.chromium.org/2803743002/diff/40001/content/common/BUILD.gn
File content/common/BUILD.gn (left):

https://codereview.chromium.org/2803743002/diff/40001/content/common/BUILD.gn...
content/common/BUILD.gn:153: "gpu_host_messages.h",
yay!!

[chiniforooshan, 2017-04-06 14:17:05]

chiniforooshan@chromium.org changed reviewers:
+ piman@chromium.org, tsepez@google.com

[chiniforooshan, 2017-04-06 14:17:05]

Thanks.

+Antoine (content owner)
+Tom (security)

https://codereview.chromium.org/2803743002/diff/40001/content/common/BUILD.gn
File content/common/BUILD.gn (left):

https://codereview.chromium.org/2803743002/diff/40001/content/common/BUILD.gn...
content/common/BUILD.gn:153: "gpu_host_messages.h",
On 2017/04/06 04:34:23, sadrul wrote:
> yay!!

:D

[chiniforooshan, 2017-04-06 14:17:58]

chiniforooshan@chromium.org changed reviewers:
+ tsepez@chromium.org
- tsepez@google.com

[Tom Sepez, 2017-04-06 16:59:06]

LGTM otherwise.

https://codereview.chromium.org/2803743002/diff/40001/content/common/gpu_host...
File content/common/gpu_host_messages.h (left):

https://codereview.chromium.org/2803743002/diff/40001/content/common/gpu_host...
content/common/gpu_host_messages.h:15: #define IPC_MESSAGE_START GpuMsgStart
Can you remove the #define of GpuMsgStart in ipc/ipc_message_start.h

[chiniforooshan, 2017-04-06 17:04:36]

https://codereview.chromium.org/2803743002/diff/40001/content/common/gpu_host...
File content/common/gpu_host_messages.h (left):

https://codereview.chromium.org/2803743002/diff/40001/content/common/gpu_host...
content/common/gpu_host_messages.h:15: #define IPC_MESSAGE_START GpuMsgStart
On 2017/04/06 16:59:05, Tom Sepez wrote:
> Can you remove the #define of GpuMsgStart in ipc/ipc_message_start.h

Done.

[piman, 2017-04-06 20:35:33]

lgtm

[chiniforooshan, 2017-04-06 20:36:54]

The CQ bit was checked by chiniforooshan@chromium.org

[chiniforooshan, 2017-04-06 20:36:55]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2803743002/#ps60001
(title: "review")

[commit-bot: I haz the power, 2017-04-06 20:37:39]

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, 2017-04-06 22:22:59]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1491511014809950,
"parent_rev": "be9dfc362a8b2b90e054b5142c55c28334a6bded", "commit_rev":
"701fa21b5ccf60143b103799f83cae7c51e6b5ee"}

[commit-bot: I haz the power, 2017-04-06 22:24:10]

Description was changed from

==========
Mojofy GpuHostMsg_FieldTrialActivated

Also, a little bit of cleaning mojo interface registration in
gpu_process_host.

BUG=698284
==========

to

==========
Mojofy GpuHostMsg_FieldTrialActivated

Also, a little bit of cleaning mojo interface registration in
gpu_process_host.

BUG=698284

Review-Url: https://codereview.chromium.org/2803743002
Cr-Commit-Position: refs/heads/master@{#462649}
Committed:
https://chromium.googlesource.com/chromium/src/+/701fa21b5ccf60143b103799f83c...
==========

[commit-bot: I haz the power, 2017-04-06 22:24:12]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/701fa21b5ccf60143b103799f83c...