Closed
Bug 859631
Opened 11 years ago
Closed 11 years ago
Intermittent webapps/test_uninstall_errors.xul, test_uninstall_errors.html | Test timed out.
Categories
(Core Graveyard :: DOM: Apps, defect)
Tracking
(firefox25 fixed, firefox26 fixed, firefox27 fixed, firefox-esr24 fixed)
RESOLVED
FIXED
mozilla27
People
(Reporter: RyanVM, Assigned: marco)
Details
(Keywords: intermittent-failure, Whiteboard: [apps-automation:P3])
Attachments
(1 file, 3 obsolete files)
7.59 KB,
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
https://tbpl.mozilla.org/php/getParsedLog.php?id=21568577&tree=Mozilla-Inbound WINNT 6.2 mozilla-inbound opt test mochitest-other on 2013-04-08 10:30:49 PDT for push e5cf40b38bd7 slave: t-w864-ix-002 17:36:21 INFO - 10234 INFO TEST-START | chrome://mochitests/content/chrome/dom/tests/mochitest/webapps/test_uninstall_errors.xul 17:41:48 INFO - 10235 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/dom/tests/mochitest/webapps/test_uninstall_errors.xul | Test timed out. 17:41:48 INFO - args: ['C:\\slave\\test\\build\\tests\\bin\\screenshot.exe', 'c:\\users\\cltbld~1.t-w\\appdata\\local\\temp\\mozilla-test-fail_ttnptl'] 17:41:48 INFO - SCREENSHOT: <see log> 17:41:49 INFO - 10236 INFO TEST-END | chrome://mochitests/content/chrome/dom/tests/mochitest/webapps/test_uninstall_errors.xul | finished in 327695ms
Updated•11 years ago
|
Whiteboard: [apps-automation]
Updated•11 years ago
|
Whiteboard: [apps-automation] → [apps-automation:P3]
Comment 1•11 years ago
|
||
(OrangeWFM bugs not touched for 2 months)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
Comment hidden (Legacy TBPL/Treeherder Robot) |
Reporter | ||
Updated•11 years ago
|
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 16•11 years ago
|
||
I think the problem here is that the install onsuccess function could be called while the installation is still in progress. So confirmNextInstall could fail to accept the installation. We could either wait for bug 906114 and see if it fixes the problem, or land this patch and see if it fixes the problem.
Attachment #806787 -
Flags: review?(fabrice)
Assignee | ||
Comment 17•11 years ago
|
||
Or we could just log the installations for now, so that we could better understand the problem.
Attachment #806791 -
Flags: review?(fabrice)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 19•11 years ago
|
||
Another solution similar to the first patch, but that also moves the test in dom/apps so that it's executed on b2g and Android too.
Attachment #806949 -
Flags: feedback?(fabrice)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 21•11 years ago
|
||
Comment on attachment 806949 [details] [diff] [review] Patch Review of attachment 806949 [details] [diff] [review]: ----------------------------------------------------------------- lgtm, but can you do a try run?
Attachment #806949 -
Flags: feedback?(fabrice) → feedback+
Updated•11 years ago
|
Attachment #806791 -
Flags: review?(fabrice)
Updated•11 years ago
|
Attachment #806787 -
Flags: review?(fabrice)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 23•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=493b45f2398b
Attachment #806787 -
Attachment is obsolete: true
Attachment #806791 -
Attachment is obsolete: true
Attachment #806949 -
Attachment is obsolete: true
Attachment #807241 -
Flags: review?(fabrice)
Assignee | ||
Updated•11 years ago
|
Attachment #807241 -
Attachment description: prova → Patch
Attachment #807241 -
Attachment filename: prova → Patch
Updated•11 years ago
|
Attachment #807241 -
Flags: review?(fabrice) → review+
Assignee | ||
Comment 24•11 years ago
|
||
Let's see if this fixes the problem before closing the bug.
Keywords: checkin-needed
Whiteboard: [apps-automation:P3] → [apps-automation:P3][leave open]
Reporter | ||
Comment 25•11 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/0a60c5cf1a1c
Flags: in-testsuite+
Keywords: checkin-needed
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•11 years ago
|
Assignee: nobody → mcastelluccio
Status: REOPENED → ASSIGNED
Whiteboard: [apps-automation:P3][leave open] → [apps-automation:P3]
Assignee | ||
Comment 30•11 years ago
|
||
Notice that now the test is dom/apps/test_uninstall_errors.html. If it fails, we shouldn't open a new bug.
Comment 31•11 years ago
|
||
(In reply to Marco Castelluccio [:marco] from comment #30) > Notice that now the test is dom/apps/test_uninstall_errors.html. If it > fails, we shouldn't open a new bug. Yes, but the xul one is still on other branches so this one may happen again.
Updated•11 years ago
|
Summary: Intermittent webapps/test_uninstall_errors.xul | Test timed out. → Intermittent webapps/test_uninstall_errors.xul, test_uninstall_errors.html | Test timed out.
Comment 32•11 years ago
|
||
(For TBPL bug suggestion matching)
Reporter | ||
Comment 33•11 years ago
|
||
(In reply to Marco Castelluccio [:marco] from comment #24) > Let's see if this fixes the problem before closing the bug. (In reply to Fabrice Desré [:fabrice] from comment #31) > Yes, but the xul one is still on other branches so this one may happen again. Once this is resolved, I can assure you that the other branches will be picking up this fix in short order :)
Assignee | ||
Comment 34•11 years ago
|
||
I think we can mark this as resolved now.
Reporter | ||
Updated•11 years ago
|
status-firefox25:
--- → affected
status-firefox26:
--- → affected
status-firefox27:
--- → fixed
status-firefox-esr24:
--- → affected
Target Milestone: --- → mozilla27
Reporter | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 35•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/c01876298cb0 https://hg.mozilla.org/releases/mozilla-beta/rev/40d5f2078ca9
Reporter | ||
Comment 36•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-esr24/rev/5165d9c53b74
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•