Closed
Bug 733184
Opened 12 years ago
Closed 12 years ago
Maple: Remove some Java logging
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: BenWa, Assigned: BenWa)
Details
Attachments
(2 files)
|
12.78 KB,
patch
|
pcwalton
:
review+
|
Details | Diff | Splinter Review |
|
6.05 KB,
patch
|
pcwalton
:
review+
|
Details | Diff | Splinter Review |
This logging is hurting performance and is getting in the way of profiling. If we care about keeping some of this should move it to a system where we can toggle it. Asking pwalton for review since he added/used most of these.
Attachment #603076 -
Flags: review?(pwalton)
|
||
Comment 1•12 years ago
|
||
Comment on attachment 603076 [details] [diff] [review] patch Review of attachment 603076 [details] [diff] [review]: ----------------------------------------------------------------- Thanks, all of the logging was really annoying.
Attachment #603076 -
Flags: review?(pwalton) → review+
|
Assignee | |
Comment 2•12 years ago
|
||
I forgot some logging in widget
|
|
||
Comment 3•12 years ago
|
||
Comment on attachment 603106 [details] [diff] [review] Remove widget logging Review of attachment 603106 [details] [diff] [review]: ----------------------------------------------------------------- r=me
Attachment #603106 -
Flags: review?(pwalton) → review+
|
||
Updated•12 years ago
|
OS: Mac OS X → Android
Hardware: x86 → ARM
|
|
Assignee | |
Comment 4•12 years ago
|
||
Landed. FYI attachment 603106 [details] [diff] [review] was modified before landing and pcwalton give me a IRC r+ for the amended changes.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
||
Updated•3 years 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
•