The default bug view has changed. See this FAQ.

Permanent Orange (windows): TEST-UNEXPECTED-FAIL | test_partsOnDemand.js | test_attachment_size.js | uncaught exception: 2147500037

RESOLVED FIXED in Thunderbird 26.0

Status

MailNews Core
Testing Infrastructure
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: standard8, Assigned: hiro)

Tracking

({intermittent-failure})

Trunk
Thunderbird 26.0
All
Windows 7
intermittent-failure

Thunderbird Tracking Flags

(thunderbird24 fixed, thunderbird25 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

4 years ago
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 (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 3

4 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 (Treeherder Robot)
(Assignee)

Comment 5

4 years ago
Created attachment 792675 [details] [diff] [review]
Remove_file_extension_to_avoid_test_orange_on_windows.patch

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

4 years ago
Created attachment 792678 [details] [diff] [review]
[checked in] Remove_file_extension_to_avoid_test_orange_on_windows.patch

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

4 years ago
Created attachment 792683 [details] [diff] [review]
Possible fix for test_partsOnDemand.js

I am not really convinced, but this patch might solve the failure of test_partsOnDemand.js. I hope!
(Assignee)

Comment 8

4 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)
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

4 years ago
Thank you, Archaeopteryx. But try server has still problems unfortunately.
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Comment 32

4 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

4 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

4 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 (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Comment 37

4 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

4 years ago
https://hg.mozilla.org/comm-central/rev/c0baf3d087a3
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 26.0
(Reporter)

Comment 39

4 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.