Remove Log.w calls from GLController.java

RESOLVED FIXED in Firefox 31

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: kats, Assigned: alexandru.deaconu)

Tracking

unspecified
Firefox 31
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mentor=kats][lang=java][good first bug])

Attachments

(1 attachment)

A while back while debugging GL issues in GLController.java we added a bunch of Log.w statements so we could get more info from crash reports. The issues seem to be resolved now so we can take out these log lines. The ones relating to egl preloading and eglInitialize can stay in, but the ones that are just tracing code (all the ones in serverSurfaceDestroyed, serverSurfaceChanged, updateCompositor, compositorCreated, and resumeCompositor) can be removed.
Assignee

Comment 1

5 years ago
Hi, I would like to work on this issue. Does it just involve the deletion of the Log lines?
Yup, that's it! Have you successfully compiled and run the Fennec build? If so I can assign this bug to you and you can start working on it. If you haven't yet gotten it compiled and running, there are instructions to do so at https://wiki.mozilla.org/Mobile/Fennec/Android - please give that a go and let me know if you run into any problems!

Feel free to comment here, or join us in the #mobile channel on mozilla's IRC server (see https://wiki.mozilla.org/IRC) for help.
Assignee

Comment 3

5 years ago
Yes, I have compiled Fennec and using Eclipse as an IDE. I have also generated the eclipse projects and imported them into Eclipse.
Great! I'm assigning the bug to you.

Basically all you need to do is remove the log lines that I pointed out in comment 0 of this bug, make sure the code still compiles and runs fine, and then create a patch with your changes. Attach the patch to this bug as an attachment and flag me for review on it.

There are instructions on how to go about creating a patch at https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch#Creating_a_patch

As always, let me know if you have any questions or get stuck!
Assignee: nobody → alexandru.deaconu
Assignee

Comment 5

5 years ago
Removed 'Log.w' lines from the 'serverSurfaceDestroyed', 'serverSurfaceChanged', 'updateCompositor', 'compositorCreated' and 'resumeCompositor' methods
Assignee

Updated

5 years ago
Attachment #8411838 - Flags: review?(bugmail.mozilla)
Comment on attachment 8411838 [details] [diff] [review]
Removed unnecessary log lines from 'GLController'

Review of attachment 8411838 [details] [diff] [review]:
-----------------------------------------------------------------

Perfect, thanks! That was really quick, good work :) I'll mark this bug for checkin and it will be closed when the patch gets merged to our mozilla-central repository. In the meantime please feel free to grab another bug to work on - let me know if you'd like me to point you to ones that might be a bit more interesting.
Attachment #8411838 - Flags: review?(bugmail.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/b170440acfed
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 31
You need to log in before you can comment on or make changes to this bug.