[HTTPS-Everywhere] New pref dialog broke Firefox 12 nightly

[Sujit Rao]

The OSes make it sound a lot like a path separator issue. It looks
like it might be related to
https://gitweb.torproject.org/https-everywhere.git/blobdiff/e662bc6f602d37458e6428ba2447b865de8327dd..faad987ad86c767bcb923d023d9c9e55066b0b33:/src/components/https-everywhere.js
. The setBoolPref method is on the branch, not  the prefs service. I
can't imagine why it would only crash on Windows, though.

On Wed, Jan 11, 2012 at 10:57 PM, Ake K. <ananuti at googlemail.com> wrote:
> Still reproducible in Windows 7 X86 but I can't reproduce in Linux X86.
> UA: Mozilla/5.0 (Windows NT 6.1; rv:12.0a1) Gecko/20120111 Firefox/12.0a1
>
> I see following errors on Error Console:
>
> HTTPS Everywhere: Error in ruleset file: TypeError: this.prefs.setBoolPref
> is not a function
> (line number: 47)
>
> Timestamp: 12/1/2555 10:35:41
> Warning: Error in parsing value for 'cursor'.  Declaration dropped.
> Source File: chrome://https-everywhere/content/preferences.xul
> Line: 0
>
> Timestamp: 12/1/2555 10:35:41
> Warning: XUL box for vbox element contained an inline #text child, forcing
> all its children to be wrapped in a block.
> Source File: chrome://https-everywhere/content/preferences.xul
> Line: 0
>
>
>
> On Thu, Jan 12, 2012 at 7:58 AM, Peter Eckersley <pde at eff.org> wrote:
>>
>> Fixed upstream, maybe?  Ake, can you still reproduce?
>>
>> On Wed, Jan 11, 2012 at 06:29:26PM -0500, Sujit Rao wrote:
>> > I just downloaded an nightly on Lion and couldn't reproduce anything,
>> > and the javascript: URI seemed to work just fine. It could still be
>> > useful to use inline options, especially for a Fennec version of HTTPS
>> > Everywhere.
>> >
>> > On Tue, Jan 10, 2012 at 5:12 PM, Peter Eckersley <pde at eff.org> wrote:
>> > >
>> > > On Mon, Jan 09, 2012 at 11:17:59PM -0500, Sujit Rao wrote:
>> > > > I suspect that this has to do with new security changes with
>> > > > javascript:
>> > > > URLs, which are used to open the preferences window in a way that
>> > > > makes it
>> > > > resizable. It could be fixed using the new add-ons options system (
>> > > > https://developer.mozilla.org/en/Extensions/Inline_Options), though
>> > > > there
>> > > > would need to be a different UI than the <tree>, or perhaps with the
>> > > > <prefwindow> and <prefpane> elements.
>> > >
>> > > By the way, let me know if you're working on this.  We'll need to
>> > > decide
>> > > whether to temporarily revert your XUL changes so that we can push a
>> > > new
>> > > development branch release without breaking FF 12 further.
>> > >
>> > > --
>> > > Peter Eckersley                            pde at eff.org
>> > > Technology Projects Director      Tel  +1 415 436 9333 x131
>> > > Electronic Frontier Foundation    Fax  +1 415 436 9993
>>
>> --
>> Peter Eckersley                            pde at eff.org
>> Technology Projects Director      Tel  +1 415 436 9333 x131
>> Electronic Frontier Foundation    Fax  +1 415 436 9993
>
>