UI cleanup, phase 5, ARM (functional changes)

UI cleanup, phase 5, ARM (functional changes)

This commit contains several fixes for several different things I found
while doing 7f31b07, as well as trying to minimise the number of
differences between the MIPS and ARM branches.

I'll note that JavaScript will sometimes let you get away with lack of
trailing semi-colons but it's a very bad habit to get into.

about.asp
  - Remove extra <br> (minimise MIPS diff)
advanced-vlan.asp
  - Remove space at start of line (minimise MIPS diff)
basic-ddns.asp
  - Bug: lack of semi-colon at end of several lines
  - Indentation fixups (minimise MIPS diff)
bwm-common.js
  - Bug: lack of semi-colon at end of line
bwm-hist.js
  - Bug: removal of duplicate ymText() function
  - Bug: lack of semi-colon at end of line
qos-settings.asp
  - Remove blank line (minimise MIPS diff)
nas-samba.asp
  - Use tabs not spaces (minimise MIPS diff)
restrict-edit.asp
  - Bug: fix error message to reflect correct limit (8192)
status-data.jsx
  - Bug: lack of semi-colon at end of line
status-devices.asp
  - Use tabs not spaces (minimise MIPS diff)
  - Remove blank line (minimise MIPS diff)
tools-survey.asp
  - Use tabs not spaces (will be fixing this on MIPS too)
vpn.js
  - Bug: lack of semi-colon at end of several lines