Closed Bug 2341 Opened 26 years ago Closed 26 years ago

Charset Handling- Parser does not use default charset value set by Default Charset Menu

Categories

(Core :: DOM: HTML Parser, defect, P1)

All
Windows 98
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: teruko, Assigned: ftang)

References

Details

Tested Jan 12 Win32 build. No encoding menu, so we cannot test the different encoding.
Status: NEW → ASSIGNED
Setting all current Open Critical and Major to M3
per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
QA Contact: 3853 → 3851
Setting QA Contact to teruko.
Bob, is there a way to hack this without having encoding selection on the menu. If there is a way to do this testing right now lets just release not that way.
Hook up the Default charset menu w/ nsParser may need to change the nsParser::Parse interface. I think we probably should do this post dog food. And I need to talk to rickg about the interface change. There is a work around for meta tag pages so we don't need this for browser. We do need this to test Japanese mail on window but naoki is working on a hack in libmime to reduce the need. If that hack is check in, I don't think we need this for M3. Change this to M4.
Hook up the Default charset menu w/ nsParser may need to change the nsParser::Parse interface. I think we probably should do this post dog food. And I need to talk to rickg about the interface change. There is a work around for meta tag pages so we don't need this for browser. We do need this to test Japanese mail on window but naoki is working on a hack in libmime to reduce the need. If that hack is check in, I don't think we need this for M3. Change this to M4.
Component: Viewer App → Internationalization
When activating a charset menu, depends on a context, more actions may be needed than just notify the charset change to the parser. For example, if a mail folder is selected, we need to notify the messenger to change a charset attribute of the mail folder. Should I create a new bug for this?
The messenger charset menu issue was separated as #3966.
Summary: [I18n]No encoding menu. → Charset Handling- No encoding menu.
Assignee: ftang → rickg
Status: ASSIGNED → NEW
Component: Internationalization → Parser
Summary: Charset Handling- No encoding menu. → Charset Handling- Parser does not use default charset value set by Default Charset Menu
We have put in the Default Charset Menu for browser, hookup the JS w/ BrowserAppsCore and pass the info into nsWebShell. Now the problem is the Parser ( or the code which create the parser) does not get that information and pass it to Parser. Reassign this to rickg since this need API change in parser. Change the summary from "Charset Handling- No encoding menu." to "Charset Handling- Parser does not use default charset value set by Default Charset Menu"
Assignee: rickg → ftang
Frank -- I'm giving this to you until we start calling: nsParser::SetDefaultCharset().
Status: NEW → ASSIGNED
Target Milestone: M4 → M5
Hardware: PC → All
Change to M5 from M4 since neeti and rickg have not implement the META tag observer yet.
change platform to all
*** Bug 4869 has been marked as a duplicate of this bug. ***
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
encoding menu is done. We now pass the Parser the default charset from the webshell.
Status: RESOLVED → VERIFIED
Character Default menu is working, so I mark this as verified.
You need to log in before you can comment on or make changes to this bug.