Test Failed : testGoogleSuggestedTerms in c:\mozilla\tests\firefox\testContent\t estGoogleSuggestions.js ERROR - Test Failure: {'exception': {'message': "timeout exceeded for waitForEle ment XPath: /html/body/span[@id='main']/center/span[@id='body']/center/form/tabl e[2]/tbody/tr[2]/td", 'lineNumber': 306 Google supplies different markup to us depending on the version of the browser you are using. That means releases will need another lookup string as nightly builds. We should differentiate between those two variants and build the lookup string appropriately. Aakash, can you please take a look at it?

Aakash, would you have time to check this failure? Would be nice to see it fixed.

Hm, you're right. After a little investigation, I got the following xpath id's for the first Namoroka: new elementslib.XPath(controller.tabs.activeTab, "/html/body/center/form/table[2]/tbody/tr[2]/td") Firefox: new elementslib.XPath(controller.tabs.activeTab, "/html/body/span[@id='main']/center/span[@id='body']/center/form/table[2]/tbody/tr[2]/td"); Minefield: new elementslib.XPath(controller.tabs.activeTab, "/html/body/center/form/table[2]/tbody/tr[2]/td") Also, a new community member, Shirish also noted that this is locale dependent as well. If that's the case, the string that we're using might not translate correctly and bring up no results...which will fail the testscript as well. What'd you like to do from here?

There is no redirect depending on the locale and the XPath expression looks always the same for me also in localized builds. So can you point us to the report from Shirish? I would say we should only differentiate between the two expression above. So we match branded and non-branded builds. If we have a problem with another locale we should investigate first.

Once bug 521458 has been landed we can use the platform version to determine if we have a release build or a non-branded one. We simply have to check if "pre" is existent inside the version identifier.

That solves our problem with release and nightly builds.

Comment on attachment 406448 [details] [diff] [review] Patch v1 The code looks right and it passes across Minefield, Namoroka and Firefox 3.5 on XP and OSX. Thanks for doing this, Henrik.

Landed as: http://hg.mozilla.org/qa/mozmill-tests/rev/6c393011d32d http://hg.mozilla.org/qa/mozmill-tests/rev/724eccaf231f

Mass move of Mozmill Test related project bugs to newly created components. You can filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.