s/\bArc\b/ARC/g for ARC related code.

s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

TBR=oshima@chromium.org, xiyuan@chromium.org, chrisha@chromium.org, isherman@chromium.org, stevenjb@chromium.org, maxbogue@chromium.org, skuhne@chromium.org, benwells@chromium.org, msw@chromium.org, steel@chromium.org, dcheng@chromium.org
BUG=None
TEST=Ran bots.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2700783002
Cr-Commit-Position: refs/heads/master@{#451267}
Committed: https://chromium.googlesource.com/chromium/src/+/5f6e56e59eb16be722d9424dafa03d19a2aa6697

[hidehiko, 2017-02-16 13:07:46]

Description was changed from

==========
s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

BUG=None
TEST=Ran bots.
==========

to

==========
s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

BUG=None
TEST=Ran bots.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[hidehiko, 2017-02-16 13:08:07]

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

[commit-bot: I haz the power, 2017-02-16 13:08:18]

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

[hidehiko, 2017-02-16 13:17:44]

hidehiko@chromium.org changed reviewers:
+ lhchavez@chromium.org

[hidehiko, 2017-02-16 13:17:45]

I'm very sure this is a boring CL... but, I was asked to replace existing (not
related but linewise-closer) "Arc" by "ARC" many times in the code review.
I'd like to avoid spending time for such thing later, so to fix them at once.
WDYT, Luis?

If you agree, I'll add OWNERs, but maybe submit with TBR because the CL totally
hits
https://chromium.googlesource.com/chromium/src/+/master/docs/code_reviews.md#...
and
https://chromium.googlesource.com/chromium/src/+/master/docs/code_reviews.md#...

Thank you for review in advance,
- hidehiko

[commit-bot: I haz the power, 2017-02-16 15:18:58]

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

[commit-bot: I haz the power, 2017-02-16 15:18:59]

Dry run: This issue passed the CQ dry run.

[Luis Héctor Chávez, 2017-02-16 18:20:04]

thank you very much for doing this :D I pre-emptively added an entry in the
style guide to avoid this in the future.

The plan to TBR sgtm.

Have an early lgtm because the following are all nits:

With your patch applied, ran `git grep '"[^"]*\bArc\b'` and `git grep
'//.*\bArc\b'`, and only two files seem to be missing:

tools/metrics/histograms/histograms.xml (there is one reference to "Arc++" and
another to "Arc Package"), but I'm not sure if changing those is safe. Happy to
leave this one out of this change.
components/policy/proto/device_management_backend.proto There are two comments
there that can be changed safely.

I also ran `git grep -i '//.*arc++'` and `git grep -i '"[^"]*arc++'` and the
following files have references to "ARC++" in comments:

ash/common/system/chromeos/audio/tray_audio.h
chrome/browser/chromeos/app_mode/arc/arc_kiosk_app_service.cc
chrome/browser/chromeos/login/arc_kiosk_controller.cc
chrome/browser/chromeos/policy/dm_token_storage.h
chrome/browser/metrics/perf/perf_provider_chromeos.cc
chrome/browser/ui/extensions/app_launch_params.h
components/arc/common/process.mojom
device/bluetooth/bluez/bluetooth_adapter_bluez.h
extensions/common/api/app_runtime.idl

[hidehiko, 2017-02-17 06:00:07]

Description was changed from

==========
s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

BUG=None
TEST=Ran bots.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

TBR=oshima@chromium.org, xiyuan@chromium.org, chrisha@chromium.org,
isherman@chromium.org, stevenjb@chromium.org, maxbogue@chromium.org,
skuhne@chromium.org, benwells@chromium.org, msw@chromium.org, steel@chromium.org
BUG=None
TEST=Ran bots.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[hidehiko, 2017-02-17 06:02:06]

Description was changed from

==========
s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

TBR=oshima@chromium.org, xiyuan@chromium.org, chrisha@chromium.org,
isherman@chromium.org, stevenjb@chromium.org, maxbogue@chromium.org,
skuhne@chromium.org, benwells@chromium.org, msw@chromium.org, steel@chromium.org
BUG=None
TEST=Ran bots.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

TBR=oshima@chromium.org, xiyuan@chromium.org, chrisha@chromium.org,
isherman@chromium.org, stevenjb@chromium.org, maxbogue@chromium.org,
skuhne@chromium.org, benwells@chromium.org, msw@chromium.org,
steel@chromium.org, dcheng@chromium.org
BUG=None
TEST=Ran bots.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[hidehiko, 2017-02-17 06:06:57]

hidehiko@chromium.org changed reviewers:
+ benwells@chromium.org, chrisha@chromium.org, dcheng@chromium.org,
isherman@chromium.org, maxbogue@chromium.org, msw@chromium.org,
oshima@chromium.org, skuhne@chromium.org, steel@chromium.org,
stevenjb@chromium.org, xiyuan@chromium.org

[hidehiko, 2017-02-17 06:06:58]

R+=OWNERs and TBR=OWNERs.
OWNERs, please let me submit with TBR under Luis's LGTM following the chromium
rule, because this change is mechanical comment/log message update.

Luis, Thank you for review. Landing with TBR.

> I pre-emptively added an entry in the style guide to avoid this in the future.
SGTM. Thank you for doing this!

> tools/metrics/histograms/histograms.xml
I intentionally left them as is, just in case.

> components/policy/proto/device_management_backend.proto
I think we can safely change in terms of build, but I also intentionally left
them as well, because these looks a part of HTTP status message.

> ARC++
Fixed them at latest PS.

[hidehiko, 2017-02-17 06:07:08]

The CQ bit was checked by hidehiko@chromium.org

[hidehiko, 2017-02-17 06:07:09]

The patchset sent to the CQ was uploaded after l-g-t-m from
lhchavez@chromium.org
Link to the patchset: https://codereview.chromium.org/2700783002/#ps40001
(title: "Address ARC++")

[commit-bot: I haz the power, 2017-02-17 06:07:48]

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

[dcheng, 2017-02-17 07:34:12]

rs lgtm for mojo

[commit-bot: I haz the power, 2017-02-17 08:16:14]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487311628391770,
"parent_rev": "2d4719164ead73b93c3d9ee64606db55ada32f20", "commit_rev":
"5f6e56e59eb16be722d9424dafa03d19a2aa6697"}

[commit-bot: I haz the power, 2017-02-17 08:16:49]

Description was changed from

==========
s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

TBR=oshima@chromium.org, xiyuan@chromium.org, chrisha@chromium.org,
isherman@chromium.org, stevenjb@chromium.org, maxbogue@chromium.org,
skuhne@chromium.org, benwells@chromium.org, msw@chromium.org,
steel@chromium.org, dcheng@chromium.org
BUG=None
TEST=Ran bots.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
s/\bArc\b/ARC/g for ARC related code.

This CL replaces "Arc" by "ARC" for standardization,
mostly for comments and logs.

Non-targets are;
- Resources (E.g. .grd, .grdp files)
- UMA names.
- Special HTTP status code name.
- Several embedded string, which we need to use Arc to follow their rule.

TBR=oshima@chromium.org, xiyuan@chromium.org, chrisha@chromium.org,
isherman@chromium.org, stevenjb@chromium.org, maxbogue@chromium.org,
skuhne@chromium.org, benwells@chromium.org, msw@chromium.org,
steel@chromium.org, dcheng@chromium.org
BUG=None
TEST=Ran bots.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2700783002
Cr-Commit-Position: refs/heads/master@{#451267}
Committed:
https://chromium.googlesource.com/chromium/src/+/5f6e56e59eb16be722d9424dafa0...
==========

[commit-bot: I haz the power, 2017-02-17 08:16:54]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/5f6e56e59eb16be722d9424dafa0...

[msw, 2017-02-17 18:37:16]

lgtm