297984 - all.js sets prefs that are duplicated in all-camino.js

Status: RESOLVED FIXED

(Camino Graveyard :: General, defect)

Description

[Kevin Brosnan]

I was working on a Mac bug for Firefox and was going to transfer a Camino
preferences patch to Firefox. MConner pointed out that the settings were already
present in all.js. So I took a look at all-camino.js and found several lines
that are the same as all.js
 
all.js 746-748 = all-camino.js 94-96
// enable horizontal scroll by shift+wheel
pref("mousewheel.horizscroll.withshiftkey.action", 0);
pref("mousewheel.horizscroll.withshiftkey.numlines", 1);
pref("mousewheel.horizscroll.withshiftkey.sysnumline", true);

all.js 740-742 = all-camino.js 99-101
// enable horizontal scrolling with horizontal scroll wheels
pref("mousewheel.horizscroll.withnokey.action",0);
pref("mousewheel.horizscroll.withnokey.numlines",1);
pref("mousewheel.horizscroll.withnokey.sysnumlines",true);

all.js 761 = all-camino.js 104
pref("mousewheel.horizscroll.withaltkey.action", 2);

I also took out the comment about fonts. All told this trims about a 1000 bytes.

Comment 1

[Kevin Brosnan]

Attachment: a removes dup lines from all-camino.js

Comment 2

[Mark Mentovai]

Fixed with the checkin from bug 293503.