Make EnterpriseInstallAttributes::GetRegistrationUser() private.

Make EnterpriseInstallAttributes::GetRegistrationUser() private.

In the long run, it should probably be removed.

BUG=638713
Committed: https://crrev.com/0c08e95b4d061021f5ecee2a38465aa238c93d2f
Cr-Commit-Position: refs/heads/master@{#412797}

[Thiemo Nagel, 2016-08-17 20:26:38]

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

[commit-bot: I haz the power, 2016-08-17 20:27:04]

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

[Thiemo Nagel, 2016-08-17 20:27:57]

Description was changed from

==========
Make EnterpriseInstallAttributes::GetRegistrationUser() private.

BUG=638713
==========

to

==========
Make EnterpriseInstallAttributes::GetRegistrationUser() private.

In the long run, it should probably be removed.

BUG=638713
==========

[Thiemo Nagel, 2016-08-17 20:30:12]

tnagel@chromium.org changed reviewers:
+ pastarmovj@chromium.org

[Thiemo Nagel, 2016-08-17 20:30:12]

Hi Julian,

I'm sure you have fond memories of EnterpriseInstallAttributes.  ;)

Could you please take a look at this tiny CL?

Thank you!
Thiemo

[commit-bot: I haz the power, 2016-08-17 21:05:09]

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

[commit-bot: I haz the power, 2016-08-17 21:05:09]

Dry run: This issue passed the CQ dry run.

[pastarmovj, 2016-08-18 08:29:38]

LGTM 

I think this used to be used for prepopulating the enrollment user in some
distant point in the past but we might have never implemented this just as well.

[Thiemo Nagel, 2016-08-18 10:56:20]

On 2016/08/18 08:29:38, pastarmovj wrote:
> LGTM 
> 
> I think this used to be used for prepopulating the enrollment user in some
> distant point in the past but we might have never implemented this just as
well.

Thank you for the context!

[Thiemo Nagel, 2016-08-18 10:56:26]

The CQ bit was checked by tnagel@chromium.org

[commit-bot: I haz the power, 2016-08-18 10:56:59]

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, 2016-08-18 11:03:51]

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

[commit-bot: I haz the power, 2016-08-18 11:03:54]

Internal error: failed to checkout. Please try again.

[Thiemo Nagel, 2016-08-18 11:23:38]

The CQ bit was checked by tnagel@chromium.org

[commit-bot: I haz the power, 2016-08-18 11:23:53]

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, 2016-08-18 12:04:18]

Description was changed from

==========
Make EnterpriseInstallAttributes::GetRegistrationUser() private.

In the long run, it should probably be removed.

BUG=638713
==========

to

==========
Make EnterpriseInstallAttributes::GetRegistrationUser() private.

In the long run, it should probably be removed.

BUG=638713
==========

[commit-bot: I haz the power, 2016-08-18 12:04:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-18 12:06:14]

Description was changed from

==========
Make EnterpriseInstallAttributes::GetRegistrationUser() private.

In the long run, it should probably be removed.

BUG=638713
==========

to

==========
Make EnterpriseInstallAttributes::GetRegistrationUser() private.

In the long run, it should probably be removed.

BUG=638713

Committed: https://crrev.com/0c08e95b4d061021f5ecee2a38465aa238c93d2f
Cr-Commit-Position: refs/heads/master@{#412797}
==========

[commit-bot: I haz the power, 2016-08-18 12:06:15]

Patchset 1 (id:??) landed as
https://crrev.com/0c08e95b4d061021f5ecee2a38465aa238c93d2f
Cr-Commit-Position: refs/heads/master@{#412797}