daveVk - 11:36 pm on Mar 27, 2012 (gmt 0)
would be better declared within function getCities as it is only used inside this function.
You are doing this on each li, it is the ul that is hidden, move to after the for loop.
This will fail as the li does not a id on it, instead try
and remove empty li's from html.