Closed
Bug 379454
Opened 17 years ago
Closed 17 years ago
nsIScriptableUnescapeHTML (feeds.xpt) missing in Windows version
Categories
(Thunderbird :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sebastian.kirsche, Assigned: philor)
Details
Attachments
(1 file)
872 bytes,
patch
|
mscott
:
review+
|
Details | Diff | Splinter Review |
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
Reporter | ||
Updated•17 years ago
|
Version: unspecified → 2.0
Comment 1•17 years ago
|
||
I can confirm the above works on linux...
Assignee | ||
Comment 2•17 years ago
|
||
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
Assignee | ||
Comment 3•17 years ago
|
||
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)
Updated•17 years ago
|
Attachment #278268 -
Flags: review?(mscott) → review+
Assignee | ||
Comment 4•17 years ago
|
||
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.
Description
•