Closed Bug 1577443 Opened 4 years ago Closed 4 years ago

Remove cmd_checkLinks that is unused


(SeaMonkey :: Composer, enhancement)

SeaMonkey 2.67
Not set


(seamonkey2.49esr wontfix, seamonkey2.63 wontfix, seamonkey2.53 fixed, seamonkey2.57esr fixed)

Tracking Status
seamonkey2.49esr --- wontfix
seamonkey2.63 --- wontfix
seamonkey2.53 --- fixed
seamonkey2.57esr --- fixed


(Reporter: m_kato, Assigned: iannbugzilla)



(Whiteboard: SM2.53.1)


(3 files, 1 obsolete file)

cmd_checkLinks seems not to use in Thunderbird and SeaMonkey. Although debugQAEditorOverlay.xul has menu for this command, this is debugging tools.

This is only user of nsIHTMLEditor.getLinkedObjects that is unused by Firefox. So I hope that cmd_checkLinks is removed since it is unused since I would like to remove nsIHTMLEditor.getLinkedObjects from Gecko.

Ian, is cmd_checkLinks still needed? This seems to be used on debugQA extension only. Although this command uses nsIHTMLEditor.getLinkedObjects method, it isn't used in Firefox and I would like to remove this method from Gecko.

Flags: needinfo?(iann_bugzilla)

I think it can be removed. It is a debugging help for Composer but seems to do nothing.

As cmd_checkLinks also relies on URIChecker which was removed in Bug 1222829 (in Gecko 45 around 4 years ago and spotted by me a year later), I'm fairly sure that nsIHTMLEditor.getLinkedObjects method can be removed.

Flags: needinfo?(iann_bugzilla)
Attached patch Remove checkLinks for cc (obsolete) — Splinter Review

Patch for comm-central that doesn't remove l10n

Assignee: nobody → iann_bugzilla
Attachment #9089457 - Flags: review?(frgrahl)
Attachment #9089457 - Attachment description: 1577443_remove_checkLinks_cc.patch → Remove checkLinks for cc

Patch for SM 2.57, does have l10n string removal

Attachment #9089460 - Flags: review?(frgrahl)
Attachment #9089460 - Flags: approval-comm-esr60?

[Approval Request Comment]
Regression caused by (bug #): 1222829
User impact if declined: None, doesn't work anyway
Testing completed (on m-c, etc.): 2.53
Risk to taking this patch (and alternatives if risky): None
String changes made by this patch: One string removal and one dtd file removal

Attachment #9089463 - Flags: review?(frgrahl)
Attachment #9089463 - Flags: approval-comm-release?

Include l10n removals as requested

Attachment #9089457 - Attachment is obsolete: true
Attachment #9089457 - Flags: review?(frgrahl)
Attachment #9089553 - Flags: review?(frgrahl)
Comment on attachment 9089553 [details] [diff] [review]
Remove checkLinks for cc v1.1

And away it goes
Attachment #9089553 - Flags: review?(frgrahl) → review+
Comment on attachment 9089463 [details] [diff] [review]
Remove checkLinks for 2.53

Attachment #9089463 - Flags: review?(frgrahl)
Attachment #9089463 - Flags: review+
Attachment #9089463 - Flags: approval-comm-release?
Attachment #9089463 - Flags: approval-comm-release+
Comment on attachment 9089460 [details] [diff] [review]
Remove checkLinks for 2.57

Attachment #9089460 - Flags: review?(frgrahl)
Attachment #9089460 - Flags: review+
Attachment #9089460 - Flags: approval-comm-esr60?
Attachment #9089460 - Flags: approval-comm-esr60+

Pushed by
Remove cmd_checkLinks that is unused. r=frg DONTBUILD

Closed: 4 years ago
Resolution: --- → FIXED

Fixed. Pushed with DONT BUILD all suite code (composer).

Version: unspecified → SeaMonkey 2.67
Whiteboard: SM2.53.1
You need to log in before you can comment on or make changes to this bug.