CreateIdentityWizard: Don't pass around wizardBox where it is not needed
- Also, in the one function where we actually need it, explain why.
CreateIdentityWizard: Rename (m)RequestedStep to (m)CurrentStep
- The previous name sounded like requesting a step did not necessarily cause the step to be actually executed. This cannot happen though. So "current" sounds more obvious.
CreateIdentityWizard: Don't have low level random FreenetURI generation code in the UI.
- Instead, use the WebOfTrust.createOwnIdentity() which auto-generates the URI
- Also, make sure that the URI parsing codepath does not get hit if the user chose to generate a random one. This might have caused problems if the user first enters an invalid URI and then switches his choices to enable random URI generation.
- Don't continue the wizard beyond the URI-choosing Step if mGenerateRandomSSK is null. This shouldn't happen but its nicer to be sure that the data of the previous Step can never be null. Also, this ensures that we can use mGenerateRandomSSK when creating the identity without checking for null.