Closed Bug 683482 Opened 13 years ago Closed 13 years ago

remove replaceWholeText()

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla10

People

(Reporter: annevk, Assigned: matjk7)

References

Details

(Keywords: addon-compat, dev-doc-complete)

Attachments

(1 file, 4 obsolete files)

Per http://www.w3.org/Bugs/Public/show_bug.cgi?id=13046 it would be great if this method got removed so it can be removed from DOM Core.

(Acid3 relies on this, but Acid3 needs to change anyway.)
OS: Mac OS X → All
Hardware: x86 → All
Version: unspecified → Trunk
Assignee: nobody → matjk7
Status: NEW → ASSIGNED
Attached patch patch (obsolete) — Splinter Review
Attachment #560768 - Flags: review?(jonas)
Comment on attachment 560768 [details] [diff] [review]
patch

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

yay
Attachment #560768 - Flags: review?(jonas) → review+
Keywords: checkin-needed
How confident are we that this is not going to break any website? Should we wait for the next merge (in 8 days) to increase the test period?
(In reply to Mounir Lamouri (:volkmar) (:mounir) from comment #3)
> How confident are we that this is not going to break any website? Should we
> wait for the next merge (in 8 days) to increase the test period?

You're right this doesn't need to land right now.
Keywords: checkin-needed
Whiteboard: [land for Firefox 10]
Keywords: dev-doc-needed
If you're doing that, please add a warning to current builds as to increase the chances that anyone using this fixes their site.
This patch would be useful if you want to use nsIDocument::WarnOnceAbout. (We've got 32 warnings already.)
Depends on: 688190
Keywords: checkin-needed
Whiteboard: [land for Firefox 10]
Attachment #561137 - Attachment is obsolete: true
This patch isn't going to apply, due to the warning added in bug 688190, which you also need to remove.
Keywords: checkin-needed
Attached patch patch for checkin (obsolete) — Splinter Review
Updated to tip.
Attachment #560768 - Attachment is obsolete: true
Attachment #563176 - Flags: review+
Keywords: checkin-needed
You forgot to update dom.properties.
Keywords: checkin-needed
Comment on attachment 563176 [details] [diff] [review]
patch for checkin

... and nsDeprecatedOperationList.h
Attachment #563176 - Flags: review+
Attached patch patch for checkin (obsolete) — Splinter Review
Attachment #563176 - Attachment is obsolete: true
Attachment #563183 - Flags: review+
Keywords: checkin-needed
patching file content/base/src/nsGenericDOMDataNode.h
Hunk #1 FAILED at 324
Attachment #563183 - Attachment is obsolete: true
Attachment #563575 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/5f0f1b44e73d
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: