Closed Bug 1297732 Opened 8 years ago Closed 8 years ago

Remove redundancies in prefs files

Categories

(Firefox for Android Graveyard :: Toolbar, defect, P3)

51 Branch
All
Android
defect

Tracking

(firefox51 fixed)

RESOLVED FIXED
Firefox 51
Tracking Status
firefox51 --- fixed

People

(Reporter: kats, Assigned: kats)

Details

Attachments

(1 file)

mobile.js has a #ifdef MOZ_WIDGET_ANDROID which is unnecessary, since mobile.js is only read on Fennec anyway.

Also all.js has a few prefs that are set to the same value both inside and outside a #ifdef ANDROID block, and so there's some redundancy there that can be removed. Both of these are leftover from bug 1291373.

Additionally since the layout.css.scroll-behavior.enabled pref is true on all platforms now, we should set the default to true in gfxPrefs.h as well.
Comment on attachment 8784400 [details]
Bug 1297732 - Eliminate some redundancies in prefs files.

https://reviewboard.mozilla.org/r/73868/#review71730
Attachment #8784400 - Flags: review?(rbarker) → review+
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ff3d89bec675
Eliminate some redundancies in prefs files. r=rbarker
https://hg.mozilla.org/mozilla-central/rev/ff3d89bec675
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: