This function is not used in current codebase, and should be removed.
Hi, I'd like to take this as my first bug. I removed the function and successfully built the code on my laptop, what's next step should I do? Thank you.
Hi, Please refer to the introduction here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Introduction Since you had built correctly, I think you may move forward to "Step 4 - Get your code reviewed"
Hi, I've removed the function never being used, would you review it? Thank you so much.
Attachment #8892802 - Flags: review?(jmuizelaar)
Thank you for working on this bug! Enjoy your first patch~
Status: NEW → ASSIGNED
Attachment #8892802 - Flags: review?(jmuizelaar) → review+
r+ granted, next step is make sure all test cases pass by your change("Step 6 - Actually get the code into the tree") After that, add the checkin-needed keyword to the bug (or if Bugzilla doesn't allow you to add the keyword, ask someone else to add it).
Thanks for all of your help :)
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/092017bb63c9 remove unused function gfxContext::HasComplexClip; r=jrmuizel
(In reply to ♥ Ami ♥ from comment #6) > Thanks for all of your help :) Thanks for working on this bug. Let us know if you'd like to take more. :)
You need to log in before you can comment on or make changes to this bug.