diff options
author | Milutin Kristofic <milutin@google.com> | 2023-05-08 14:43:52 +0200 |
---|---|---|
committer | Paladox none <thomasmulhall410@yahoo.com> | 2023-05-08 13:41:52 +0000 |
commit | 2954269bc051d3c385d989820c8e1b237dfd2165 (patch) | |
tree | 5ead63af7c9739dc23e1745e01580db3f27c78fd | |
parent | aa48a9041bb7dbae5d2c6f112febe357444dd552 (diff) |
Fix StartupChangeDisplayed on tabs !== Files
StartupChangeDisplayed should wait on FileList only when File tabs
is selected. When ?tab=checks we don't need to wait for file list,
it's not loading because it's hidden.
This caused wrong p99 values, because it was waiting for users to
click on file tab.
Release-Notes: skip
Google-Bug-Id: b/280828064
Change-Id: If6dc770bf3fbb68dbe8dedee1b369490282bc70a
(cherry picked from commit bfce9b023e93c27c5e711306ace306bc1dbf8315)
-rw-r--r-- | polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts b/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts index 0bee919418..abd3ff0798 100644 --- a/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts +++ b/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts @@ -2275,8 +2275,10 @@ export class GrChangeView extends LitElement { private async reportChangeDisplayed() { await waitUntil(() => !!this.metadata); await untilRendered(this.metadata!); - await waitUntil(() => !!this.fileList); - await untilRendered(this.fileList!); + if (this.activeTab === Tab.FILES) { + await waitUntil(() => !!this.fileList); + await untilRendered(this.fileList!); + } await waitUntil(() => !!this.messagesList); await untilRendered(this.messagesList!); // We are ending the timer after each change view update, because ending a @@ -2289,8 +2291,10 @@ export class GrChangeView extends LitElement { private async reportFullyLoaded() { await waitUntil(() => !!this.metadata); await untilRendered(this.metadata!); - await waitUntil(() => !!this.fileList); - await untilRendered(this.fileList!); + if (this.activeTab === Tab.FILES) { + await waitUntil(() => !!this.fileList); + await untilRendered(this.fileList!); + } await waitUntil(() => !!this.messagesList); await untilRendered(this.messagesList!); await waitUntil(() => this.mergeable !== undefined); |