Вы не можете действовать таким образом: window.onunload = goodBye ();
Если вы хотите действовать таким образом, у вас есть три способа:
// by this way you use the name of the function, so you override the onunload function with goodBye function
window.onunload = goodBye;
или
// This way you redefine the function
window.onunload = function(){goodBye()};
И мой любимый, потому что он позволяет добавить больше функций:
// This way you create a event listener which allows you to add as many functions as you ant
window.addEventListener("unload", goodBye, false);