Closed
Bug 904162
Opened 11 years ago
Closed 11 years ago
Permanent Orange (windows): TEST-UNEXPECTED-FAIL | test_partsOnDemand.js | test_attachment_size.js | uncaught exception: 2147500037
Categories
(MailNews Core :: Testing Infrastructure, defect)
Tracking
(thunderbird24 fixed, thunderbird25 fixed)
RESOLVED
FIXED
Thunderbird 26.0
People
(Reporter: standard8, Assigned: hiro)
References
Details
(Keywords: intermittent-failure)
Attachments
(2 files, 1 obsolete file)
2.10 KB,
patch
|
standard8
:
review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
861 bytes,
patch
|
standard8
:
review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
Windows-only bug, currently seen on Beta on tbpl, probably other branches as well: https://tbpl.mozilla.org/php/getParsedLog.php?id=26442386&tree=Thunderbird-Beta#error21 TEST-UNEXPECTED-FAIL | ../../../resources/logHelper.js | Error console says [stackFrame uncaught exception: 2147500037] - See following stack: JS frame :: ../../../resources/logHelper.js :: _errorConsoleTunnel.observe :: line 63 native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0 TEST-INFO | (xpcshell/head.js) | exiting test
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 3•11 years ago
|
||
I am guessing the cause of this error is similar to bug 903946, it is in mozilla/uriloader/exthandler/win/nsOSHelperAppService.cpp. Is there someone using Windows?
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 5•11 years ago
|
||
This is a patch for the failure of test_attachment_size.js On my enviroment test_partsOnDemand.js does not fail unfortunately.
Assignee: nobody → hiikezoe
Attachment #792675 -
Flags: review?(mbanner)
Assignee | ||
Comment 6•11 years ago
|
||
The previous patch was wrong one....
Attachment #792675 -
Attachment is obsolete: true
Attachment #792675 -
Flags: review?(mbanner)
Attachment #792678 -
Flags: review?(mbanner)
Assignee | ||
Comment 7•11 years ago
|
||
I am not really convinced, but this patch might solve the failure of test_partsOnDemand.js. I hope!
Assignee | ||
Comment 8•11 years ago
|
||
Archaeopteryx, could you please push both two patches on try server with "-p all -u xpcshell -t none" options. Thank you,
Flags: needinfo?(archaeopteryx)
Comment 9•11 years ago
|
||
Pushed the patches to Thunderbird-Try a hour ago, should be finished in two hours: https://tbpl.mozilla.org/?tree=Thunderbird-Try&showall=1&rev=b35523ae644e
Flags: needinfo?(archaeopteryx)
Assignee | ||
Comment 10•11 years ago
|
||
Thank you, Archaeopteryx. But try server has still problems unfortunately.
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) |
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) |
Reporter | ||
Comment 32•11 years ago
|
||
Comment on attachment 792678 [details] [diff] [review] [checked in] Remove_file_extension_to_avoid_test_orange_on_windows.patch Looks good, thanks. r=Standard8.
Attachment #792678 -
Flags: review?(mbanner) → review+
Reporter | ||
Comment 33•11 years ago
|
||
Comment on attachment 792678 [details] [diff] [review] [checked in] Remove_file_extension_to_avoid_test_orange_on_windows.patch [Triage Comment] Taking straight onto aurora/beta as I know this fixes the issues there.
Attachment #792678 -
Flags: approval-comm-beta+
Attachment #792678 -
Flags: approval-comm-aurora+
Reporter | ||
Comment 34•11 years ago
|
||
Comment on attachment 792678 [details] [diff] [review] [checked in] Remove_file_extension_to_avoid_test_orange_on_windows.patch https://hg.mozilla.org/comm-central/rev/2712c4d0039b https://hg.mozilla.org/releases/comm-aurora/rev/3a0afb97438a https://hg.mozilla.org/releases/comm-beta/rev/1ebed862c635
Attachment #792678 -
Attachment description: Remove_file_extension_to_avoid_test_orange_on_windows.patch → [checked in] Remove_file_extension_to_avoid_test_orange_on_windows.patch
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Reporter | ||
Comment 37•11 years ago
|
||
Comment on attachment 792683 [details] [diff] [review] Possible fix for test_partsOnDemand.js Ok, try server says this works for comm-beta. I'm going to assume there's some difference in the builders that we've got on central versus beta, but as this passes, I think its fine to take it everywhere, as it'll possibly solve failures for developers as well.
Attachment #792683 -
Flags: review+
Attachment #792683 -
Flags: approval-comm-beta+
Attachment #792683 -
Flags: approval-comm-aurora+
Reporter | ||
Comment 38•11 years ago
|
||
https://hg.mozilla.org/comm-central/rev/c0baf3d087a3
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 26.0
Reporter | ||
Comment 39•11 years ago
|
||
https://hg.mozilla.org/releases/comm-aurora/rev/cbfa43a464f8 https://hg.mozilla.org/releases/comm-beta/rev/194b53499fe1
status-thunderbird24:
--- → fixed
status-thunderbird25:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•