Async would be preferred.
I would love for someone to try fixing this. https://developer.mozilla.org/En/XMLHttpRequest/Using_XMLHttpRequest has an example of how sync vs async XHR code differs. Once that change is made, it's just a matter of making sure the tests still pass.
Created attachment 590513 [details] [diff] [review]
Not sure what tests to run.
Thanks! You should run the geolocation mochitests (https://developer.mozilla.org/en/Mochitest): TEST_PATH=dom/tests/mochitest/geolocation/ make -C objdir mochitest-plain
As far as I can tell, the tests pass. (Sorry for huge delay, midterms + illness struck me)
Comment on attachment 590513 [details] [diff] [review]
Great! Let's send it through try to make sure.
Branch: mozilla-central => try
Try run started, revision 6ad4c8d89359. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=6ad4c8d89359
Try run for 6ad4c8d89359 is complete.
Detailed breakdown of the results available here:
Results (out of 209 total builds):
Builds (or logs if builds failed) available at: