Das "Form1" gehört da auch nicht hin, entweder durch "self" ersetzen oder ganz weglassen.
Das hatte ich mal in der Schule vor 8 Jahren gelernt. Dort hieß es, dass man immer die aktuelle Form davor schreiben sollte.
Dann hast Du etwas Falsches gelernt. Form1 ist die globale Variable, die Delphi automatisch in die entsprechende Formularunit schreibt. Die kann man aber auch umbenennen oder ganz löschen, wenn man das Formular nicht automatisch erzeugen lässt. Das bedeutet dann, dass die ganze Klasse nicht mehr funktioniert (sich nicht mehr kompilieren lässt), nur weil die Variable, die die Klasse TForm1 ja eigentlich gar nicht benötigt, nicht vorhanden ist. Noch schlimmer wird es dann, wenn man die Variable beibehält, aber eine TForm1-Instanz in einer anderen Variablen ablegt, da damit immer die falsche Instanz (sofern überhaupt vorhanden) angesprochen wird. Das merkt man aber erst während der Laufzeit.