Rename omaha_client and similar tokens to update_client in all contexts.

Rename omaha_client and similar tokens to update_client in all contexts.

This is a mechanical change.

The idea here is to refactor the common code involved in installing
components and extensions in a common module. In the future, we want
more update-related Chrome stuff to go in this module.
Therefore, it is desirable that this module have a name that describes
its purpose.

BUG=445949
Committed: https://crrev.com/39eab2f9d3d0435422f65c6664276fdd72326f89
Cr-Commit-Position: refs/heads/master@{#310022}

[Sorin Jianu, 2015-01-03 00:15:14]

sorin@chromium.org changed reviewers:
+ waffles@chromium.org

[Sorin Jianu, 2015-01-03 00:15:40]

Please take a look, thank you!

[waffles, 2015-01-05 18:52:42]

lgtm, thanks!

[Sorin Jianu, 2015-01-05 19:02:09]

sorin@chromium.org changed reviewers:
+ asargent@chromium.org, cpu@chromium.org, rockot@chromium.org

[Sorin Jianu, 2015-01-05 19:04:48]

sorin@chromium.org changed reviewers:
+ yoz@chromium.org

[Sorin Jianu, 2015-01-05 19:08:22]

I need owner approvals for several files.

cpu: please look at everything, since you own most of the files

rockot, asargent: please review the code under everything extensions

yoz: please review the changes to /extensions/browser/DEPS

Thank you.

[cpu_(ooo_6.6-7.5), 2015-01-05 20:19:03]

lgtm

https://codereview.chromium.org/803313003/diff/40001/components/update_client...
File components/update_client/BUILD.gn (right):

https://codereview.chromium.org/803313003/diff/40001/components/update_client...
components/update_client/BUILD.gn:11: ]
I must confess I have no idea about these change here. Or more specifically why
it was that way, the right side makes more sense.

[Yoyo Zhou, 2015-01-05 20:21:33]

extensions LGTM

https://codereview.chromium.org/803313003/diff/40001/components/update_client...
File components/update_client/BUILD.gn (right):

https://codereview.chromium.org/803313003/diff/40001/components/update_client...
components/update_client/BUILD.gn:11: ]
On 2015/01/05 20:19:03, cpu wrote:
> I must confess I have no idea about these change here. Or more specifically
why
> it was that way, the right side makes more sense.

This is one of those cases where rietveld identified the wrong "original source"
due to similarity.

[Sorin Jianu, 2015-01-05 22:06:17]

The CQ bit was checked by sorin@chromium.org

[commit-bot: I haz the power, 2015-01-05 22:06:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/803313003/40001

[commit-bot: I haz the power, 2015-01-05 22:06:40]

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

[commit-bot: I haz the power, 2015-01-05 22:06:41]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
in order for the CQ to process them

[Ken Rockot(use gerrit already), 2015-01-05 23:10:08]

On 2015/01/05 22:06:41, I haz the power (commit-bot) wrote:
> CLs for remote refs other than refs/pending/heads/master must contain
NOTRY=true
> in order for the CQ to process them

looks like this CL is based on lkgr rather than master?

[Ken Rockot(use gerrit already), 2015-01-05 23:10:08]

On 2015/01/05 22:06:41, I haz the power (commit-bot) wrote:
> CLs for remote refs other than refs/pending/heads/master must contain
NOTRY=true
> in order for the CQ to process them

looks like this CL is based on lkgr rather than master?

[Sorin Jianu, 2015-01-05 23:13:05]

Correct, this is the workflow I've been using so far, work off lkgr instead of
master, for I wanted to avoid syncing to a bad master revision and breaking my
build.

[Ken Rockot(use gerrit already), 2015-01-05 23:17:12]

On 2015/01/05 23:13:05, Sorin Jianu wrote:
> Correct, this is the workflow I've been using so far, work off lkgr instead of
> master, for I wanted to avoid syncing to a bad master revision and breaking my
> build.

That's fair, but you'll need to rebase onto master for the CQ to work. Please
don't follow the automated NOTRY=true advice in this case.

[Sorin Jianu, 2015-01-06 00:19:03]

The CQ bit was checked by sorin@chromium.org

[commit-bot: I haz the power, 2015-01-06 00:20:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/803313003/60001

[commit-bot: I haz the power, 2015-01-06 01:09:37]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-01-06 01:10:38]

Patchset 4 (id:??) landed as
https://crrev.com/39eab2f9d3d0435422f65c6664276fdd72326f89
Cr-Commit-Position: refs/heads/master@{#310022}