Closed Bug 56224 Opened 24 years ago Closed 24 years ago

Wrong RDF value in mimeTypes.rdf for NC:handleinteral text/html

Categories

(SeaMonkey :: UI Design, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.8

People

(Reporter: jrgmorrison, Assigned: timeless)

References

()

Details

Attachments

(1 file)

In http://lxr.mozilla.org/mozilla/source/profile/defaults/mimeTypes.rdf there is the following RDF. 19 <NC:handlerProp> 20 <NC:handler about="urn:mimetype:handler:text/html"> 21 <NC:saveToDisk>false</NC:saveToDisk> 22 <NC:handleInternal>true</NC:handleInternal> 23 <NC:handleInternal>false</NC:handleInternal> 24 <NC:externalApplicationProp> The second instance of NC:handleInternal, overrides the first, making the value 'false' when it should be 'true'. Ben says "not good". Details?
rtm -- assuming that this may hork some aspect of content-handling in the browser or mailnews.
Keywords: rtm
nav triage team: rtm- because no user scenario described, too late in the schedule.
Whiteboard: [rtm-]
QA Contact: sairuh → shrir
This won't effect any runtime behavior for seamonkey. We don't use this text/html entry at run time. I put it in there for test and to have an example value for the UI to show the user. It's not correct that I put that flag in there twice but we aren't using that information...rtm- is the right thing to do for this one.
When i click that entry in prefs, JS error appears: JavaScript error: line 4: uncaught exception: [Exception... "Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface]" nsresult: "0x80004002 (NS_NOINTERFACE)" location: "JS frame :: chrome://communicator/content/pref/overrideHandler.js :: anonymous :: line 103" data: no] If i click "edit" button, the same error. In addition, no entries about the type is filled in in the edit dialog. When i dismiss the Edit dialog, the same JS error appears once more. Mentioning this here in case it is related.
The above comment is already filed as bug 56226 (as a simply bugzilla query would have found).
Attached patch sillySplinter Review
would someone r= this? ben would you a= this?
Assignee: ben → timeless
Whiteboard: [rtm-]
Target Milestone: --- → mozilla0.8
r=jrgm
sr=alecf
Status: NEW → ASSIGNED
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
verified that the fix was checked-in in version 1.4 in lxr.
Product: Core → Mozilla Application Suite
Component: XP Apps: GUI Features → UI Design
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: