Bug 42763

Summary: Add a diagram explaining how a DOM element is attached to the site.
Product: WebKit Reporter: Dimitri Glazkov (Google) <dglazkov>
Component: New BugsAssignee: Dimitri Glazkov (Google) <dglazkov>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Omnigraffle Original
none
Patch darin: review+

Dimitri Glazkov (Google)
Reported 2010-07-21 09:46:52 PDT
Add a diagram explaining how a DOM element is attached to the site.
Attachments
Patch (363.65 KB, patch)
2010-07-21 09:48 PDT, Dimitri Glazkov (Google) 		no flags
DetailsFormatted DiffDiff
Omnigraffle Original (65.94 KB, application/octet-stream)
2010-07-21 10:18 PDT, Dimitri Glazkov (Google) 		no flags
Details
Patch (363.58 KB, patch)
2010-07-21 10:21 PDT, Dimitri Glazkov (Google) 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dimitri Glazkov (Google)
Comment 1 2010-07-21 09:48:06 PDT
Created attachment 62198 [details] Patch
Darin Adler
Comment 2 2010-07-21 10:09:05 PDT
Comment on attachment 62198 [details] Patch I see a typo in the diagram: "initalizes". Is there a way to attach the diagram in an editbale form?
Dimitri Glazkov (Google)
Comment 3 2010-07-21 10:18:17 PDT
Created attachment 62204 [details] Omnigraffle Original Here's the editable file.
Dimitri Glazkov (Google)
Comment 4 2010-07-21 10:21:43 PDT
Created attachment 62205 [details] Patch
Darin Adler
Comment 5 2010-07-21 10:25:02 PDT
Comment on attachment 62205 [details] Patch It's fine to add this diagram. Please make sure that editable form is in Subversion too, so in the future someone doesn't have to start over from scratch if there are changes. Also, I'm worried about a diagram that has a detailed list of functions and how they work since it's likely we will change that later.
Dimitri Glazkov (Google)
Comment 6 2010-07-21 10:31:49 PDT
Committed r63838: <http://trac.webkit.org/changeset/63838>
Note You need to log in before you can comment on or make changes to this bug.