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)

All
Windows 7
defect
Not set
normal

Tracking

(thunderbird24 fixed, thunderbird25 fixed)

RESOLVED FIXED
Thunderbird 26.0
Tracking Status
thunderbird24 --- fixed
thunderbird25 --- fixed

People

(Reporter: standard8, Assigned: hiro)

References

Details

(Keywords: intermittent-failure)

Attachments

(2 files, 1 obsolete file)

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
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?
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)
The previous patch was wrong one....
Attachment #792675 - Attachment is obsolete: true
Attachment #792675 - Flags: review?(mbanner)
Attachment #792678 - Flags: review?(mbanner)
I am not really convinced, but this patch might solve the failure of test_partsOnDemand.js. I hope!
Archaeopteryx, could you please push both two patches on try server with "-p all -u xpcshell -t none" options.

Thank you,
Flags: needinfo?(archaeopteryx)
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)
Thank you, Archaeopteryx. But try server has still problems unfortunately.
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+
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+
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 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+
https://hg.mozilla.org/comm-central/rev/c0baf3d087a3
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 26.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: