Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 967ecbc754218b54fdca06b1db0c38bbd1380fd1
      
https://github.com/WebKit/WebKit/commit/967ecbc754218b54fdca06b1db0c38bbd1380fd1
  Author: Yusuke Suzuki <[email protected]>
  Date:   2024-11-08 (Fri, 08 Nov 2024)

  Changed paths:
    M Source/ThirdParty/d3flamegraphjs/d3-flamegraph.min.js

  Log Message:
  -----------
  Tweak d3-flamegraph JS to avoid error
https://bugs.webkit.org/show_bug.cgi?id=282862
rdar://139544461

Reviewed by Yijia Huang.

d3-flamegraph is doing,

    Math.max.apply(null, super-large-array)

And it does not work well if super-large-array is too large since it can
exhaust stack space and stack overflow will be thrown. We observed this
for flamegraph for JetStream2/WSL test. This patch fixes it by avoiding
the above pattern. Just using reduce function to extract the largest
value.

* Source/ThirdParty/d3flamegraphjs/d3-flamegraph.min.js:

Canonical link: https://commits.webkit.org/286368@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to