Return undefined, not null from onbeforeunload

IE doesn't accept null as a "do nothing" value, only undefined.
If you return null, it'll show a popup with the text "null".

Change-Id: I984a7ded483c4c178cbca2a9af9abd895e6ded56
Merge "Return undefined, not null from onbeforeunload"