Skip to content
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

Add support for Firefox Oculus VR browser #752

Merged
merged 1 commit into from
Aug 2, 2024
Merged

Conversation

Schamper
Copy link
Member

@Schamper Schamper commented Jul 18, 2024

Depends on and separated from #419

@Schamper Schamper requested a review from Miauwkeru July 18, 2024 11:45
Copy link

codecov bot commented Jul 18, 2024

Codecov Report

Attention: Patch coverage is 94.87179% with 2 lines in your changes missing coverage. Please review.

Project coverage is 75.26%. Comparing base (286f5cc) to head (e28303c).

Files Patch % Lines
dissect/target/plugins/apps/browser/firefox.py 93.54% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #752      +/-   ##
==========================================
+ Coverage   75.23%   75.26%   +0.02%     
==========================================
  Files         296      296              
  Lines       25556    25573      +17     
==========================================
+ Hits        19228    19248      +20     
+ Misses       6328     6325       -3     
Flag Coverage Δ
unittests 75.26% <94.87%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Base automatically changed from android-backup to main July 31, 2024 11:15
@Schamper Schamper force-pushed the firefox-vrbrowser branch from ddcb80c to 4092327 Compare August 1, 2024 16:51
@Schamper Schamper force-pushed the firefox-vrbrowser branch from 4092327 to e28303c Compare August 2, 2024 09:13
@Schamper Schamper merged commit e3becf6 into main Aug 2, 2024
18 checks passed
@Schamper Schamper deleted the firefox-vrbrowser branch August 2, 2024 11:46
@@ -66,6 +70,10 @@ class FirefoxPlugin(BrowserPlugin):
"Library/Application Support/Firefox",
]

SYSTEM_DIRS = [
"/data/data/org.mozilla.vrbrowser/files/mozilla",
Copy link
Contributor

Choose a reason for hiding this comment

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

just a random question, is there a difference between the normal android firefox browser and the vrbrowser except for the uri in the path?

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't have an Android device to verify this with at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants