This simply removes the mUserInteraction logic from LLLoginInstance, which is not used other than in this behavior. Internal discussion indicates that the current behavior is undesirable and is not being relied upon by test automation, and in fact interferes with test automation.

Prior to this change, if the viewer is launched with:

SecondLife.exe --login FIRST LAST PASSWORD

... and if the login response is that a TOS, Critical Message, or Update must be acknowledged, the viewer simply returns to the credentials screen (login panel) but does not indicate why. 

After this change, the viewer will display the appropriate follow-on UI even if --login is used. If necessary, an additional option could be specified to inhibit showing these UI elements for test automation scenarios (e.g. auto-accepting or exiting the viewer, as appropriate).

(the bulk of the raw diff is whitespace changes due to re-indentation)

Launched viewer normally (with "normal" account), entered credentials, logged in successfully.
Launched viewer via --login (with "normal" account), logged in successfully.
Launched viewer via --login with account needing to accept TOS update; viewer displayed TOS dialog; accepted and logged in successfully.