Since #1737, if you specify both jad and downloadJAD, jad takes precedence. But jad is set to tests/runtests.jad by default (in config/runtests.js, which is the default value of CONFIG in Makefile). So if you |make| and then specify downloadJAD in a URL parameter, it'll be ignored, which is why @brendandahl and I have been having trouble loading midlets via URLs like http://localhost:8000/index.html?downloadJAD=….
This change makes downloadJAD take precedence over jad, so URLs like that work as before. The automation.js changes simply undo a workaround for the problem when loading test URLs that relied on downloadJAD (which I completely forgot about, otherwise the problem would have been more obvious).
Since #1737, if you specify both jad and downloadJAD, jad takes precedence. But jad is set to tests/runtests.jad by default (in config/runtests.js, which is the default value of CONFIG in Makefile). So if you |make| and then specify downloadJAD in a URL parameter, it'll be ignored, which is why @brendandahl and I have been having trouble loading midlets via URLs like http://localhost:8000/index.html?downloadJAD=….
This change makes downloadJAD take precedence over jad, so URLs like that work as before. The automation.js changes simply undo a workaround for the problem when loading test URLs that relied on downloadJAD (which I completely forgot about, otherwise the problem would have been more obvious).