Summary: | Web Inspector: [Cocoa] WKInspectorResourceURLSchemeHandler can exceed the soft thread limit when opening multiple inspectors simultaneously. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Patrick Angle <pangle> | ||||||
Component: | Web Inspector | Assignee: | Patrick Angle <pangle> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bburg, ews-watchlist, hi, inspector-bugzilla-changes, joepeck, pangle, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Patrick Angle
2021-03-30 14:41:57 PDT
Created attachment 424701 [details]
Patch v1.0
Comment on attachment 424701 [details] Patch v1.0 View in context: https://bugs.webkit.org/attachment.cgi?id=424701&action=review r=me > Source/WebKit/UIProcess/Inspector/mac/WKInspectorResourceURLSchemeHandler.mm:62 > + // The default value (NSOperationQueueDefaultMaxConcurrentOperationCount) results in a large number of threads I think this second comment is nice but redundant with the changelog, and could be removed. Created attachment 424862 [details]
Patch v1.1 - Shorten redundant part of comment
Committed r275341: <https://commits.webkit.org/r275341> All reviewed patches have been landed. Closing bug and clearing flags on attachment 424862 [details]. |