misc(viewer): accept lhr from fragment

[connorjclark]

(#12519 should land first)

This will make "open in viewer" possible in devtools.

The fragment url is cleared when the page loads. Treemap keeps it around. Seems good for Viewer b/c there is a "save as gist" button to get a short link, but treemap has no such thing.

[connorjclark]

https://github.com/GoogleChrome/lighthouse/blob/0366f6f/lighthouse-viewer/app/src/lighthouse-report-viewer.js#L216

[connorjclark]

We want to update the "View Report" link in web.dev to just link to our Viewer app, so we don't need to maintain the one web.dev is using. Should we do postMessage there, or use a fragment like this PR is proposing?

[patrickhulce]

We want to update the "View Report" link in web.dev to just link to our Viewer app, so we don't need to maintain the one web.dev is using. Should we do postMessage there, or use a fragment like this PR is proposing?

For places that can use the postMessage, that feels much safer for LHRs. I've definitely seen a 2MB LHR before (though I guess with gzip we'll probably always be OK? LHR structure is pretty repetitive). Might be worth preventing the open in viewer message in DevTools when we know it'll be too big.

[google-cla]

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

[google-cla]

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

[google-cla]

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

[paulirish]

after team discussion we decided to adopt this, but if it starts causing problems we’ll remove it.

this lgtm

[paulirish]

@googlebot I consent.