Closed
Bug 666665
Opened 13 years ago
Closed 13 years ago
Remove isindex content code
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: Ms2ger, Assigned: Ms2ger)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
23.17 KB,
patch
|
sicking
:
review+
hsivonen
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Flags: in-testsuite-
Attachment #541439 -
Flags: review?(jonas)
Comment on attachment 541439 [details] [diff] [review] Patch v1 >From: Ms2ger <ms2ger@gmail.com> > >diff --git a/content/base/src/nsGkAtomList.h b/content/base/src/nsGkAtomList.h >--- a/content/base/src/nsGkAtomList.h >+++ b/content/base/src/nsGkAtomList.h >@@ -474,17 +474,16 @@ GK_ATOM(insertafter, "insertafter") > GK_ATOM(insertbefore, "insertbefore") > GK_ATOM(instanceOf, "instanceOf") > GK_ATOM(int32, "int32") > GK_ATOM(int64, "int64") > GK_ATOM(integer, "integer") > GK_ATOM(intersection, "intersection") > GK_ATOM(iscontainer, "iscontainer") > GK_ATOM(isempty, "isempty") >-GK_ATOM(isindex, "isindex") Don't know if the HTML5 parser might need (or befit from) this. r=me if Henri signs off on this too.
Attachment #541439 -
Flags: review?(jonas)
Attachment #541439 -
Flags: review?(hsivonen)
Attachment #541439 -
Flags: review+
Comment 2•13 years ago
|
||
Comment on attachment 541439 [details] [diff] [review] Patch v1 Removing this is OK. For the time being, the HTML5 parser makes an "isindex" atom exist by other means.
Attachment #541439 -
Flags: review?(hsivonen) → review+
Assignee | ||
Comment 3•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/0f00661af740
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Updated•13 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Comment 4•13 years ago
|
||
For documentation, note that we haven't created nsIDOMHTMLIsIndexElements since bug 611352. Checks for "node instanceof nsIDOMHTMLIsIndexElement" can be replaced with "node instanceof nsIDOMHTMLInputElement".
Comment 5•13 years ago
|
||
Added notes to Firefox 8 release notes. The interface is not documented elsewhere on MDC. https://developer.mozilla.org/en/Firefox_8_for_developers#section_2
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•