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

Native bridge #1014

Merged
merged 3 commits into from
Aug 29, 2017
Merged

Native bridge #1014

merged 3 commits into from
Aug 29, 2017

Conversation

ben-clayton
Copy link
Contributor

No description provided.

Used for getting the native ABI for NativeBridge emulators.
It's been completely deprecated.
May fix tracing Android applications on Chromebooks where the ABI translator cannot deal with libinterceptor.
@ben-clayton ben-clayton requested a review from AWoloszyn August 29, 2017 10:56
@ben-clayton ben-clayton merged commit 695f69b into google:master Aug 29, 2017
@ben-clayton ben-clayton deleted the native-bridge branch August 29, 2017 12:42

// NativeBridgeABI returns the native ABI for the given emulated ABI for the
// device by consulting the ro.dalvik.vm.isa.<emulated_isa>=<native_isa>
// system properties.
Copy link
Member

Choose a reason for hiding this comment

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

I think this should also say that it returns the passed in ABI if there is no emulation.

Copy link
Member

Choose a reason for hiding this comment

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

never mind... for some reason GH was showing me out of date information and this PR was still open.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Still a good point. Will fix shortly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

purvisa-at-google-com pushed a commit that referenced this pull request Sep 29, 2022
Leaving it selected after destruction was causing a UI exception on
macOS.

Bug: b/191637172
Test: Manual
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