Bug 256683

Summary: Web Inspector: Group source maps error under a unique details/summary
Product: WebKit Reporter: Karl Dubost <karlcow>
Component: Web InspectorAssignee: Jon Davis <jond>
Status: RESOLVED FIXED    
Severity: Normal CC: inspector-bugzilla-changes, jond, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
See Also: https://bugs.webkit.org/show_bug.cgi?id=264490
https://bugs.webkit.org/show_bug.cgi?id=264489
Attachments:
Description Flags
console log with source maps errors
none
Source Map load errors before the patch
none
Grouped Source Map loading errors enabled, in collapsed state
none
Grouped Source Map loading errors enabled, expanded
none
Updated Grouped Soruce Map loading errors based on ConsoleGroup, error icon added none

Description Karl Dubost 2023-05-11 18:51:57 PDT
Created attachment 466329 [details]
console log with source maps errors

Many sites include source maps, but with an error, either the URL doesn't exist seen from outside, or the auth is not working, etc.
This results in a long list of errors or warnings in the console. 

Suggestions:
Groups them together under a summary/detail elements. 

<details>
    <summary>Source Maps Error</summary>
    Here the list of source maps errors. 
</details>


see the screenshot.
Comment 1 Radar WebKit Bug Importer 2023-05-11 18:52:12 PDT
<rdar://problem/109239646>
Comment 2 Jon Davis 2023-10-05 16:44:24 PDT
Created attachment 468081 [details]
Source Map load errors before the patch
Comment 3 Jon Davis 2023-10-05 16:45:21 PDT
Created attachment 468082 [details]
Grouped Source Map loading errors enabled, in collapsed state
Comment 4 Jon Davis 2023-10-05 16:46:10 PDT
Created attachment 468083 [details]
Grouped Source Map loading errors enabled, expanded
Comment 5 Jon Davis 2023-10-05 17:11:35 PDT
Pull request: https://github.com/WebKit/WebKit/pull/18733
Comment 6 Jon Davis 2023-10-06 16:32:55 PDT
Created attachment 468102 [details]
Updated Grouped Soruce Map loading errors based on ConsoleGroup, error icon added
Comment 7 EWS 2023-11-16 10:21:49 PST
Committed 270834@main (869419c2de21): <https://commits.webkit.org/270834@main>

Reviewed commits have been landed. Closing PR #18733 and removing active labels.