LGTM3

On Tuesday, March 18, 2025 at 8:47:30 AM UTC-4 Mike Taylor wrote:

>  LGTM2 - I see that Mozilla has proposed a positive position via comment.
> On 3/17/25 2:25 PM, 'Dan Clark' via blink-dev wrote:
>
> LGTM1
>
> On Monday, March 17, 2025 at 11:24:06 AM UTC-7 sligh...@chromium.org 
> wrote:
>
>> I'm recused on this one, but would like to see it move forward quickly. 
>> Great work.
>>
>> On Friday, March 14, 2025 at 1:32:38 PM UTC-7 Chromestatus wrote:
>>
>>> Contact emails issac...@microsoft.com, seth.b...@microsoft.com, 
>>> icle...@google.com 
>>>
>>> Explainer 
>>> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/CrashReporting/AddStackToCrashReports.md
>>>  
>>> https://github.com/WICG/crash-reporting/issues/12 
>>>
>>> Specification https://wicg.github.io/crash-reporting 
>>>
>>> Design docs 
>>>
>>> https://docs.google.com/document/d/19DpvHIiYbmB9wgIP0BdI4vOnfVLcAZFmfIAml7SqRQA/edit?usp=sharing
>>>  
>>>
>>> Summary 
>>>
>>> This feature captures the JS call stack when a web page becomes 
>>> unresponsive due to JavaScript code running an infinite loop or other very 
>>> long computation. This helps developers to identify the cause of the 
>>> unresponsiveness and fix it more easily. The JS call stack is included in 
>>> the crash reporting API when the reason is unresponsive.
>>>
>>>
>>> Blink component Chromium > Internals > Network > ReportingAndNEL 
>>> <https://issues.chromium.org/issues?q=customfield1222907:%22Chromium%20%3E%20Internals%20%3E%20Network%20%3E%20ReportingAndNEL%22>
>>>  
>>>
>>> TAG review https://github.com/w3ctag/design-reviews/issues/981 
>>>
>>> TAG review status Issues addressed 
>>>
>>> Origin Trial Name Call stacks in crash reports 
>>>
>>> Chromium Trial Name DocumentPolicyIncludeJSCallStacksInCrashReports 
>>>
>>> Origin Trial documentation link 
>>> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/CrashReporting/AddStackToCrashReports.md
>>>  
>>>
>>> WebFeature UseCounter name 
>>> kDocumentPolicyIncludeJSCallStacksInCrashReports 
>>>
>>> Risks 
>>>
>>>
>>> Interoperability and Compatibility 
>>>
>>> "The stack trace format itself is not compatible across browsers." 
>>> However, "It is already exposed throughout the web platform (via the 
>>> `error.stack` getter), and there is already a lot of software, both client- 
>>> and server-side, which deals with parsing the different browsers' formats."
>>>
>>>
>>> *Gecko*: No signal (
>>> https://github.com/mozilla/standards-positions/issues/1057) 
>>>
>>> *WebKit*: No signal (
>>> https://github.com/WebKit/standards-positions/issues/380) 
>>>
>>> *Web developers*: Positive 
>>> https://github.com/denoland/deno/issues/26919#issuecomment-2584648202 
>>> https://github.com/WICG/crash-reporting/issues/21 
>>> https://github.com/electron/electron/issues/45356#issuecomment-2635086589 
>>> https://github.com/electron/electron/pull/44204#discussion_r1845994904 
>>> https://github.com/TurboWarp/desktop/issues/1113 
>>> https://github.com/getsentry/sentry-javascript/pull/14044#issuecomment-2468480434
>>>  
>>> https://www.electronjs.org/blog/electron-34-0#unresponsive-renderer-javascript-call-stacks
>>>  
>>>
>>> *Other signals*: Origin trial participants have found the feature 
>>> useful, and their feedback has been positive. 
>>>
>>> Security 
>>>
>>> Stack frames from cross-domain scripts that were not loaded with CORS 
>>> are omitted.
>>>
>>>
>>> WebView application risks 
>>>
>>> Does this intent deprecate or change behavior of existing APIs, such 
>>> that it has potentially high risk for Android WebView-based applications?
>>>
>>> No, the feature does not deprecate or change the behavior of existing 
>>> APIs such that it has potentially high risk for Android WebView-based 
>>> applications.
>>>
>>>
>>> Debuggability 
>>>
>>> Developers can launch DevTools, go to the "Application" Tab, then in the 
>>> "Background services" section click on "Reporting API" where they can 
>>> inspect reports that are queued to be sent. Application --> Reporting API
>>>
>>>
>>> Will this feature be supported on all six Blink platforms (Windows, Mac, 
>>> Linux, ChromeOS, Android, and Android WebView)? Yes 
>>>
>>> Is this feature fully tested by web-platform-tests 
>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md>
>>> ? No 
>>>
>>> This feature is not currently testable on WPT, since triggering it 
>>> requires crashing the browser.
>>>
>>>
>>> DevTrial instructions 
>>> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/CrashReporting/HOWTO.md
>>>  
>>>
>>> Flag name on about://flags 
>>>
>>> Finch feature name DocumentPolicyIncludeJSCallStacksInCrashReports 
>>>
>>> Requires code in //chrome? False 
>>>
>>> Tracking bug 
>>> https://bugs.chromium.org/p/chromium/issues/detail?id=1445539 
>>>
>>> Availability expectation Initially the feature will only be available 
>>> in Chromium browsers. 
>>>
>>> Non-OSS dependencies 
>>>
>>> Does the feature depend on any code or APIs outside the Chromium open 
>>> source repository and its open-source dependencies to function?
>>> No. 
>>>
>>> Estimated milestones 
>>> Shipping on desktop 136 
>>> Origin trial desktop first 127 
>>> Origin trial desktop last 132 
>>> Origin trial extension 1 end milestone 135 
>>> DevTrial on desktop 125 
>>> Shipping on Android 136 
>>> Origin trial Android first 127 
>>> Origin trial Android last 132 
>>> DevTrial on Android 125 
>>> Shipping on WebView 136 
>>> Origin trial WebView first 127 
>>> Origin trial WebView last 132 
>>>
>>> Anticipated spec changes 
>>>
>>> Open questions about a feature may be a source of future web compat or 
>>> interop issues. Please list open issues (e.g. links to known github issues 
>>> in the project for the feature specification) whose resolution may 
>>> introduce web compat/interop risk (e.g., changing to naming or structure of 
>>> the API in a non-backward-compatible way).
>>> None 
>>>
>>> Link to entry on the Chrome Platform Status 
>>> https://chromestatus.com/feature/4731248572628992?gate=5150803040141312 
>>>
>>> Links to previous Intent discussions Intent to Prototype: 
>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/MW2PPF6784DDB763E2DA7BFC75AE51613ABC27B2%40MW2PPF6784DDB76.namprd00.prod.outlook.com
>>>  
>>> Ready for Trial: 
>>> https://groups.google.com/a/chromium.org/g/blink-dev/c/aVpeVRO1Sy4 
>>> Intent to Experiment: 
>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/f43b4e9f-5af8-4c4b-bcd9-342332dd57edn%40chromium.org
>>>  
>>> Intent to Extend Experiment 1: 
>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/67819b5d.2b0a0220.275672.09ce.GAE%40google.com
>>>  
>>>
>>>
>>> This intent message was generated by Chrome Platform Status 
>>> <https://chromestatus.com>. 
>>>
>> -- 
> You received this message because you are subscribed to the Google Groups 
> "blink-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to blink-dev+unsubscr...@chromium.org.
> To view this discussion visit 
> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/1f369b7c-91f2-4074-a902-1cefdeaa7dfbn%40chromium.org
>  
> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/1f369b7c-91f2-4074-a902-1cefdeaa7dfbn%40chromium.org?utm_medium=email&utm_source=footer>
> .
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to blink-dev+unsubscr...@chromium.org.
To view this discussion visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/b3fe8e88-c8f2-4400-a8bc-bfca9056e7dcn%40chromium.org.

Reply via email to