Closed
Bug 1116641
Opened 6 years ago
Closed 6 years ago
Get rid of JNI macros AndroidBridgeUtilities.h
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 37
People
(Reporter: jchen, Assigned: jchen)
Details
Attachments
(2 files)
1.90 KB,
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
18.62 KB,
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
The JNI macros in widget/android/AndroidBridgeUtilities.h declare a 'jClass' variable in the local scope, which is unexpected and potentially dangerous. We should replace it with something better.
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Attachment #8544171 -
Flags: review?(snorp)
Assignee | ||
Updated•6 years ago
|
Attachment #8544172 -
Flags: review?(snorp)
Attachment #8544171 -
Flags: review?(snorp) → review+
Attachment #8544172 -
Flags: review?(snorp) → review+
Assignee | ||
Comment 3•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/0f955b9395c8
Comment 4•6 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0f955b9395c8
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 37
Updated•4 months ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•