Closed
Bug 1044278
Opened 10 years ago
Closed 10 years ago
Intermittent test_packaged_app_install.html | Check installState - got "installed", expected "pending" | Check readyToApplyDownload - got false, expected true
Categories
(Core Graveyard :: DOM: Apps, defect)
Tracking
(firefox32 unaffected, firefox33 unaffected, firefox34 fixed, firefox-esr24 unaffected, firefox-esr31 unaffected, b2g-v1.4 unaffected, b2g-v2.0 unaffected, b2g-v2.1 fixed)
RESOLVED
FIXED
mozilla34
Tracking | Status | |
---|---|---|
firefox32 | --- | unaffected |
firefox33 | --- | unaffected |
firefox34 | --- | fixed |
firefox-esr24 | --- | unaffected |
firefox-esr31 | --- | unaffected |
b2g-v1.4 | --- | unaffected |
b2g-v2.0 | --- | unaffected |
b2g-v2.1 | --- | fixed |
People
(Reporter: KWierso, Assigned: amac)
Details
(Keywords: intermittent-failure)
Attachments
(1 file, 1 obsolete file)
5.75 KB,
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
https://tbpl.mozilla.org/php/getParsedLog.php?id=44619243&tree=B2g-Inbound b2g_emulator_vm b2g-inbound debug test mochitest-debug-7 on 2014-07-25 12:50:18 PDT for push 83deca5eb082 slave: tst-linux64-spot-1165 13:30:57 INFO - 3135 INFO dumping last 100 message(s) 13:30:57 INFO - 3136 INFO if you need more context, please use SimpleTest.requestCompleteLog() in your test 13:30:57 INFO - 3137 INFO Got expected INVALID_MANIFEST 13:30:57 INFO - 3138 INFO == TEST == No manifest in the zip file 13:30:57 INFO - 3139 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App installed 13:30:57 INFO - 3140 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Got downloaderror MISSING_MANIFEST 13:30:57 INFO - 3141 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Got expected MISSING_MANIFEST 13:30:57 INFO - 3142 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check name mini-manifest 13:30:57 INFO - 3143 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check manifestURL 13:30:57 INFO - 3144 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installOrigin 13:30:57 INFO - 3145 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Removable app 13:30:57 INFO - 3146 INFO TEST-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Check progress 13:30:57 INFO - 3147 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installState 13:30:57 INFO - 3148 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check download available 13:30:57 INFO - 3149 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloading 13:30:57 INFO - 3150 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloadSize 13:30:57 INFO - 3151 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check readyToApplyDownload 13:30:57 INFO - 3152 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App is not launchable 13:30:57 INFO - 3153 INFO App uninstalled 13:30:57 INFO - 3154 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | setAppVersion OK 13:30:57 INFO - 3155 INFO == TEST == Mini-manifest app name is different from webapp manifest name 13:30:57 INFO - 3156 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App installed 13:30:57 INFO - 3157 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Got downloaderror MANIFEST_MISMATCH 13:30:57 INFO - 3158 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Got expected MANIFEST_MISMATCH 13:30:57 INFO - 3159 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check name mini-manifest 13:30:57 INFO - 3160 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check manifestURL 13:30:57 INFO - 3161 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installOrigin 13:30:57 INFO - 3162 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Removable app 13:30:57 INFO - 3163 INFO TEST-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Check progress 13:30:57 INFO - 3164 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installState 13:30:57 INFO - 3165 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check download available 13:30:57 INFO - 3166 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloading 13:30:57 INFO - 3167 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloadSize 13:30:57 INFO - 3168 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check readyToApplyDownload 13:30:57 INFO - 3169 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App is not launchable 13:30:57 INFO - 3170 INFO App uninstalled 13:30:57 INFO - 3171 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | == TEST == Mini-manifest dev name is different from manifest dev name 13:30:57 INFO - 3172 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App installed 13:30:57 INFO - 3173 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Got downloaderror MANIFEST_MISMATCH 13:30:57 INFO - 3174 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Got expected MANIFEST_MISMATCH 13:30:57 INFO - 3175 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check name mini-manifest 13:30:57 INFO - 3176 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check manifestURL 13:30:57 INFO - 3177 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installOrigin 13:30:57 INFO - 3178 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Removable app 13:30:57 INFO - 3179 INFO TEST-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Check progress 13:30:57 INFO - 3180 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installState 13:30:57 INFO - 3181 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check download available 13:30:57 INFO - 3182 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloading 13:30:57 INFO - 3183 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloadSize 13:30:57 INFO - 3184 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check readyToApplyDownload 13:30:57 INFO - 3185 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App is not launchable 13:30:57 INFO - 3186 INFO App uninstalled 13:30:57 INFO - 3187 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | == TEST == Mini-manifest dev url is different from manifest dev url 13:30:57 INFO - 3188 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App installed 13:30:57 INFO - 3189 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Got downloaderror MANIFEST_MISMATCH 13:30:57 INFO - 3190 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Got expected MANIFEST_MISMATCH 13:30:57 INFO - 3191 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check name mini-manifest 13:30:57 INFO - 3192 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check manifestURL 13:30:57 INFO - 3193 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installOrigin 13:30:57 INFO - 3194 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Removable app 13:30:57 INFO - 3195 INFO TEST-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Check progress 13:30:57 INFO - 3196 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installState 13:30:57 INFO - 3197 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check download available 13:30:57 INFO - 3198 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloading 13:30:57 INFO - 3199 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloadSize 13:30:57 INFO - 3200 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check readyToApplyDownload 13:30:57 INFO - 3201 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App is not launchable 13:30:57 INFO - 3202 INFO App uninstalled 13:30:57 INFO - 3203 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | setAppVersion OK 13:30:57 INFO - 3204 INFO == TEST == Install packaged app 13:30:57 INFO - 3205 INFO Application installed 13:30:57 INFO - 3206 INFO Got oninstall event 13:30:57 INFO - 3207 INFO App downloaded 13:30:57 INFO - 3208 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check name 13:30:57 INFO - 3209 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check name mini-manifest 13:30:57 INFO - 3210 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check version 13:30:57 INFO - 3211 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check size 13:30:57 INFO - 3212 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check launch path 13:30:57 INFO - 3213 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check manifestURL 13:30:57 INFO - 3214 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installOrigin 13:30:57 INFO - 3215 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Removable app 13:30:57 INFO - 3216 INFO TEST-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Check progress 13:30:57 INFO - 3217 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installState 13:30:57 INFO - 3218 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check download available 13:30:57 INFO - 3219 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloading 13:30:57 INFO - 3220 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloadSize 13:30:57 INFO - 3221 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check readyToApplyDownload 13:30:57 INFO - 3222 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | setAppVersion OK 13:30:57 INFO - 3223 INFO == TEST == Install packaged app with a cancel/resume 13:30:57 INFO - 3224 INFO Application installed 13:30:57 INFO - 3225 INFO Got oninstall event 13:30:57 INFO - 3226 INFO Cancelling the download 13:30:57 INFO - 3227 INFO App downloaded 13:30:57 INFO - 3228 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check name 13:30:57 INFO - 3229 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check name mini-manifest 13:30:57 INFO - 3230 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check version 13:30:57 INFO - 3231 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check size 13:30:57 INFO - 3232 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check launch path 13:30:57 INFO - 3233 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check manifestURL 13:30:57 INFO - 3234 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check installOrigin 13:30:57 INFO - 3235 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Removable app 13:30:57 INFO - 3236 INFO TEST-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Check progress 13:30:57 INFO - 3237 INFO TEST-UNEXPECTED-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Check installState - got "installed", expected "pending" 13:30:57 INFO - TEST-INFO | expected PASS 13:30:58 INFO - 3238 INFO dumping last 3 message(s) 13:30:58 INFO - 3239 INFO if you need more context, please use SimpleTest.requestCompleteLog() in your test 13:30:58 INFO - 3240 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check download available 13:30:58 INFO - 3241 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloading 13:30:58 INFO - 3242 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | Check downloadSize 13:30:58 INFO - 3243 INFO TEST-UNEXPECTED-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Check readyToApplyDownload - got false, expected true 13:30:58 INFO - TEST-INFO | expected PASS 13:31:00 INFO - 3244 INFO TEST-OK | /tests/dom/apps/tests/test_packaged_app_install.html | took 74536ms 13:31:00 INFO - 3245 INFO [Child 727] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1274 13:31:01 INFO - 3246 INFO [Child 727] WARNING: Failed to retarget HTML data delivery to the parser thread.: file ../../../gecko/parser/html/nsHtml5StreamParser.cpp, line 947 13:31:02 INFO - 3247 INFO TEST-START | /tests/dom/apps/tests/test_packaged_app_update.html
Updated•10 years ago
|
Keywords: intermittent-failure
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 12•10 years ago
|
||
Fabrice, can you suggest somebody to look into this recent regression?
Flags: needinfo?(fabrice)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 14•10 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #12) > Fabrice, can you suggest somebody to look into this recent regression? Maybe Antonio ?
Flags: needinfo?(fabrice) → needinfo?(amac.bug)
Assignee | ||
Comment 15•10 years ago
|
||
Yeah, I'll take a look tomorrow.
Assignee: nobody → amac.bug
Flags: needinfo?(amac.bug)
Assignee | ||
Comment 16•10 years ago
|
||
I've written two versions of this patch, one with the timer for the cancellable download set to 15 seconds, and the other set with 1.5 seconds. This is the one with 15 seconds. The reason I was using a low timer was that even running them locally setting a timer of more than 5 seconds usually just hanged the server forever (the timer never fired). Now it seems to be working locally... Try run with a 15 seconds timer: https://tbpl.mozilla.org/?tree=Try&rev=19639073fbcd And with a 1.5 seconds timer: https://tbpl.mozilla.org/?tree=Try&rev=a40bf5b35a2a I've also changed the test code so now it can detect the app install wasn't cancelled and fire a todo (instead of failing).
Assignee | ||
Comment 17•10 years ago
|
||
New try run at https://tbpl.mozilla.org/?tree=Try&rev=898c6298bc0b
Attachment #8465297 -
Attachment is obsolete: true
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 20•10 years ago
|
||
Comment on attachment 8465568 [details] [diff] [review] Update timers and remove a race condition between the download and the cancellation Multiple retriggers of the tests look good. Asking for rreview :)
Attachment #8465568 -
Flags: review?(fabrice)
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) |
Updated•10 years ago
|
Attachment #8465568 -
Flags: review?(fabrice) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 32•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/0e683d5d1e2e
Keywords: checkin-needed
Comment 33•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0e683d5d1e2e
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Comment 34•10 years ago
|
||
AFAICT, these failures were only hitting trunk, but please nominate this for b2g32 uplift if you think this patch should land on v2.0 as well. Not sure if the underlying condition is present there or not.
status-b2g-v1.4:
--- → unaffected
status-b2g-v2.0:
--- → unaffected
status-b2g-v2.1:
--- → fixed
status-firefox32:
--- → unaffected
status-firefox33:
--- → unaffected
status-firefox34:
--- → fixed
status-firefox-esr24:
--- → unaffected
status-firefox-esr31:
--- → unaffected
Flags: needinfo?(amac.bug)
Assignee | ||
Comment 35•10 years ago
|
||
I don't think this particular race condition is going to impact any users, and it's quite late for 2.0 anyway. WDYT, Fabrice?
Flags: needinfo?(amac.bug) → needinfo?(fabrice)
Comment 36•10 years ago
|
||
(In reply to Antonio Manuel Amaya Calvo (:amac) from comment #35) > I don't think this particular race condition is going to impact any users, > and it's quite late for 2.0 anyway. WDYT, Fabrice? I agree, we don't need that on 2.0
Flags: needinfo?(fabrice)
Updated•10 years ago
|
QA Whiteboard: [qa-]
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
•