Closed Bug 839831 Opened 12 years ago Closed 12 years ago

jsjni_ functions should be usable without MOZILLA_INTERNAL_API (e.g. in WebRTC code)

Categories

(Firefox for Android Graveyard :: General, enhancement)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
ARM
Android
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Firefox 23

People

(Reporter: dmosedale, Assigned: gcp)

References

Details

(Whiteboard: [android-trunk-needed][android-gum+][qa-])

Attachments

(1 file, 1 obsolete file)

Patch 1. move jsjni_ functions into their own files, rebased
8.11 KB, patch
blassey
: review+
Details | Diff | Splinter Review
Right now, using things like jsjni_FindClass requires including AndroidBridge.h, which blows up because of internal string glop if it's include where MOZILLA_INTERNAL_API isn't defined. gcp put together a patch that puts them in their own files and exports them independently.
Landed on alder: https://hg.mozilla.org/projects/alder/rev/fd3a3e5d72e3 Next steps: rebase to trunk and request review.
Whiteboard: [android-trunk-needed]
Attachment #712211 - Attachment description: move jsjni_ functions into their own files, v1 → move jsjni_ functions into their own files, v1 (landed alder)
Blocks: 839836
Whiteboard: [android-trunk-needed] → [android-trunk-needed][android-gum+]
Rebased to m-c.
Attachment #712211 - Attachment is obsolete: true
Attachment #726206 - Flags: review?(blassey.bugs)
Attachment #726206 - Flags: review?(blassey.bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/6354187a454d
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
Whiteboard: [android-trunk-needed][android-gum+] → [android-trunk-needed][android-gum+][qa-]
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: