Closed Bug 379454 Opened 17 years ago Closed 17 years ago

nsIScriptableUnescapeHTML (feeds.xpt) missing in Windows version

Categories

(Thunderbird :: General, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sebastian.kirsche, Assigned: philor)

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3
Build Identifier: 2.0.0.0 (20070326) and 2.0.0.4pre (20070501)

The interface nsIScriptableUnescapeHTML is missing in the Windows version of Thunderbird. It is available in Firefox for Windows and Thunderbird for GNU/Linux and Mac OS X. This interface seems to be contained in the file feeds.xpt, which doesn't exist in the Windows build.

There's a post about it in the newsgroup:
http://groups.google.com/group/mozilla.dev.extensions/browse_thread/thread/2a95e2d13cac1ef1

Reproducible: Always

Steps to Reproduce:
Run the following JavaScript code inside a custom extension: 

var unescapeHTMLTemp = Components.classes["@mozilla.org/feed-unescapehtml;1"];
var unescapeHTML = unescapeHTMLTemp.getService(Components.interfaces.nsIScriptableUnescapeHTML);
alert(unescapeHTML.unescape("<ul><li><b>Hello</b></li><li>World<li></ul>"));

Actual Results:  
The second statement fails with the following message:

Error: uncaught exception: [Exception... "Invalid InterfaceID"
nsresult: "0x80570018 (NS_ERROR_XPC_BAD_IID)" 

Expected Results:  
A alert window that says:

* Hello
* World
Version: unspecified → 2.0
I can confirm the above works on linux...
Certainly a bug that we ship 21 .xpts on Linux (18 of them on Mac) that we don't ship on Windows, though I'm not entirely sure whether it's generally a bug that we ship them, or that we don't.

I'm pretty sure, though, that we shouldn't ship feeds.xpt without saxparser.xpt.
Assignee: mscott → philringnalda
Status: UNCONFIRMED → NEW
Ever confirmed: true
Version: 2.0 → Trunk
Attached patch FixSplinter Review
Though what matters is the 14 that we actually build, but don't package, on Windows. And after spending far too long pondering whether or not anyone could do anything useful with necko_viewsource.xpt or contentprefs.xpt, I finally realized that they could just ask for them, if they need them (well, assuming they can figure out that they need them). But these two, we know that not only does Sebastian want them, we'll want them soon enough too.
Attachment #278268 - Flags: review?(mscott)
Attachment #278268 - Flags: review?(mscott) → review+
mail/installer/windows/packages-static 1.75
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: