TTFB Audit followup

[paulirish]

Forking from #2126

@wardpeet added:

I had a few changes ready to go, was fixing the details export of audit and cleaning up the criticalRequestChains but I can open an extra PR just for that 😄

i am also seeing this in the new report

and this in the old one:

[ebidel]

Is "details export" adding a details rendering to the audit? FYI, I'm working on a criticalRequestChains details renderer.

[paulirish]

i think so, but cc @wardpeet to be sure.

it'll just be a basic table renderer.

i'm working on a criticalRequestChains details renderer.

awesome.

[paulirish]

Additional for @wardpeet: rename time-to-firstbyte to time-to-first-byte

[wardpeet]

I was looking into the gatherer as @patrickhulce mentioned:

I was thinking just expose as a flattened property of the existing computed artifact so we don't bother with the computation twice although if you're feeling fancy we could refactor the existing one to call requestCriticalRequests and then build the tree :)

[wardpeet]

Just need a rebase and #2231 is ready.

Some records don't have _timing so I ignore them

[brendankenny]

closed by #2231