Prefix CommandLine usage with base namespace (Part 8: components)

Prefix CommandLine usage with base namespace (Part 8: components)

Prefix all CommandLine usage in the components/ directory's following subdirectories:
autofill, cloud_devices, crash, data_reduction_proxy and dom_distiller
with the base:: namespace.

BUG=422426
Committed: https://crrev.com/c3348259183454d20a97f875ed70316776332d78
Cr-Commit-Position: refs/heads/master@{#306576}

[kkristof, 2014-11-26 16:20:43]

kkosztyo.u-szeged@partner.samsung.com changed reviewers:
+ bengr@chromium.org, caitkp@chromium.org, jochen@chromium.org

[kkristof, 2014-11-26 16:20:44]

Please look at this.

[jochen (gone - plz use gerrit), 2014-11-27 14:55:01]

lgtm

[kkristof, 2014-11-28 12:48:43]

The CQ bit was checked by kkosztyo.u-szeged@partner.samsung.com

[commit-bot: I haz the power, 2014-11-28 12:49:04]

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

[commit-bot: I haz the power, 2014-11-28 12:52:27]

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

[commit-bot: I haz the power, 2014-11-28 12:52:28]

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

[kkristof, 2014-11-28 15:46:17]

On 2014/11/28 12:52:28, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux
>
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

Hi, I have updated the patch to match with the 80 characters line limit.

[Cait (Slow), 2014-12-01 16:08:49]

lgtm

[kkristof, 2014-12-02 07:35:49]

The CQ bit was checked by kkosztyo.u-szeged@partner.samsung.com

[commit-bot: I haz the power, 2014-12-02 07:36:26]

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

[commit-bot: I haz the power, 2014-12-02 08:04:47]

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

[commit-bot: I haz the power, 2014-12-02 08:04:48]

Try jobs failed on following builders:
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/93146)

[kkristof, 2014-12-02 09:56:11]

Hi, I have fixed the build issue in the win port and also rebased it.

[bengr, 2014-12-02 16:17:32]

components/data_reduction_proxy and components/dom_distiller lgtm with two nits.

https://codereview.chromium.org/762723003/diff/40001/components/dom_distiller...
File components/dom_distiller/standalone/content_extractor.cc (right):

https://codereview.chromium.org/762723003/diff/40001/components/dom_distiller...
components/dom_distiller/standalone/content_extractor.cc:372:
*base::CommandLine::ForCurrentProcess();
Indent two more.

https://codereview.chromium.org/762723003/diff/40001/components/dom_distiller...
components/dom_distiller/standalone/content_extractor.cc:384:
command_line.GetSwitchValuePath(kOutputFile);
This should fit on the previous line now.

[kkristof, 2014-12-02 17:06:59]

On 2014/12/02 16:17:32, bengr wrote:
> components/data_reduction_proxy and components/dom_distiller lgtm with two
nits.

Okay, I have uploaded the new patch.

[bengr, 2014-12-02 20:21:14]

On 2014/12/02 17:06:59, kkristof wrote:
> On 2014/12/02 16:17:32, bengr wrote:
> > components/data_reduction_proxy and components/dom_distiller lgtm with two
> nits.
> 
> Okay, I have uploaded the new patch.

still lgtm.

[kkristof, 2014-12-03 07:57:48]

The CQ bit was checked by kkosztyo.u-szeged@partner.samsung.com

[commit-bot: I haz the power, 2014-12-03 07:58:21]

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

[commit-bot: I haz the power, 2014-12-03 08:49:00]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2014-12-03 08:49:51]

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