Remove unused LOGGED_IN_OOBE LoggedInState.

Remove unused LOGGED_IN_OOBE LoggedInState.

BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282645

[Thiemo Nagel, 2014-07-07 19:11:02]

Hi Nikita and Mitsuru,

could you please take a look at this small cleanup, as discussed via email? 
(I'll add Mitsuru to the thread.)

Thank you and best regards,
Thiemo

[Nikita (slow), 2014-07-07 21:07:46]

lgtm

[Nikita (slow), 2014-07-07 21:07:46]

lgtm

lgtm

[oshima, 2014-07-07 21:19:49]

lgtm

[Thiemo Nagel, 2014-07-08 08:39:30]

On 2014/07/07 21:19:49, oshima wrote:
> lgtm

I just realized that SetLoggedInState() can be completely omitted from
ShowLoginWizard() because LoginState is initialized to
(LoginState::LOGGED_IN_NONE, LoginState::LOGGED_IN_USER_NONE) anyways.

Do you agree, Nikita?

[Nikita (slow), 2014-07-08 08:57:56]

On 2014/07/08 08:39:30, Thiemo Nagel wrote:
> On 2014/07/07 21:19:49, oshima wrote:
> > lgtm
> 
> I just realized that SetLoggedInState() can be completely omitted from
> ShowLoginWizard() because LoginState is initialized to
> (LoginState::LOGGED_IN_NONE, LoginState::LOGGED_IN_USER_NONE) anyways.
> 
> Do you agree, Nikita?

sgtm

[Thiemo Nagel, 2014-07-08 09:31:21]

> sgtm

Thank you!

[Thiemo Nagel, 2014-07-10 12:36:05]

Hi Jun,

could you please take a look whether you agree with the changes to
login_state_notification_blocker_chromeos_unittest.cc?

Thank you!
Thiemo

[Jun Mukai, 2014-07-10 17:06:47]

notification lgtm

[Thiemo Nagel, 2014-07-11 11:41:51]

On 2014/07/10 17:06:47, Jun Mukai wrote:
> notification lgtm

Thank you!

[Thiemo Nagel, 2014-07-11 11:41:54]

The CQ bit was checked by tnagel@chromium.org

[commit-bot: I haz the power, 2014-07-11 11:42:30]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tnagel@chromium.org/371033005/40001

[commit-bot: I haz the power, 2014-07-11 12:25:05]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/9...)
  android_chromium_gn_compile_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_chromium_gn_c...)
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/20...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  ios_dbg_simulator on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_dbg_simulator/bui...)
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)
  ios_rel_device_ninja on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device_ninja/...)
  linux_chromium_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_clang_...)
  linux_chromium_gn_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_gn_rel...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)
  mac_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_compile_...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win8_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win8_chromium_rel/bui...)
  win_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_compile_...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/27499)

[commit-bot: I haz the power, 2014-07-11 12:26:37]

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

[commit-bot: I haz the power, 2014-07-11 12:26:38]

Try jobs failed on following builders:
  ios_dbg_simulator on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_dbg_simulator/bui...)
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)
  ios_rel_device_ninja on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device_ninja/...)
  mac_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_compile_...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/27507)

[Thiemo Nagel, 2014-07-11 12:33:51]

The CQ bit was checked by tnagel@chromium.org

[commit-bot: I haz the power, 2014-07-11 12:34:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tnagel@chromium.org/371033005/60001

[commit-bot: I haz the power, 2014-07-11 17:45:21]

Change committed as 282645