Skip to content

Commit cfa5963

Browse files
szuendDevtools-frontend LUCI CQ
authored andcommitted
[workspace] Don't create IgnoreListManager lazily
It's not necessary so only create a new instance when 'forceNew' is provided explicitly. [email protected] Bug: 458180550 Change-Id: Ibce92c620f8c082b19724e0b52d33afceab2f289 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/7147062 Reviewed-by: Philip Pfaffe <[email protected]> Commit-Queue: Simon Zünd <[email protected]> Auto-Submit: Simon Zünd <[email protected]>
1 parent d9fde31 commit cfa5963

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

front_end/models/workspace/IgnoreListManager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ export class IgnoreListManager extends Common.ObjectWrapper.ObjectWrapper<EventT
8888
forceNew: null,
8989
}): IgnoreListManager {
9090
const {forceNew} = opts;
91-
if (!Root.DevToolsContext.globalInstance().has(IgnoreListManager) || forceNew) {
91+
if (forceNew) {
9292
Root.DevToolsContext.globalInstance().set(
9393
IgnoreListManager,
9494
new IgnoreListManager(

0 commit comments

Comments
 (0)