Convert accessibility_helper.mojom to snake casing

Convert accessibility_helper.mojom to snake casing
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2878503003
Cr-Commit-Position: refs/heads/master@{#471556}
Committed: https://chromium.googlesource.com/chromium/src/+/4c1b94d4d7d0dc79e8cf787c50317428dae76eb7

[David Tseng, 2017-05-10 21:46:56]

Description was changed from

==========
Convert to snake casing

BUG=
==========

to

==========
Convert accessibility_helper.mojom to snake casing
==========

[David Tseng, 2017-05-10 21:46:56]

dtseng@chromium.org changed reviewers:
+ dcheng@chromium.org

[David Tseng, 2017-05-10 21:49:10]

Thanks for catching this; I think the mojom file was first written from someone
developing on Android and I don't think we ever looked at the other mojom files
to get the proper mojo format. Verified that the same mojom file compiles
without code changes on Android, so I believe no versioning is needed as you
pointed out.

[dcheng, 2017-05-10 21:58:57]

mojo lgtm

[David Tseng, 2017-05-11 03:13:01]

dtseng@chromium.org changed reviewers:
+ yawano@chromium.org

[David Tseng, 2017-05-11 03:13:01]

+ yawano

[yawano, 2017-05-11 03:26:51]

c/b/chromeos/arc/accessibility lgtm. Thank you for fixing this!

[David Tseng, 2017-05-12 06:04:22]

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

[commit-bot: I haz the power, 2017-05-12 06:04:47]

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-05-12 07:03:53]

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

[commit-bot: I haz the power, 2017-05-12 07:03:54]

Dry run: 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_...)

[David Tseng, 2017-05-12 22:36:51]

Description was changed from

==========
Convert accessibility_helper.mojom to snake casing
==========

to

==========
Convert accessibility_helper.mojom to snake casing
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2017-05-12 22:37:50]

Patchset #2 (id:20001) has been deleted

[David Tseng, 2017-05-12 22:39:02]

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

[commit-bot: I haz the power, 2017-05-12 22:40:08]

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-05-12 23:21:11]

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

[commit-bot: I haz the power, 2017-05-12 23:21:12]

Dry run: This issue passed the CQ dry run.

[David Tseng, 2017-05-13 04:00:54]

The CQ bit was checked by dtseng@chromium.org

[David Tseng, 2017-05-13 04:00:55]

The patchset sent to the CQ was uploaded after l-g-t-m from dcheng@chromium.org,
yawano@chromium.org
Link to the patchset: https://codereview.chromium.org/2878503003/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2017-05-13 04:01:10]

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-05-13 04:06:32]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494648054765010,
"parent_rev": "55946f131f740c6a5ee9eb12995f06529c7d7b90", "commit_rev":
"4c1b94d4d7d0dc79e8cf787c50317428dae76eb7"}

[commit-bot: I haz the power, 2017-05-13 04:06:41]

Description was changed from

==========
Convert accessibility_helper.mojom to snake casing
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Convert accessibility_helper.mojom to snake casing
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2878503003
Cr-Commit-Position: refs/heads/master@{#471556}
Committed:
https://chromium.googlesource.com/chromium/src/+/4c1b94d4d7d0dc79e8cf787c5031...
==========

[commit-bot: I haz the power, 2017-05-13 04:06:42]

Committed patchset #2 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/4c1b94d4d7d0dc79e8cf787c5031...

[Luis Héctor Chávez, 2017-05-15 19:50:15]

On 2017/05/10 21:49:10, David Tseng wrote:
> Thanks for catching this; I think the mojom file was first written from
someone
> developing on Android and I don't think we ever looked at the other mojom
files
> to get the proper mojo format. Verified that the same mojom file compiles
> without code changes on Android, so I believe no versioning is needed as you
> pointed out.

FYI the names of the structs / methods / parameters are never sent through the
wire (as in with protobufs), so it's always safe to rename them.