650676 - browser_autocompletesearch.js throws getting ProfD, then times out, on Android

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Phil Ringnalda (:philor)]

http://tinderbox.mozilla.org/showlog.cgi?log=Mobile/1303066294.1303067175.20037.gz#err1
Android Tegra 250 mozilla-central opt test browser-chrome on 2011/04/17 11:51:34
s: tegra-071
(and every other log I looked at)

TEST-START | chrome://mochitests/content/browser/mobile/chrome/browser_autocompletesearch.js
TEST-INFO | chrome://mochitests/content/browser/mobile/chrome/browser_autocompletesearch.js | Console message: [JavaScript Error: "[Exception... "Component returned failure code: 0x8000ffff (NS_ERROR_UNEXPECTED) [nsIFileProtocolHandler.getFileFromURLSpec]"  nsresult: "0x8000ffff (NS_ERROR_UNEXPECTED)"  location: "JS frame :: chrome://mochikit/content/chrome-harness.js :: getChromeDir :: line 87"  data: no]" {file: "chrome://mochikit/content/chrome-harness.js" line: 87}]
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_autocompletesearch.js | Test timed out
INFO TEST-END | chrome://mochitests/content/browser/mobile/chrome/browser_autocompletesearch.js | finished in 30523ms

Comment 1

[Phil Ringnalda (:philor)]

Attachment: disable for now

On top of the patch in bug 650578, and with the same motivation.

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

this bug should stay open until the test case is fixed, but disabling it for now is a good short term approach.

Comment 3

[Phil Ringnalda (:philor)]

Attachment: rebased disable

Okay, that was sneaky, fixing browser_addons.js so I couldn't apply this disable-patch on top of that disable-patch, but you can't defeat my desire to disable things that easily!

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: debugging patch

Just some debugging statements that might help figure out why the mock file is not loaded. We don't have tests on try server, and I don't have a local android test setup working.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed debugging code:
http://hg.mozilla.org/mozilla-central/rev/6e63aec37a85

Comment 6

[Phil Ringnalda (:philor)]

mock path: chrome://mochitests/content/browser/mobile/chrome/browser_autocompletesearch.js
mock URI: jar:file:///mnt/sdcard/tests/profile/tests.jar!/content/browser/mobile/chrome/browser_autocompletesearch.js

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

This patch will check to see if the tests are in a JAR file and will extract the file from the JAR. Otherwise it will copy the file.

I tested the JAR extraction as best I could locally, and it worked. Time to test for realz.

Comment 8

[Phil Ringnalda (:philor)]

http://hg.mozilla.org/mozilla-central/rev/078a4bcba8c5

Comment 9

[Anna (Waverley)]

Can someone verify this, please?

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

the test is green