Closed Bug 903402 Opened 11 years ago Closed 11 years ago

Permanent Orange: TEST-UNEXPECTED-FAIL | test_index_junk_imap_offline.js | uncaught exception: 2147500037

Categories

(MailNews Core :: Testing Infrastructure, defect)

Product:
MailNews Core
Component:
Testing Infrastructure
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(thunderbird24 fixed, thunderbird25 fixed)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 26.0
Tracking Flags:
Tracking Status
thunderbird24 --- fixed
thunderbird25 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Workaround
1.03 KB, patch
jcranmer
: review+
standard8
: approval-comm-aurora+
standard8
: approval-comm-beta+
Details | Diff | Splinter Review 
Seen since we've had the major orange:

https://tbpl.mozilla.org/php/getParsedLog.php?id=26347955&tree=Thunderbird-Trunk#error0

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
Attached patch Workaround — — Splinter Review 
I've traced this through via the debugger. This is roughly what is happening:

- The test sets up the message folders, and configures for offline storage
- Attempting to get one of the messages (in test_never_indexes_a_message_marked_as_junk), causes the IMAP service to initialise the offline cache
- It then looks in the offline storage, it is not there, so it looks in the "browser" cache
- The "browser" cache gets initialised, and as part of this it looks for "cachePDir" from the nsIDirectoryServiceProvider.
- The implemented directory service providers throw as they should do.
- The javascript console records the error as uncaught, even though c++ handles the result.

So I think this is really some sort of xpconnect issue. Unfortunately I've not been able to come up with a simple test case just yet.

I'm therefore proposing that we work around it by setting two prefs that will mean we avoid calling the directory service with "cachePDir". (See nsCacheProfilePrefObserver::ReadPrefs for more info).

When I get time, I'll then see if I can come up with a test case later on and get a core bug filed.
Assignee: nobody → mbanner
Status: NEW → ASSIGNED
Attachment #788118 - Flags: review?(Pidgeot18)
Attachment #788118 - Flags: review?(Pidgeot18) → review+
https://hg.mozilla.org/comm-central/rev/1526a145fd9e
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 26.0
Comment on attachment 788118 [details] [diff] [review]
Workaround

[Triage Comment]
Passes on trunk, so taking to aurora & beta
Attachment #788118 - Flags: approval-comm-beta+
Attachment #788118 - Flags: approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: