Всё норм. Просто если не объявить переменную `temp`, скрипт писал бы в `window.temp`, что работает, но медленно и чревато ошибками. В strict mode теперь будет выдавать ошибку.

Алсо, по возможности я бы избегал замыкания функций (у тебя замыкается `scrollpForm`) в часто-вызываемых методах, разве что если без них совсем не обойтись, ибо будут жрать память очень и очень сильно. Лучше как-то размещать в существующих объектах или создавать новые.

И да, если будешь замыкать функцию, объявляй её так `var fun = function(arg) { body };`, а потом делай `fun = null;` когда функция будет не нужна. Таким образом мы скажем GC, что можно уже убирать функцию из памяти.