The link checker backend works, but we need a way to display it that's better than printing to stdout. Even a simple UI would be enough to allow us to turn it on in the UI.

The backend broke temporarily; there's a patch in bug 91388 for that.

Will do this soon!

Darn, didn't finish in time for 0.9.7

Comment on attachment 63903 [details] [diff] [review] First draft to use for testing r=akkana

Comment on attachment 63903 [details] [diff] [review] First draft to use for testing sr=ben@netscape.com

Charley's patch is messed up, line numbers that don't correspond to the actual number of lines in the patch, etc. I'm about to attach a patch that should (I hope) work better. But it doesn't work for me -- the dialog is blank, and stays blank even after stdout messages say that the callback has fired. I don't think the checker code is quite working, either; I'm getting a dump of Exception ERROR in Link checker. e.result=2147746065, gNumLinksToCheck=4 for every link, even links that do exist. So something is wrong (perhaps in the way I hand-applied the patch?)

It was difficult to produce a clean patch because of conflicts with other fixes in my tree. I checked my files based on the "first draft" patch, which did work (i.e., did display dialog without the exception error.) I'll reconcile with Akkana's patch next.

moving milestone

The addref problem causing the crash described above is fixed. We can proceed with this, which is closely related to the publishing feature.

changing milestone

This leverages new Publish Progress patterns to show a simple listbox of links to be checked. In this version, I get an exception error after finding the first link to check.

Please update this bug with an [adt1] - [adt3] impact rating (or take it off the list if it doesn't even rate adt3.) Thanks!

not going to make it for 1.0

nsbeta1+ per buffy triage

Composer triage team: nsbeta1+/adt2 It appears we need to move the ``Check Links'' menuitem from the ``Debug'' menu to the ``Tools'' menu.

Composer triage team: nsbeta1+/adt3

Nav triage team: nsbeta1-

(In reply to comment #17) > It appears we need to move the ``Check Links'' menuitem from the ``Debug'' menu > to the ``Tools'' menu. Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.1pre) Gecko/2008062601 SeaMonkey/2.0a1pre "Check Links" is still under "Debug" (in the Composer window). So: Jag, are you still working on this, should it be reassigned to nobody@ -- or else resolved WONTFIX? (BTW, I'm removing "Mozilla 1.4 beta" as the target milestone. Feel free to set something more realistic.)