Closed Bug 1115327 Opened 7 years ago Closed 7 years ago

Transition more fields of FrameMetrics to use getters/setters

Categories

(Core :: Panning and Zooming, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: kats, Assigned: lvf, Mentored)

Details

(Whiteboard: [lang=c++] [good first bug])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1113774 +++

We are transitioning all fields of FrameMetrics to use getters/setter.

Anyone: feel free to pick one of the public fields in FrameMetrics (mScrollableRect is a good one to take), transition it, and post a patch here, flagging me for review. Thanks! See bug 1113774 for an example.
I'd like to be assigned to this bug, could someone assign me please. (Kartikaya Gupta?)
Hi Leonid,

The first step is to get Firefox building and running locally. Once you have that going I'd be happy to assign this bug to you and walk you through what needs to be done. There are instructions on how to get the Firefox code and build it at https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions - let me know if you run into any problems with those.

Thanks for your interest in Mozilla and in working on this bug!
Hi Kartikaya,

Thanks for your feedback. I've got Firefox Nightly built and running.

Also, looking at other similar bugs (eg. Bug #1113774), I prepared a patch for this bug; I'm sure there is something wrong, though.

Please, review and tell what need to be changed. Thanks.
Attachment #8541831 - Flags: review?(bugmail.mozilla)
Assignee: nobody → leonidsbox
Comment on attachment 8541831 [details] [diff] [review]
0001-Bug-1115327-Changed-FrameMetrics.mScrollableRect-to-.patch

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

This looks perfect to me, thanks for putting up the patch so quickly! I will do a push to our tryserver to make sure it builds on all platforms and then land it.
Attachment #8541831 - Flags: review?(bugmail.mozilla) → review+
Thank you, Kartikaya. What should I do next to resolve this bug?
https://hg.mozilla.org/mozilla-central/rev/3f980229dfc1
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
(In reply to Leonid V. Fedorenchik [:lvf] from comment #7)
> Thank you, Kartikaya. What should I do next to resolve this bug?

Nothing! :) The usual process is that about once a day code that lands on the integration branches (such as the inbound branch I landed your patch on) gets merged to our main "mozilla-central" branch. When that happens the bug is marked as fixed, as you can see from comment 8.

Thanks for the patch! If you'd like suggestions on what to work on next please let me know. You can look around on http://www.joshmatthews.net/bugsahoy/ to see if you can find something that matches your interests as well.
Hi Kartikaya, thank you! I want to continue to contribute to Firefox and Mozilla.
Yes, I'd love to get suggestion on what to work on next, please, could you give me advise? Also, I'd like to get to know the architecture of Firefox from developer point of view.
(In reply to Leonid V. Fedorenchik [:lvf] from comment #10)
> Hi Kartikaya, thank you! I want to continue to contribute to Firefox and
> Mozilla.
> Yes, I'd love to get suggestion on what to work on next, please, could you
> give me advise?

I listed a few bugs at https://bugzilla.mozilla.org/show_bug.cgi?id=1113774#c22 which I can mentor you through. If none of those appeal to you let me know and we can look for something else.

> Also, I'd like to get to know the architecture of Firefox
> from developer point of view.

There's lots of documentation scattered about and linked to from https://developer.mozilla.org/en-US/Firefox but honestly it's a little hard to navigate. There closest thing I can point to that it's an overall architecture overview of Firefox internals is the wiki page at https://wiki.mozilla.org/Gecko:Overview. Let me know if that's the sort of stuff you're looking for.
You need to log in before you can comment on or make changes to this bug.