Bug 233961 - Web Inspector: evaluateScriptForExtension() incorrectly unwraps internal errors
Summary: Web Inspector: evaluateScriptForExtension() incorrectly unwraps internal errors
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: BJ Burg
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-12-07 16:56 PST by BJ Burg
Modified: 2021-12-08 16:07 PST (History)
7 users (show)

See Also:


Attachments
Patch v1.0 (3.53 KB, patch)
2021-12-07 22:22 PST, BJ Burg
no flags Details | Formatted Diff | Diff
Patch v1.1 - for landing (3.53 KB, patch)
2021-12-08 14:55 PST, BJ Burg
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description BJ Burg 2021-12-07 16:56:15 PST
Splitting this change from 233935.
Comment 1 BJ Burg 2021-12-07 16:58:32 PST
<rdar://86123763>
Comment 2 BJ Burg 2021-12-07 22:22:14 PST
Created attachment 446299 [details]
Patch v1.0
Comment 3 Patrick Angle 2021-12-08 13:03:56 PST
Comment on attachment 446299 [details]
Patch v1.0

View in context: https://bugs.webkit.org/attachment.cgi?id=446299&action=review

r=me

> Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.cpp:255
> +                LOG(Inspector, "Internal error encountered while evaluating upon the frontend: at %s:%d:%d: %s", exceptionDetails.sourceURL.utf8().data(), exceptionDetails.lineNumber, exceptionDetails.columnNumber, exceptionDetails.message.utf8().data());

Nit: Odd placement of the first colon IMO... s/`frontend: at`/`frontend at`
Comment 4 BJ Burg 2021-12-08 14:55:38 PST
Created attachment 446429 [details]
Patch v1.1 - for landing
Comment 5 EWS 2021-12-08 16:07:12 PST
Committed r286747 (244992@main): <https://commits.webkit.org/244992@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 446429 [details].