Closed
Bug 637559
Opened 13 years ago
Closed 13 years ago
Add logging to Android Bridge
Categories
(Core Graveyard :: Widget: Android, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dougt, Assigned: dougt)
Details
Attachments
(2 files)
17.78 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
781 bytes,
patch
|
dougt
:
review+
dougt
:
approval2.0+
|
Details | Diff | Splinter Review |
Add logging to Android Bridge. Basically the interface from native code into java isn't noisy enough. There have been perf hot spots (getting mime information) that went unnoticed, and crashes (dalvik leaks) caused by this bridge. We should add basic logging so that we can see what is being called.
Assignee | ||
Comment 1•13 years ago
|
||
Assignee: nobody → doug.turner
Attachment #515831 -
Flags: review?(blassey.bugs)
Updated•13 years ago
|
Attachment #515831 -
Flags: review?(blassey.bugs) → review+
Comment 2•13 years ago
|
||
pushed http://hg.mozilla.org/mozilla-central/rev/e244bdf7eff7
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 3•13 years ago
|
||
this one is unnecessarily noisy
Attachment #516050 -
Flags: review?(doug.turner)
Assignee | ||
Comment 4•13 years ago
|
||
Comment on attachment 516050 [details] [diff] [review] move HideProgressDialogOnce logging thanks!
Attachment #516050 -
Flags: review?(doug.turner)
Attachment #516050 -
Flags: review+
Attachment #516050 -
Flags: approval2.0+
Comment 5•13 years ago
|
||
pushed http://hg.mozilla.org/mozilla-central/rev/72fa9452e56a
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•