Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Remove tag observers

RESOLVED FIXED in mozilla12

Status

()

Core
HTML: Parser
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: hsivonen, Assigned: hsivonen)

Tracking

Trunk
mozilla12
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Tag observers have been unused since Firefox 4. Let's get rid of the code.
Created attachment 587645 [details] [diff] [review]
Remove the dead code

Another patch that just removes old code.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #587645 - Flags: review?(mrbkap)
Summary: Remote tag observers → Remove tag observers
Comment on attachment 587645 [details] [diff] [review]
Remove the dead code

Review of attachment 587645 [details] [diff] [review]:
-----------------------------------------------------------------

::: parser/htmlparser/public/nsIHTMLContentSink.h
@@ -221,5 @@
> -   * the tag
> -   *
> -   * @param aErrorResult the error code
> -   */
> -  NS_IMETHOD NotifyTagObservers(nsIParserNode* aNode) = 0;

Need to bump the IID here.
Attachment #587645 - Flags: review?(mrbkap) → review+
Thanks. Landed with revised IID.
https://hg.mozilla.org/integration/mozilla-inbound/rev/418f713f7092
No need for add-on compat keywords, because this stuff hasn't actually been used since Firefox 4.
https://hg.mozilla.org/mozilla-central/rev/418f713f7092
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.