[chromeos] Simplify ephemeral users tests

[chromeos] Simplify ephemeral users tests

Now that /root/.forget_usernames is gone, remove the hacks from
chromeos_ephemeral.py

BUG=None
TEST=chromeos_ephemeral.py
NOTRY=true

R=bartfab@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148730

[Nirnimesh, 2012-07-27 01:26:13]

I'll get around to removing the actual calls (like:
TryToEnableLocalStateAutoClearingOnChromeOS) after I remove it from all tests.

[bartfab (slow), 2012-07-27 08:11:22]

One question, otherwise LGTM.

http://codereview.chromium.org/10832046/diff/1/chrome/test/functional/chromeo...
File chrome/test/functional/chromeos_ephemeral.py (left):

http://codereview.chromium.org/10832046/diff/1/chrome/test/functional/chromeo...
chrome/test/functional/chromeos_ephemeral.py:114: self.set_clear_profile(False)
Is this no longer needed either with the recent changes? Will Logout() not clear
out the profile by default?

[Nirnimesh, 2012-07-27 08:24:21]

http://codereview.chromium.org/10832046/diff/1/chrome/test/functional/chromeo...
File chrome/test/functional/chromeos_ephemeral.py (left):

http://codereview.chromium.org/10832046/diff/1/chrome/test/functional/chromeo...
chrome/test/functional/chromeos_ephemeral.py:114: self.set_clear_profile(False)
Logout() does not clear user profile (/home/chronos/user).
"Local State" used to get cleared on logout when /root/.forget_usernames was
around. Not anymore.

[commit-bot: I haz the power, 2012-07-27 08:25:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nirnimesh@chromium.org/10832046/1

[commit-bot: I haz the power, 2012-07-27 08:25:18]

Change committed as 148730

[bartfab (slow), 2012-07-27 08:28:39]

http://codereview.chromium.org/10832046/diff/1/chrome/test/functional/chromeo...
File chrome/test/functional/chromeos_ephemeral.py (left):

http://codereview.chromium.org/10832046/diff/1/chrome/test/functional/chromeo...
chrome/test/functional/chromeos_ephemeral.py:114: self.set_clear_profile(False)
There is a set_clear_profile()/get_clear_profile() combo that, when set, would
cause __SetUp() to clear the user's profile. But I see now that as you say,
Logout() actually clears the flag before doing anything, so clearing it
ourselves is unnecessary. Perfect. Thanks for clarifying.