Created attachment 614009 [details] [diff] [review]
The current `forgetNetwork' implementation is buggy because it couldn’t be tested so far.
Here’s a quick fix that works with my current Gaia tree:
mrbkap > for my information, what’s the difference between
var request = this.createRequest();
var request = Services.DOMRequest.createRequest(this._window);
? In DOMWifiManager.js, the latter is only used for `forget' — the former is used everywhere else.
Comment on attachment 614009 [details] [diff] [review]
The two versions of createRequest are equivalent. Please do change the one in forget for consistency though.
Thanks a lot and sorry for the typos!
Created attachment 614042 [details] [diff] [review]
same patch, now using `this.createRequest()' for better consistency.
Created attachment 614043 [details] [diff] [review]
sorry, did a `git diff >> patch' instead of a `git diff > patch'.