Skip to content

Conversation

@Swatinem
Copy link
Member

This moves view hierarchy handling into its own file, and changes the java language processor in such a way that it only loads the view hierarchy once.

Previously, it would load them twice, once to collect all the window class names, and then a second time to actually deobfuscate it.

This moves view hierarchy handling into its own file, and changes the java language processor in such a way that it only loads the view hierarchy once.

Previously, it would load them twice, once to collect all the window class names, and then a second time to actually deobfuscate it.
@Swatinem Swatinem requested a review from buenaflor August 29, 2025 09:43
@Swatinem Swatinem self-assigned this Aug 29, 2025
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 29, 2025
cursor[bot]

This comment was marked as outdated.

@codecov
Copy link

codecov bot commented Aug 29, 2025

Codecov Report

❌ Patch coverage is 75.00000% with 12 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/lang/java/view_hierarchies.py 72.72% 12 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #98512      +/-   ##
==========================================
- Coverage   81.18%   81.16%   -0.02%     
==========================================
  Files        8546     8547       +1     
  Lines      376600   376806     +206     
  Branches    23961    23961              
==========================================
+ Hits       305738   305851     +113     
- Misses      70496    70589      +93     
  Partials      366      366              

Copy link
Contributor

@buenaflor buenaflor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not too familiar with the view hierarchy symbolication for Java so I don't think I can comment too much on this PR

@Swatinem Swatinem requested a review from loewenheim August 29, 2025 10:36
Copy link
Contributor

@loewenheim loewenheim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@Swatinem Swatinem merged commit 4709b2d into master Sep 8, 2025
64 checks passed
@Swatinem Swatinem deleted the swatinem/vh-once branch September 8, 2025 06:23
andrewshie-sentry pushed a commit that referenced this pull request Sep 9, 2025
This moves view hierarchy handling into its own file, and changes the
java language processor in such a way that it only loads the view
hierarchy once.

Previously, it would load them twice, once to collect all the window
class names, and then a second time to actually deobfuscate it.
@github-actions github-actions bot locked and limited conversation to collaborators Sep 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants