Alex N. Jose
Alex N. Jose
Closing this PR; will raise a new one that addresses the feedback.
The issue doesn't seem reproducible. Lighthouse runs fine on the URL. Closing this out, please feel free to reopen if the issue still persists.
@connorjclark #14135 should add the feature required, but we'll likely need to migrate the tests to use the new method to resolve this issue.
I can be a reviewer if help is needed!
I'd be happy to be a reviewer if help is needed!
This seems to be related to #6708. I can repro on PSI with `vividmotion.co`. Looking further into it.
Summary from the discussion today: This would make sense to be a part of the prioritized audit/opportunities redesign/refactor.
Bumping this (old) issue — is there a supported/recommended way to extract HTML of a node (`.innerHTML` and `.outerHTML`) with HTMLRewriter/lol-html? Or would it be advisable to switch to full...
I just ran into this issue while implementing entity-classification into third-party-summary audit. The issue I'm seeing with timespan is that it allows cross-origin navigations. Eg., I could do a timespan...
Alternatively, we could implicitly create multiple timespans each time origin changes — so each timespan would still make sense and the overall report still is usable. This should also work...