• Ulan Degenbaev's avatar
    Reland^2 "[heap-profiler] Merge DOM node and its JS wrapper node in heap snapshot." · a49090b7
    Ulan Degenbaev authored
    This relands commit d690c638.
    
    This time the new test is marked as timeout in MSAN expectation.
    
    Original change's description:
    > [heap-profiler] Merge DOM node and its JS wrapper node in heap snapshot.
    >
    > Each DOM node has a corresponding JS wrapper node. In heap snapshot
    > they appear as duplicates.
    >
    > Example retaining path with duplicates:
    > - [1] in InternalNode @2041178784
    > - [1] in HTMLDivElement @2041079168  // DOM node
    > - [3] in HTMLDivElement @2231// JS wrapper
    > - retainer in Window / @2105
    >
    > The heap snapshot generator now can merge an embedder node with a node
    > returned by EmbedderGraph::Node::WrapperNode() function.
    >
    > This patch implements the WrapperNode() function for each DOM node.
    >
    > Bug: chromium:811925
    > Change-Id: I9ea6bc7e45f8ab3d54828b9ac61f7230d04d8019
    > Reviewed-on: https://chromium-review.googlesource.com/928503
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#539132}
    
    Change-Id: I0523477d562c68512a2a92902022826df8a32820
    Reviewed-on: https://chromium-review.googlesource.com/955243Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
    Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#541775}
    a49090b7
heap-snapshot-with-multiple-retainers.js 1.91 KB