commit 96ef2c7cf3211d5e00a7a3daac46f3379877deac
author Ben Rohlfs <brohlfs@google.com> Mon Oct 30 18:02:35 2023 +0100
committer Ben Rohlfs <brohlfs@google.com> Thu Nov 02 17:23:57 2023 +0100
tree 2183be3952e3edb93315538634eb6d2db4d41cb7
parent 84e4594769c970b187a28240cc3fdf2e81d03e82

Move blame information into diff model

And only render blame cells, if the blame column is explicitly enabled.
Stop using a `display:none` css rule for that.

This also means removing one usage of the ugly `querySelector` based
`findRow()` lookup.

Release-Notes: skip
Change-Id: I26a860a705238e1c979d455535b59049ee628952

polygerrit-ui/app/embed/diff/gr-diff/gr-diff-utils.ts

diff --git a/polygerrit-ui/app/embed/diff/gr-diff/gr-diff-utils.ts b/polygerrit-ui/app/embed/diff/gr-diff/gr-diff-utils.ts
index 219f16e..6dc292e 100644
--- a/polygerrit-ui/app/embed/diff/gr-diff/gr-diff-utils.ts
+++ b/polygerrit-ui/app/embed/diff/gr-diff/gr-diff-utils.ts
@@ -3,7 +3,7 @@
  * Copyright 2020 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
-import {CommentRange} from '../../../types/common';
+import {BlameInfo, CommentRange} from '../../../types/common';
 import {Side, SpecialFilePath} from '../../../constants/constants';
 import {
   DiffContextExpandedExternalDetail,
@@ -462,3 +462,10 @@
   groups: GrDiffGroup[];
   numLines: number;
 }
+
+export function findBlame(blameInfos: BlameInfo[], line?: LineNumber) {
+  if (typeof line !== 'number') return undefined;
+  return blameInfos.find(info =>
+    info.ranges.find(range => range.start <= line && line <= range.end)
+  );
+}