Last Comment Bug 903946 - TEST-UNEXPECTED-FAIL | ../../../resources/logHelper.js | Error console says [stackFrame uncaught exception: 2147500037] caused by nsIExternalProtocolService
: TEST-UNEXPECTED-FAIL | ../../../resources/logHelper.js | Error console says [...
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Testing Infrastructure (show other bugs)
: Trunk
: All Linux
: -- normal (vote)
: Thunderbird 26.0
Assigned To: Hiroyuki Ikezoe (:hiro)
:
:
Mentors:
Depends on:
Blocks: 896738
  Show dependency treegraph
 
Reported: 2013-08-11 22:23 PDT by Hiroyuki Ikezoe (:hiro)
Modified: 2013-08-12 06:15 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
fixed
fixed


Attachments
fix_for_bug903946.patch (1.49 KB, patch)
2013-08-11 22:32 PDT, Hiroyuki Ikezoe (:hiro)
standard8: review+
standard8: approval‑comm‑aurora+
standard8: approval‑comm‑beta+
Details | Diff | Splinter Review

Description Hiroyuki Ikezoe (:hiro) 2013-08-11 22:23:52 PDT
From https://tbpl.mozilla.org/php/getParsedLog.php?id=26350615#error3


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 1 Hiroyuki Ikezoe (:hiro) 2013-08-11 22:32:46 PDT
Created attachment 788787 [details] [diff] [review]
fix_for_bug903946.patch

This errors are happened in nsIExternalProtocolService.

On linux, helpers.private_mime_types_file is set to ~/.mailcap and helpers.private_mime_types_file is set to ~/.mime.types. But those files do not exist normally. 

This patch fixes errors of the following tests:

test_detachToFile.js
test_junkingWhenDisabled.js
test_mimealtdetach.js
test_partsOnDemand.js
test_hidden_attachments.js
test_attachment_size.js
Comment 2 Mark Banner (:standard8) 2013-08-12 05:29:41 PDT
Comment on attachment 788787 [details] [diff] [review]
fix_for_bug903946.patch

Looks great, thanks.
Comment 3 Mark Banner (:standard8) 2013-08-12 05:47:52 PDT
https://hg.mozilla.org/comm-central/rev/dbfea6e6fb52
Comment 4 Mark Banner (:standard8) 2013-08-12 05:48:37 PDT
Comment on attachment 788787 [details] [diff] [review]
fix_for_bug903946.patch

[Triage Comment]
Taking onto branches as we'll need this there as well.

Note You need to log in before you can comment on or make changes to this bug.