>1) Criar um padrão desse tipo, para a Activity, seria uma má prática? Deveria ser usado esse padrão apenas quando fosse necessário a "reciclagem" de Views?

A finalidade do Padrão ViewHolder é evitar o uso repetido de `findViewById()` para obter as referências às views.  
Ele é normalmente referido na utilização da ListView/RecyclerView mas pode e é, nesse sentido/finalidade, utilizado em outras situações.

Quando você declara atributos da Activity para guardar as views e os usa para as aceder em locais diferentes dela, apesar de não ter uma classe para o efeito(ViewHolder), está efectivamente a evitar o uso repetido de `findViewById()`.

>2) Passar uma ViewHolder como parâmetro é uma má prática? (supondo que na classe auxiliar eu use todas as views dentro do ViewHolder)

Não me parece que seja. É comum usar uma classe para agrupar informações que se relacionam para evitar ter de passar, individualmente, muitos parâmetros a um método.

>3) Usando esse tipo de prática na Activity, e passando como parâmetro (como no exemplo acima), eu deveria configurar o objeto viewHolder = null quando a Activity fosse destruída?

Atribuir `null` a uma variável não garante que não haja vazamento de memória.  
O que importa é que o objecto, neste caso VerificarLogin, tenha um "tempo de vida" inferior ao da Activity.