| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
[inspector] Make collectGarbage of HeapProfiler precise Instead of forcing GC right away, the function now post a task and performance GC from the task with an empty stack to avoid false positive pointers in conservative stack scanning. Bug: chromium:1098187 Change-Id: I88864845a1e395056c5d5f6e867ad774b87dbb6a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2307217 Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by: Simon Zünd <szuend@chromium.org> Reviewed-by: Peter Marshall <petermarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#69444} | 5 年前 | |
[inspector] Make collectGarbage of HeapProfiler precise Instead of forcing GC right away, the function now post a task and performance GC from the task with an empty stack to avoid false positive pointers in conservative stack scanning. Bug: chromium:1098187 Change-Id: I88864845a1e395056c5d5f6e867ad774b87dbb6a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2307217 Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by: Simon Zünd <szuend@chromium.org> Reviewed-by: Peter Marshall <petermarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#69444} | 5 年前 | |
[heap-profiler] Annotate global handles retained by console and debugger. Bug: chromium:811842 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel Change-Id: I72163abf0b20b123fb541fe0a1b168e036ef044e Reviewed-on: https://chromium-review.googlesource.com/919063 Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by: Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#51360} | 8 年前 | |
Revert "[inspector][test] Fix tests assuming precise GC control" This reverts commit 05e05c59106bcc0c7511489bff2f7438b48026c5. Reason for revert: failing test case for Linux64 TSAN - stress-incremental-marking Original change's description: > [inspector][test] Fix tests assuming precise GC control > > The tests inspector/debugger/set-breakpoint-after-gc and > inspector/debugger/get-possible-breakpoints-after-gc rely on precise > garbage collection, as they test --- among other things --- that some > resources have been collected after a GC. They may fail if conservative > stack scanning is used. This CL fixes the tests by using > Protocol.HeapProfiler.collectGarbage, instead of the native > %CollectGarbage, to invoke the GC in asynchronous execution mode, thus > ensuring that it will be invoked from the event loop without a stack. > > The CL also changes TaskRunner::RunMessageLoop to pump the isolate's > foreground task queue even with --stress-incremental-marking, to ensure > progress. This allows us to reenable some tests that were skipped in > this mode. Furthermore, the CL reviews the skip list for inspector > tests and updates some flag adjustments. > > Bug: 41480448 > Bug: 40789559 > Change-Id: I9e6b9796ea985268136bb1c8c99f0d71c9e19dc9 > Cq-Include-Trybots: luci.v8.try:v8_linux64_css_dbg > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6328305 > Commit-Queue: Nikolaos Papaspyrou <nikolaos@chromium.org> > Reviewed-by: Simon Zünd <szuend@chromium.org> > Cr-Commit-Position: refs/heads/main@{#99211} Bug: 41480448 Bug: 40789559 Cq-Include-Trybots: luci.v8.try:v8_linux64_css_dbg No-Presubmit: true No-Tree-Checks: true No-Try: true Change-Id: Iab002b5de30861ae76e03781faf1e6b81c48a391 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6349036 Reviewed-by: Nikolaos Papaspyrou <nikolaos@chromium.org> Commit-Queue: Matthias Liedtke <mliedtke@chromium.org> Cr-Commit-Position: refs/heads/main@{#99220} | 1 年前 | |
Add snapshot test for JSWeakRef and FinalizationRegistry This is a regression test for the fixes in https://crrev.com/c/v8/v8/+/3299592 . Some of the helper functions were copied from console-retaining-path.js in the same directory. Bug: v8:12112 Change-Id: I3c313ad003ede5e5036f886161e1d164c98f87fe Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3400149 Reviewed-by: Simon Zünd <szuend@chromium.org> Commit-Queue: Seth Brenith <seth.brenith@microsoft.com> Cr-Commit-Position: refs/heads/main@{#78689} | 4 年前 | |
Revert "[inspector][test] Fix tests assuming precise GC control" This reverts commit 05e05c59106bcc0c7511489bff2f7438b48026c5. Reason for revert: failing test case for Linux64 TSAN - stress-incremental-marking Original change's description: > [inspector][test] Fix tests assuming precise GC control > > The tests inspector/debugger/set-breakpoint-after-gc and > inspector/debugger/get-possible-breakpoints-after-gc rely on precise > garbage collection, as they test --- among other things --- that some > resources have been collected after a GC. They may fail if conservative > stack scanning is used. This CL fixes the tests by using > Protocol.HeapProfiler.collectGarbage, instead of the native > %CollectGarbage, to invoke the GC in asynchronous execution mode, thus > ensuring that it will be invoked from the event loop without a stack. > > The CL also changes TaskRunner::RunMessageLoop to pump the isolate's > foreground task queue even with --stress-incremental-marking, to ensure > progress. This allows us to reenable some tests that were skipped in > this mode. Furthermore, the CL reviews the skip list for inspector > tests and updates some flag adjustments. > > Bug: 41480448 > Bug: 40789559 > Change-Id: I9e6b9796ea985268136bb1c8c99f0d71c9e19dc9 > Cq-Include-Trybots: luci.v8.try:v8_linux64_css_dbg > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6328305 > Commit-Queue: Nikolaos Papaspyrou <nikolaos@chromium.org> > Reviewed-by: Simon Zünd <szuend@chromium.org> > Cr-Commit-Position: refs/heads/main@{#99211} Bug: 41480448 Bug: 40789559 Cq-Include-Trybots: luci.v8.try:v8_linux64_css_dbg No-Presubmit: true No-Tree-Checks: true No-Try: true Change-Id: Iab002b5de30861ae76e03781faf1e6b81c48a391 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6349036 Reviewed-by: Nikolaos Papaspyrou <nikolaos@chromium.org> Commit-Queue: Matthias Liedtke <mliedtke@chromium.org> Cr-Commit-Position: refs/heads/main@{#99220} | 1 年前 | |
[cdp] Add stackDepth parameter for HeapProfiler.startSampling The Sampling Heap Profiler already supports a stackDepth option internally, but the CDP method HeapProfiler.startSampling did not expose it. This CL adds support for passing stackDepth through CDP. Bug: 443840066 Change-Id: I18ccad126d75b49785a8fde37564be57ecc9c84c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6931850 Commit-Queue: Roman Dvornov <romandvornov@microsoft.com> Reviewed-by: Simon Zünd <szuend@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/main@{#102429} | 10 个月前 | |
[cdp] Add stackDepth parameter for HeapProfiler.startSampling The Sampling Heap Profiler already supports a stackDepth option internally, but the CDP method HeapProfiler.startSampling did not expose it. This CL adds support for passing stackDepth through CDP. Bug: 443840066 Change-Id: I18ccad126d75b49785a8fde37564be57ecc9c84c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6931850 Commit-Queue: Roman Dvornov <romandvornov@microsoft.com> Reviewed-by: Simon Zünd <szuend@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/main@{#102429} | 10 个月前 | |
[heap profiler] Plumb samples through the protocol. BUG=chromium:889545 Cq-Include-Trybots: luci.chromium.try:linux_chromium_headless_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ic00ffa9968cffaf2e20682e247747b5f7dc0f145 Reviewed-on: https://chromium-review.googlesource.com/c/1285394 Commit-Queue: Alexei Filippov <alph@chromium.org> Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#56874} | 7 年前 | |
Reland "[tiering] Remove outdated interrupt budget flags" Change the test to always optimize the "generateTrash" function. Change-Id: Ia2cf8a8e1e7c8ec0618bf846b2961cf692372e2c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4430993 Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/main@{#87111} | 3 年前 | |
Reland "[tiering] Remove outdated interrupt budget flags" Change the test to always optimize the "generateTrash" function. Change-Id: Ia2cf8a8e1e7c8ec0618bf846b2961cf692372e2c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4430993 Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/main@{#87111} | 3 年前 | |
[heap profiler] Plumb samples through the protocol. BUG=chromium:889545 Cq-Include-Trybots: luci.chromium.try:linux_chromium_headless_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ic00ffa9968cffaf2e20682e247747b5f7dc0f145 Reviewed-on: https://chromium-review.googlesource.com/c/1285394 Commit-Queue: Alexei Filippov <alph@chromium.org> Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#56874} | 7 年前 | |
[inspector] fixed taskHeapSnapshot on pause Blink uses access checks to be sure that objects from one context doesn't access objects in another. Heap profiler uses current context to call this checks, we need to be sure that current context is empty to allow heap profiler collect all objects without crash. BUG=chromium:661223 R=alph@chromium.org,ulan@chromium.org Review-Url: https://codereview.chromium.org/2669393002 Cr-Commit-Position: refs/heads/master@{#42939} | 9 年前 | |
[inspector] Refactor protocol-test.js This refactoring makes it easier to write advanced tests and gives full control over what's happening to the test code. It also forces description for every test. BUG=none Review-Url: https://codereview.chromium.org/2891213002 Cr-Commit-Position: refs/heads/master@{#45412} | 9 年前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 5 年前 | ||
| 5 年前 | ||
| 8 年前 | ||
| 1 年前 | ||
| 4 年前 | ||
| 1 年前 | ||
| 10 个月前 | ||
| 10 个月前 | ||
| 7 年前 | ||
| 3 年前 | ||
| 3 年前 | ||
| 7 年前 | ||
| 9 年前 | ||
| 9 年前 |