mirror of
https://github.com/zebrajr/react.git
synced 2026-01-15 12:15:22 +00:00
[DevTools] Track suspensey CSS on "suspended by" (#34166)
We need to track that Suspensey CSS (Host Resources) can contribute to the loading state. We can pick up the start/end time from the Performance Observer API since we know which resource was loaded. If DOM nodes are not filtered there's a link to the `<link>` instance. The `"awaited by"` stack is the callsite of the JSX creating the `<link>`. <img width="591" height="447" alt="Screenshot 2025-08-11 at 1 35 21 AM" src="https://github.com/user-attachments/assets/63af0ca9-de8d-4c74-a797-af0a009b5d73" /> Inspecting the link itself: <img width="592" height="344" alt="Screenshot 2025-08-11 at 1 31 43 AM" src="https://github.com/user-attachments/assets/89603dbc-6721-4bbf-8b58-6010719b29e3" /> In this approach I only include it if the page currently matches the media query. It might contribute in some other scenario but we're not showing every possible state but every possible scenario that might suspend if timing changes in the current state.
This commit is contained in:
committed by
GitHub
parent
2c9a42dfd7
commit
3c67bbe5f9
@@ -3219,6 +3219,94 @@ export function attach(
|
||||
}
|
||||
}
|
||||
|
||||
const hostAsyncInfoCache: WeakMap<{...}, ReactAsyncInfo> = new WeakMap();
|
||||
|
||||
function trackDebugInfoFromHostResource(
|
||||
devtoolsInstance: DevToolsInstance,
|
||||
fiber: Fiber,
|
||||
): void {
|
||||
const resource: ?{
|
||||
type: 'stylesheet' | 'style' | 'script' | 'void',
|
||||
instance?: null | HostInstance,
|
||||
...
|
||||
} = fiber.memoizedState;
|
||||
if (resource == null) {
|
||||
return;
|
||||
}
|
||||
|
||||
// Use a cached entry based on the resource. This ensures that if we use the same
|
||||
// resource in multiple places, it gets deduped and inner boundaries don't consider it
|
||||
// as contributing to those boundaries.
|
||||
const existingEntry = hostAsyncInfoCache.get(resource);
|
||||
if (existingEntry !== undefined) {
|
||||
insertSuspendedBy(existingEntry);
|
||||
return;
|
||||
}
|
||||
|
||||
const props: {
|
||||
href?: string,
|
||||
media?: string,
|
||||
...
|
||||
} = fiber.memoizedProps;
|
||||
|
||||
// Stylesheet resources may suspend. We need to track that.
|
||||
const mayResourceSuspendCommit =
|
||||
resource.type === 'stylesheet' &&
|
||||
// If it doesn't match the currently debugged media, then it doesn't count.
|
||||
(typeof props.media !== 'string' ||
|
||||
typeof matchMedia !== 'function' ||
|
||||
matchMedia(props.media));
|
||||
if (!mayResourceSuspendCommit) {
|
||||
return;
|
||||
}
|
||||
|
||||
const instance = resource.instance;
|
||||
if (instance == null) {
|
||||
return;
|
||||
}
|
||||
|
||||
// Unlike props.href, this href will be fully qualified which we need for comparison below.
|
||||
const href = instance.href;
|
||||
if (typeof href !== 'string') {
|
||||
return;
|
||||
}
|
||||
let start = -1;
|
||||
let end = -1;
|
||||
// $FlowFixMe[method-unbinding]
|
||||
if (typeof performance.getEntriesByType === 'function') {
|
||||
// We may be able to collect the start and end time of this resource from Performance Observer.
|
||||
const resourceEntries = performance.getEntriesByType('resource');
|
||||
for (let i = 0; i < resourceEntries.length; i++) {
|
||||
const resourceEntry = resourceEntries[i];
|
||||
if (resourceEntry.name === href) {
|
||||
start = resourceEntry.startTime;
|
||||
end = start + resourceEntry.duration;
|
||||
}
|
||||
}
|
||||
}
|
||||
const value = instance.sheet;
|
||||
const promise = Promise.resolve(value);
|
||||
(promise: any).status = 'fulfilled';
|
||||
(promise: any).value = value;
|
||||
const ioInfo: ReactIOInfo = {
|
||||
name: 'stylesheet',
|
||||
start,
|
||||
end,
|
||||
value: promise,
|
||||
// $FlowFixMe: This field doesn't usually take a Fiber but we're only using inside this file.
|
||||
owner: fiber, // Allow linking to the <link> if it's not filtered.
|
||||
};
|
||||
const asyncInfo: ReactAsyncInfo = {
|
||||
awaited: ioInfo,
|
||||
// $FlowFixMe: This field doesn't usually take a Fiber but we're only using inside this file.
|
||||
owner: fiber._debugOwner == null ? null : fiber._debugOwner,
|
||||
debugStack: fiber._debugStack == null ? null : fiber._debugStack,
|
||||
debugTask: fiber._debugTask == null ? null : fiber._debugTask,
|
||||
};
|
||||
hostAsyncInfoCache.set(resource, asyncInfo);
|
||||
insertSuspendedBy(asyncInfo);
|
||||
}
|
||||
|
||||
function mountVirtualChildrenRecursively(
|
||||
firstChild: Fiber,
|
||||
lastChild: null | Fiber, // non-inclusive
|
||||
@@ -3446,6 +3534,7 @@ export function attach(
|
||||
throw new Error('Did not expect a host hoistable to be the root');
|
||||
}
|
||||
aquireHostResource(nearestInstance, fiber.memoizedState);
|
||||
trackDebugInfoFromHostResource(nearestInstance, fiber);
|
||||
} else if (
|
||||
fiber.tag === HostComponent ||
|
||||
fiber.tag === HostText ||
|
||||
@@ -4282,6 +4371,7 @@ export function attach(
|
||||
}
|
||||
releaseHostResource(nearestInstance, prevFiber.memoizedState);
|
||||
aquireHostResource(nearestInstance, nextFiber.memoizedState);
|
||||
trackDebugInfoFromHostResource(nearestInstance, nextFiber);
|
||||
} else if (
|
||||
(nextFiber.tag === HostComponent ||
|
||||
nextFiber.tag === HostText ||
|
||||
|
||||
@@ -178,9 +178,12 @@ function SuspendedByRow({
|
||||
}
|
||||
/>
|
||||
)}
|
||||
{(showIOStack || !showAwaitStack) &&
|
||||
ioOwner !== null &&
|
||||
ioOwner.id !== inspectedElement.id ? (
|
||||
{ioOwner !== null &&
|
||||
ioOwner.id !== inspectedElement.id &&
|
||||
(showIOStack ||
|
||||
!showAwaitStack ||
|
||||
asyncOwner === null ||
|
||||
ioOwner.id !== asyncOwner.id) ? (
|
||||
<OwnerView
|
||||
key={ioOwner.id}
|
||||
displayName={ioOwner.displayName || 'Anonymous'}
|
||||
|
||||
@@ -24,6 +24,8 @@ export function getIODescription(value: any): string {
|
||||
return String(value.message);
|
||||
} else if (typeof value.url === 'string') {
|
||||
return value.url;
|
||||
} else if (typeof value.href === 'string') {
|
||||
return value.href;
|
||||
} else if (typeof value.command === 'string') {
|
||||
return value.command;
|
||||
} else if (
|
||||
|
||||
Reference in New Issue
Block a user