- 
                Notifications
    You must be signed in to change notification settings 
- Fork 18
[wpe] Update to WPE WebKit 2.48.2 #194
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| This is marked as draft because fdsan reports a file descriptor being wrongly closed and I have not been able to track this down yet. We do funny things with file descriptors in WebKit and also in WPEBackend-android, so this might be a false positive. Here's the top of the traceback.  | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it is just a ParcelFile handle that is not released when shutting down the app this should not be a big issue. I think you can merge it as it is. Maybe open an issue with this bug to keep track of it. But normally it should be cleaned by the Android system when the app is shut down.
| 
 Unfortunately this happens during startup and we never get to see anything rendered before the crash. There are a few of the changes from Igalia/wpe-android-cerbero#57 which I should be able to do separately so I am going to try and split the big PR with the 2.48 update, maybe that helps isolate the change that brings in the issue. As a separate, but complimentary thought, we may want to use the fdsan API to make it aware of what our code does with the file descriptors, in case it might help. | 
| Hi @aperezdc, I'm testing your PR on my Android device. I'm also encountering the same FD error after launching MiniBrowser. I've attached the trace for reference (missing symbols from  I am running on: Android 14, SDK level 34, Samsung Galaxy S24+. | 
Web Inspector resources now being shipped as a .gresource file instead of a shared library, which needed changes in the bootstrap script and setting the relevant environment variable. For now the .gresource file is installed along the injected bundle, as the shared library was previously, but this may change in the future.
10abc37    to
    88b0908      
    Compare
  
    | We have  | 
Web Inspector resources now being shipped as a
.gresourcefile instead of a shared library, which needed changes in the bootstrap script and setting the relevant environment variable. For now the.gresourcefile is installed along the injected bundle, as the shared library was previously, but this may change in the future.Note this is expected to fail CI until bootstrap packages for the corresponding release are available.Bootstrap packages for 2.48.2 are now available 😄