5 years ago
Created attachment 628648 [details] [diff] [review] Upload 1 Turns out this was a test failure. This is trivial enough that I'm not going to bother with a pull request. But for your delectation, the commit message: Bug 734045 (script0): Use getLookup not getPredictions The results of getPredictions are filtered through the matcher which prevents the return of more than 10 results, so it's silly doing maths on the lengths expecting it to work. Instead we should be using getLookup directly which isn't filtered, and so the length maths will work.
(Also fwiw: https://tbpl.mozilla.org/?tree=Try&rev=3e09d6ee6d89)