Bug 42763 - Add a diagram explaining how a DOM element is attached to the site.
Summary: Add a diagram explaining how a DOM element is attached to the site.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Dimitri Glazkov (Google)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-21 09:46 PDT by Dimitri Glazkov (Google)
Modified: 2010-07-21 10:31 PDT (History)
0 users

See Also:


Attachments
Patch (363.65 KB, patch)
2010-07-21 09:48 PDT, Dimitri Glazkov (Google)
no flags Details | Formatted Diff | Diff
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+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitri Glazkov (Google) 2010-07-21 09:46:52 PDT
Add a diagram explaining how a DOM element is attached to the site.
Comment 1 Dimitri Glazkov (Google) 2010-07-21 09:48:06 PDT
Created attachment 62198 [details]
Patch
Comment 2 Darin Adler 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?
Comment 3 Dimitri Glazkov (Google) 2010-07-21 10:18:17 PDT
Created attachment 62204 [details]
Omnigraffle Original

Here's the editable file.
Comment 4 Dimitri Glazkov (Google) 2010-07-21 10:21:43 PDT
Created attachment 62205 [details]
Patch
Comment 5 Darin Adler 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.
Comment 6 Dimitri Glazkov (Google) 2010-07-21 10:31:49 PDT
Committed r63838: <http://trac.webkit.org/changeset/63838>