Closed Bug 968735 Opened 7 years ago Closed 7 years ago

Misc tweaks in PanelsPreferenceCategory

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 30

People

(Reporter: lucasr, Assigned: lucasr)

Details

Attachments

(2 files)

What the summary says.
Comment on attachment 8371339 [details] [diff] [review]
Rename displayPanelConfig() to displayHomeConfig() (r=liuche)

For consistency with loadHomeConfig() and saveHomeConfig()
Attachment #8371339 - Flags: review?(liuche)
Attachment #8371340 - Flags: review?(liuche)
Comment on attachment 8371339 [details] [diff] [review]
Rename displayPanelConfig() to displayHomeConfig() (r=liuche)

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

lgtm
Attachment #8371339 - Flags: review?(liuche) → review+
Comment on attachment 8371340 [details] [diff] [review]
Don't use 'm' prefix for non-member variables (r=liuche)

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

Looks good. Just so I get the convention - we want to use the 'm' prefix for member instances in a class; are there any other prefix conventions that we use?
Attachment #8371340 - Flags: review?(liuche) → review+
(In reply to Chenxia Liu [:liuche] from comment #5)
> Comment on attachment 8371340 [details] [diff] [review]
> Don't use 'm' prefix for non-member variables (r=liuche)
> 
> Review of attachment 8371340 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks good. Just so I get the convention - we want to use the 'm' prefix for
> member instances in a class; are there any other prefix conventions that we
> use?

AFAIK, the only other prefix conversion is 's' for static class members.
https://hg.mozilla.org/mozilla-central/rev/31b96e201b86
https://hg.mozilla.org/mozilla-central/rev/ea70f78db5f5
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
You need to log in before you can comment on or make changes to this bug.