Skip to content

Commit 07ee22b

Browse files
Lightning00BladeDevtools-frontend LUCI CQ
authored andcommitted
Add heap snapshot model to MCP bundle
Bug: 454624251 Change-Id: I88655adc76095e2387fd5ac07f3aa5cadf90e070 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/7726281 Commit-Queue: Nikolay Vitkov <nvitkov@chromium.org> Reviewed-by: Alex Rudenko <alexrudenko@chromium.org>
1 parent 3e5924f commit 07ee22b

File tree

5 files changed

+6
-6
lines changed

5 files changed

+6
-6
lines changed

front_end/BUILD.gn

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,6 @@ group("unittests") {
139139
"models/extensions:unittests",
140140
"models/geometry:unittests",
141141
"models/har:unittests",
142-
"models/heap_snapshot:unittests",
143142
"models/issues_manager:unittests",
144143
"models/javascript_metadata:unittests",
145144
"models/live-metrics:unittests",
@@ -288,6 +287,7 @@ group("foundation_unittests") {
288287
"models/ai_assistance:foundation_unittests",
289288
"models/crux-manager:foundation_unittests",
290289
"models/formatter:unittests",
290+
"models/heap_snapshot:unittests",
291291
"models/issues_manager:foundation_unittests",
292292
"models/stack_trace:unittests",
293293
]

front_end/models/heap_snapshot/BUILD.gn

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,14 +28,15 @@ devtools_entrypoint("bundle") {
2828

2929
visibility = [
3030
":*",
31+
"../../../mcp/*",
3132
"../../entrypoints/heap_snapshot_worker/*",
3233
"../../panels/profiler/*",
3334
]
3435

3536
visibility += devtools_models_visibility
3637
}
3738

38-
ts_library("unittests") {
39+
devtools_foundation_module("unittests") {
3940
testonly = true
4041

4142
sources = [ "heap_snapshot.test.ts" ]

front_end/models/heap_snapshot/heap_snapshot.test.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,7 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
import {describeWithEnvironment} from '../../testing/EnvironmentHelpers.js';
6-
7-
describeWithEnvironment('HeapSnapshotModel', () => {
5+
describe('HeapSnapshotModel', () => {
86
it('module can be imported', async () => {
97
await import('./heap_snapshot.js');
108
});

mcp/BUILD.gn

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ devtools_module("mcp") {
2424
"../front_end/models/bindings:bundle",
2525
"../front_end/models/crux-manager:bundle",
2626
"../front_end/models/formatter:bundle",
27+
"../front_end/models/heap_snapshot:bundle",
2728
"../front_end/models/issues_manager:bundle",
2829
"../front_end/models/stack_trace:bundle",
2930
"../front_end/models/trace:bundle",

mcp/mcp.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ export {AgentFocus} from '../front_end/models/ai_assistance/performance/AIContex
3838
export {DebuggerWorkspaceBinding} from '../front_end/models/bindings/DebuggerWorkspaceBinding.js';
3939
export {CrUXManager} from '../front_end/models/crux-manager/CrUXManager.js';
4040
export * as Formatter from '../front_end/models/formatter/formatter.js';
41+
export * as HeapSnapshotModel from '../front_end/models/heap_snapshot/heap_snapshot.js';
4142
export {Issue} from '../front_end/models/issues_manager/Issue.js';
4243
export {
4344
AggregatedIssue,
@@ -53,7 +54,6 @@ export {
5354
export * as MarkdownIssueDescription from '../front_end/models/issues_manager/MarkdownIssueDescription.js';
5455
export * as StackTrace from '../front_end/models/stack_trace/stack_trace.js';
5556
export * as TraceEngine from '../front_end/models/trace/trace.js';
56-
export {IgnoreListManager} from '../front_end/models/workspace/IgnoreListManager.js';
5757
export * as Marked from '../front_end/third_party/marked/marked.js';
5858

5959
installInspectorFrontendHost(new McpHostBindings());

0 commit comments

Comments
 (0)