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)
Tracking
()
VERIFIED
FIXED
M5
People
(Reporter: teruko, Assigned: ftang)
References
Details
Tested Jan 12 Win32 build.
No encoding menu, so we cannot test the different encoding.
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Comment 2•26 years ago
|
||
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Comment 4•26 years ago
|
||
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.
Assignee | ||
Comment 5•26 years ago
|
||
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.
Assignee | ||
Comment 6•26 years ago
|
||
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.
Comment 7•26 years ago
|
||
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?
Comment 8•26 years ago
|
||
The messenger charset menu issue was separated as #3966.
Assignee | ||
Updated•26 years ago
|
Summary: [I18n]No encoding menu. → Charset Handling- No encoding menu.
Assignee | ||
Updated•26 years ago
|
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
Assignee | ||
Comment 9•26 years ago
|
||
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"
Comment 10•26 years ago
|
||
Frank -- I'm giving this to you until we start calling:
nsParser::SetDefaultCharset().
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M4 → M5
Assignee | ||
Updated•26 years ago
|
Hardware: PC → All
Assignee | ||
Comment 11•26 years ago
|
||
Change to M5 from M4 since neeti and rickg have not implement the META tag
observer yet.
Assignee | ||
Comment 12•26 years ago
|
||
change platform to all
Assignee | ||
Comment 13•26 years ago
|
||
*** Bug 4869 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 14•26 years ago
|
||
encoding menu is done. We now pass the Parser the default charset from the
webshell.
Reporter | ||
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 15•26 years ago
|
||
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.
Description
•