...with the intention of using it for `GPUProcess` as well, in a future patch.
Created attachment 433445 [details] Patch
Comment on attachment 433445 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=433445&action=review > Source/WebKit/WebProcess/WebProcess.h:413 > + RetainPtr<NSDictionary> dictionaryForStateDumper() const final; Can we make it more clear in the name what this is about? Every time I have encountered this function before it was wildly unclear what state was being dumped and why.
Comment on attachment 433445 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=433445&action=review >> Source/WebKit/WebProcess/WebProcess.h:413 >> + RetainPtr<NSDictionary> dictionaryForStateDumper() const final; > > Can we make it more clear in the name what this is about? Every time I have encountered this function before it was wildly unclear what state was being dumped and why. Sounds good! I'll rename this to `additionalStateForDiagnosticReport` as we discussed on Slack.
Created attachment 433470 [details] Rename method
Committed r279902 (239651@main): <https://commits.webkit.org/239651@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 433470 [details].
<rdar://problem/80552993>