Commit a2b76fa7 authored by Santiago Aboy Solanes's avatar Santiago Aboy Solanes Committed by V8 LUCI CQ

[turbolizer] Solve bug where the node 0 was not being selected

In the schedule phase (and only in this phase!) the node 0 was not
being selected due to `!0` being true. We meant to be checking
against undefined rather than this.

Bug: v8:7327
Notry: true
Change-Id: Ie675979219868725b0e345065cec7a2a7091fda8
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3008215Reviewed-by: 's avatarNico Hartmann <nicohartmann@chromium.org>
Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#75585}
parent 2889a3c2
...@@ -21,7 +21,7 @@ export class MySelection { ...@@ -21,7 +21,7 @@ export class MySelection {
select(s: Iterable<any>, isSelected?: boolean) { select(s: Iterable<any>, isSelected?: boolean) {
for (const i of s) { for (const i of s) {
if (!i) continue; if (i == undefined) continue;
if (isSelected == undefined) { if (isSelected == undefined) {
isSelected = !this.selection.has(this.stringKey(i)); isSelected = !this.selection.has(this.stringKey(i));
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment