Skip to content

Conversation

@zhongqiliang
Copy link
Contributor

@zhongqiliang zhongqiliang commented Sep 16, 2025

The previous FindFirstJavaFileAndLine only extracted the first
file and line number from a Java exception, which is often
insufficient for debugging.

This updates the regex to also capture the method name and
iterates to find the top 4 frames of the stack trace.
The resulting string (e.g., method1@file1:10&method2@file2:20)
will provide much richer context for crash analysis.

Bug: 430385989
Change-Id: Ifd005980c49de10e5a257d10e8169026c88449c6

Change-Id: Ifd005980c49de10e5a257d10e8169026c88449c6
@zhongqiliang zhongqiliang requested a review from a team as a code owner September 16, 2025 22:30
@zhongqiliang zhongqiliang changed the title draft android: Capture top 4 Java stack frames on exception Sep 16, 2025
@zhongqiliang zhongqiliang requested a review from a team September 16, 2025 22:36
@zhongqiliang zhongqiliang enabled auto-merge (squash) September 16, 2025 23:25
@zhongqiliang zhongqiliang merged commit e7c6693 into youtube:main Sep 16, 2025
384 of 387 checks passed
@zhongqiliang zhongqiliang added the cp-26.android Cherry Pick to the 26.android branch label Sep 16, 2025
cobalt-github-releaser-bot pushed a commit that referenced this pull request Sep 16, 2025
The previous FindFirstJavaFileAndLine only extracted the first
file and line number from a Java exception, which is often
insufficient for debugging.

This updates the regex to also capture the method name and
iterates to find the top 4 frames of the stack trace.
The resulting string (e.g., method1@file1:10&method2@file2:20)
will provide much richer context for crash analysis.

Bug: 430385989
Change-Id: Ifd005980c49de10e5a257d10e8169026c88449c6
(cherry picked from commit e7c6693)
johnxwork pushed a commit that referenced this pull request Sep 17, 2025
…eption (#7193)

Refer to the original PR: #7191

The previous FindFirstJavaFileAndLine only extracted the first
file and line number from a Java exception, which is often
insufficient for debugging.

This updates the regex to also capture the method name and
iterates to find the top 4 frames of the stack trace.
The resulting string (e.g., method1@file1:10&method2@file2:20)
will provide much richer context for crash analysis.

Bug: 430385989
Change-Id: Ifd005980c49de10e5a257d10e8169026c88449c6

Co-authored-by: Colin Liang <colinliang@google.com>
github-actions bot pushed a commit that referenced this pull request Sep 17, 2025
The previous FindFirstJavaFileAndLine only extracted the first
file and line number from a Java exception, which is often
insufficient for debugging.

This updates the regex to also capture the method name and
iterates to find the top 4 frames of the stack trace.
The resulting string (e.g., method1@file1:10&method2@file2:20)
will provide much richer context for crash analysis.

Bug: 430385989
Change-Id: Ifd005980c49de10e5a257d10e8169026c88449c6

original-hexsha: e7c6693
jammel-yeboah pushed a commit to jammel-yeboah/cobalt that referenced this pull request Oct 2, 2025
The previous FindFirstJavaFileAndLine only extracted the first
file and line number from a Java exception, which is often
insufficient for debugging.

This updates the regex to also capture the method name and
iterates to find the top 4 frames of the stack trace.
The resulting string (e.g., method1@file1:10&method2@file2:20)
will provide much richer context for crash analysis.

Bug: 430385989
Change-Id: Ifd005980c49de10e5a257d10e8169026c88449c6
andrewsavage1 pushed a commit that referenced this pull request Oct 13, 2025
The previous FindFirstJavaFileAndLine only extracted the first
file and line number from a Java exception, which is often
insufficient for debugging.

This updates the regex to also capture the method name and
iterates to find the top 4 frames of the stack trace.
The resulting string (e.g., method1@file1:10&method2@file2:20)
will provide much richer context for crash analysis.

Bug: 430385989
Change-Id: Ifd005980c49de10e5a257d10e8169026c88449c6

original-hexsha: e7c6693
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cp-26.android Cherry Pick to the 26.android branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants