Generate and upload class inheritance diagrams for MDC

RESOLVED FIXED

Status

Firefox Build System
Source Code Analysis
RESOLVED FIXED
10 years ago
2 months ago

People

(Reporter: Benjamin Smedberg, Assigned: Benjamin Smedberg)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 344131 [details] [diff] [review]
Graph string class hierarchy, rev. 1

This is a small update for the automatic string documentation: it takes the two base classes nsAString and nsACString and generates a dot-format graph of all subclasses. It then converts this to PNG and a image map and uploads all that to MDC.

Asking jorendorff for review mostly because the hairiest parts of the patch are the additions to deki.py
Attachment #344131 - Flags: review?(jorendorff)
Attachment #344131 - Flags: review?(jorendorff) → review+
Comment on attachment 344131 [details] [diff] [review]
Graph string class hierarchy, rev. 1

>+    def create_file(self, page, name, data, mimetype, description=None):

Please name this something like attach_file, save_file, or upload_file.  (You don't just use it to create files.)  Given "MDC-attach.py", maybe attach_file is best.

>+          data - fd or str - a string or file object with the file data

Does this really work with a filelike object?  I couldn't find where that's implemented.

Do you want to revise this patch now that the graphviz plugin is supposedly up and running on MDC?

If not, r=me with the first 2 comments addressed.
(Assignee)

Comment 2

10 years ago
http://hg.mozilla.org/mozilla-central/rev/6a2e65f15f48

No, it doesn't work with file-like objects, I was confused about what urllib2 accepted.

I would like to do this with the graphviz plugin, but that doesn't work yet... probably some configuration issue, but I'm not sure.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Updated

2 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.