Minor RegExp issue in the webSearchTab.js code that tries to validate clicked links come from Google search requests.
A regexp of "\/url?" will match "/ur"-anything because the question mark makes the "l" optional. What you really want there is /^\/url\?/.test()
Not really a big deal since you're also checking the host, unlikely that Google would try to fool Thunderbird for malicious purposes. If anything you'll match too much and then send legit intra-Google URLs to the external browser.
It seems that Google doesn't use redirects for its search results anymore (nor does Yahoo), so we should just remove this. We need to keep the special-casing for the Yahoo host name though, since image search is on a different subdomain.
Do we need to fix this before we release 10.0 ?
Created attachment 591270 [details] [diff] [review]
As discussed on irc, this should fix it so that just search? urls stay within google (assuming the pre & post hosts are the same).
Comment on attachment 591270 [details] [diff] [review]
Looks good, as discussed on IRC.