New issue
Advanced search Search tips

Issue 742715 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Explain how DOM works, in terms of document tree, shadow tree, flat tree and layout tree

Project Member Reported by hayato@chromium.org, Jul 14 2017

Issue description

We should have a good documentation about how DOM trees works in Blink.

That should cover:

- A concept of document tree, shadow tree, flat tree and layout tree
- How these trees are interacted, and how these trees are constructed from others 
- The role of UpdateDistribution and when and why we should call
- The role of <slot> element

core/dom/README.md might be a good place to explain.
 
Labels: -Pri-3 Pri-2
Owner: hayato@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 9 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/08fe21982eb37d9ba58cf29f28e141155923d72f

commit 08fe21982eb37d9ba58cf29f28e141155923d72f
Author: Hayato Ito <hayato@chromium.org>
Date: Wed Aug 09 08:31:07 2017

Documentation: Update core/dom/README.md

Add "Node and DOM Tree" section.
Future CLs will add more sections.

TBR=kochi

Bug: 742715
Change-Id: I78adb1a702519bd758efe508af6d9b95a8582395
Reviewed-on: https://chromium-review.googlesource.com/607751
Commit-Queue: Hayato Ito <hayato@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#492904}
[modify] https://crrev.com/08fe21982eb37d9ba58cf29f28e141155923d72f/third_party/WebKit/Source/core/dom/README.md

Project Member

Comment 3 by bugdroid1@chromium.org, Aug 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/48ab5901c0f546363ff8b44076daeb0c8ab51dc8

commit 48ab5901c0f546363ff8b44076daeb0c8ab51dc8
Author: Hayato Ito <hayato@chromium.org>
Date: Thu Aug 10 08:04:38 2017

Documentation: Update core/dom/README.md

Add "C++11 range-based for loops for traversing a tree" section.

TBR=kochi

Bug: 742715
Change-Id: I389b2bc6281dbb07ad9dac89cefcdd836f835587
Reviewed-on: https://chromium-review.googlesource.com/608731
Commit-Queue: Hayato Ito <hayato@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493328}
[modify] https://crrev.com/48ab5901c0f546363ff8b44076daeb0c8ab51dc8/third_party/WebKit/Source/core/dom/README.md

Project Member

Comment 4 by bugdroid1@chromium.org, Aug 16 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bc33717af8bea2e2c11c250798f38568443a6040

commit bc33717af8bea2e2c11c250798f38568443a6040
Author: Hayato Ito <hayato@chromium.org>
Date: Wed Aug 16 10:07:07 2017

Documentation: Update core/dom/README.md

Add "Shadow Tree" section.

TBR=kochi

Bug: 742715
Change-Id: Ia984646f5dd2b3f23c26d141454800e03eb58428
Reviewed-on: https://chromium-review.googlesource.com/616370
Reviewed-by: Hayato Ito <hayato@chromium.org>
Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#494741}
[modify] https://crrev.com/bc33717af8bea2e2c11c250798f38568443a6040/third_party/WebKit/Source/core/dom/README.md

Project Member

Comment 5 by bugdroid1@chromium.org, Aug 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0a7a8c624ba28f9129d6e273aee220fa2477080b

commit 0a7a8c624ba28f9129d6e273aee220fa2477080b
Author: Hayato Ito <hayato@chromium.org>
Date: Thu Aug 17 07:29:50 2017

Documentation: Update core/dom/README.md

Add "TreeScope" section.

TBR=kochi

Bug: 742715
Change-Id: Ie19497ad9f79dc143f9743c34080cc64013b2e39
Reviewed-on: https://chromium-review.googlesource.com/617849
Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495099}
[modify] https://crrev.com/0a7a8c624ba28f9129d6e273aee220fa2477080b/third_party/WebKit/Source/core/dom/README.md

Project Member

Comment 6 by bugdroid1@chromium.org, Aug 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c46223f38615c5da0a8ac68be1484177531c3799

commit c46223f38615c5da0a8ac68be1484177531c3799
Author: Hayato Ito <hayato@chromium.org>
Date: Tue Aug 22 09:21:25 2017

Documentation: Update core/dom/README.md

Add "Composed Tree" section.

TBR=kochi

Bug: 742715
Change-Id: I603891669501dd14c51218ec115ffa6260fa3942
Reviewed-on: https://chromium-review.googlesource.com/625601
Reviewed-by: Hayato Ito <hayato@chromium.org>
Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#496259}
[modify] https://crrev.com/c46223f38615c5da0a8ac68be1484177531c3799/third_party/WebKit/Source/core/dom/README.md

Project Member

Comment 7 by bugdroid1@chromium.org, Aug 25 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/12c9d7d17d2bf7ba6be49cb04a5d0226b46c2a21

commit 12c9d7d17d2bf7ba6be49cb04a5d0226b46c2a21
Author: Hayato Ito <hayato@chromium.org>
Date: Fri Aug 25 08:34:39 2017

Documentation: Update core/dom/README.md, adding "Flat Tree" section

TBR=kochi

Bug: 742715
Change-Id: Ie7888c9dd2953a1c6001913b251478d725dbce9f
Reviewed-on: https://chromium-review.googlesource.com/635063
Commit-Queue: Hayato Ito <hayato@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#497356}
[modify] https://crrev.com/12c9d7d17d2bf7ba6be49cb04a5d0226b46c2a21/third_party/WebKit/Source/core/dom/README.md

Project Member

Comment 8 by bugdroid1@chromium.org, Oct 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e4c1553194483b9134d60fcb60aae320c91cfb67

commit e4c1553194483b9134d60fcb60aae320c91cfb67
Author: Hayato Ito <hayato@chromium.org>
Date: Thu Oct 19 06:48:03 2017

Documentation: Update core/dom/README.md, adding a link to "Incremental Shadow DDM"

TBR=kochi

Bug: 742715
Change-Id: I1bc4e37d5d1e707276f38bc11dfae2fe93c2d260
Reviewed-on: https://chromium-review.googlesource.com/727389
Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510019}
[modify] https://crrev.com/e4c1553194483b9134d60fcb60aae320c91cfb67/third_party/WebKit/Source/core/dom/README.md

Project Member

Comment 9 by bugdroid1@chromium.org, Jul 3

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/80c08c3f413136b8c1ec4c3c5c3d51f712ab0638

commit 80c08c3f413136b8c1ec4c3c5c3d51f712ab0638
Author: Hayato Ito <hayato@chromium.org>
Date: Tue Jul 03 04:26:40 2018

Format renderer/core/dom/README.md

Format renderer/core/dom/README.md by prettier.

BUG: 742715
TBR=kochi
NOTRY=true

Change-Id: Ib67571d3490a64681ef32f2bfcc47578e7282012
Reviewed-on: https://chromium-review.googlesource.com/1123975
Commit-Queue: Hayato Ito <hayato@chromium.org>
Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572105}
[modify] https://crrev.com/80c08c3f413136b8c1ec4c3c5c3d51f712ab0638/third_party/blink/renderer/core/dom/README.md

Project Member

Comment 10 by bugdroid1@chromium.org, Aug 1

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5c364a015037a91cab0538f6fd9910bef7679642

commit 5c364a015037a91cab0538f6fd9910bef7679642
Author: Hayato Ito <hayato@chromium.org>
Date: Wed Aug 01 02:18:19 2018

Documentation: Update renderer/core/dom/README.md

Add "Slots and node assignments" section.

TBR=rakina

Bug: 742715
Change-Id: I4e5ad0da310c186643ce42695459a17d1a853654
Reviewed-on: https://chromium-review.googlesource.com/1157732
Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579671}
[modify] https://crrev.com/5c364a015037a91cab0538f6fd9910bef7679642/third_party/blink/renderer/core/dom/README.md

Project Member

Comment 11 by bugdroid1@chromium.org, Aug 1

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c3d4ae8eb2ba132ad0954a1e9c3584860247bb6c

commit c3d4ae8eb2ba132ad0954a1e9c3584860247bb6c
Author: Hayato Ito <hayato@chromium.org>
Date: Wed Aug 01 02:34:15 2018

Documentation: Update core/dom/README.md

Fix links to *.{h,cc} files.

TBR=rakina

Bug: 742715
Change-Id: I13df9ba392405dfdba37d12279b98dccc35be1a2
Reviewed-on: https://chromium-review.googlesource.com/1157736
Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579675}
[modify] https://crrev.com/c3d4ae8eb2ba132ad0954a1e9c3584860247bb6c/third_party/blink/renderer/core/dom/README.md

Project Member

Comment 12 by bugdroid1@chromium.org, Aug 1

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8b12538158aa93802bc84f95ee1a8b111e1d2364

commit 8b12538158aa93802bc84f95ee1a8b111e1d2364
Author: Hayato Ito <hayato@chromium.org>
Date: Wed Aug 01 06:06:00 2018

Documentation: Update renderer/core/dom/README.md

Add FlatTreeTraversal section.

TBR=rakina

Bug: 742715
Change-Id: I42cee6b8940f07cd65adb1932de59c844fcdace1
Reviewed-on: https://chromium-review.googlesource.com/1158084
Commit-Queue: Hayato Ito <hayato@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579711}
[modify] https://crrev.com/8b12538158aa93802bc84f95ee1a8b111e1d2364/third_party/blink/renderer/core/dom/README.md

Project Member

Comment 13 by bugdroid1@chromium.org, Aug 14

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b0198b937f700cc219121ae1e0afc2eb187d0810

commit b0198b937f700cc219121ae1e0afc2eb187d0810
Author: Hayato Ito <hayato@chromium.org>
Date: Tue Aug 14 09:08:41 2018

Documentation: Update renderer/core/dom/README.md

Add Event path and Event Retargeting section.

TBR=rakina

Bug: 742715
Change-Id: I969f020cc0a762a6cc8695c4e5dc7cd03fcb32e0
Reviewed-on: https://chromium-review.googlesource.com/1174196
Commit-Queue: Hayato Ito <hayato@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#582870}
[modify] https://crrev.com/b0198b937f700cc219121ae1e0afc2eb187d0810/third_party/blink/renderer/core/dom/README.md

Project Member

Comment 14 by bugdroid1@chromium.org, Aug 14

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/efb957f2ae482c5c26d21a4303d1aec293c6d4f4

commit efb957f2ae482c5c26d21a4303d1aec293c6d4f4
Author: Hayato Ito <hayato@chromium.org>
Date: Tue Aug 14 09:51:55 2018

Documentation: Update renderer/core/dom/README.md

Add links to some useful pictures to understand how DOM works.

This CL doesn't add pictures in the repository.  At this point, pictures are
served from a web site which hayato@chromium.org ownes.

TBR=rakina

Bug: 742715
Change-Id: Ic48854b4cc37fe4aca55bd2142ff4cbc26af823f
Reviewed-on: https://chromium-review.googlesource.com/1174293
Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#582876}
[modify] https://crrev.com/efb957f2ae482c5c26d21a4303d1aec293c6d4f4/third_party/blink/renderer/core/dom/README.md

Sign in to add a comment