Google Mail Calendar Documents Reader Web more »
Recently Visited Groups | Help | Sign in
Google Groups Home
Firefox 3.5...let's fix it.
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  7 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Follow-up To:
Add Cc | Add Follow-up to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers that you hear
 
Dan Webb  
View profile   Translate to Translated (View Original)
 More options 30 June, 17:22
From: Dan Webb <d...@danwebb.net>
Date: Tue, 30 Jun 2009 17:22:31 +0100
Local: Tues 30 June 2009 17:22
Subject: Firefox 3.5...let's fix it.
Yes, so this looks like it needs addressing urgently.  I was really
hoping it was just a regression but no such luck.  I need to read back
through all these posts but if anyone can summarise the problems
they've encountered so far here that would be a great help.

Thanks,

Dan

--
Dan Webb
http://massiverobot.co.uk
http://www.danwebb.net

aim: danwrong123
twitter/skype: danwrong


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Dan Webb  
View profile   Translate to Translated (View Original)
 More options 30 June, 18:16
From: Dan Webb <d...@danwebb.net>
Date: Tue, 30 Jun 2009 18:16:47 +0100
Local: Tues 30 June 2009 18:16
Subject: Re: Firefox 3.5...let's fix it.
Woah, just from running the test pages it seems that things are pretty
messy.  It looks like its something to do with the inheritance code
which is nearly identical to prototypes except for the way the
constructor is created...and that bit is pretty hairy to be fair.  So,
I'm going to dig in to this and try to root out the problem.  This
should probably fix most if not all of the problems we are
experiencing though.

If anyone else wants to dig in and assist that would be great.

--
Dan Webb
http://massiverobot.co.uk
http://www.danwebb.net

aim: danwrong123
twitter/skype: danwrong


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
alex wallace  
View profile   Translate to Translated (View Original)
 More options 30 June, 20:57
From: alex wallace <alexmlwall...@gmail.com>
Date: Tue, 30 Jun 2009 12:57:40 -0700 (PDT)
Local: Tues 30 June 2009 20:57
Subject: Re: Firefox 3.5...let's fix it.
I've fixed the problems with Lowpro 0.5 in FF3.5 by changing the
second definition of "var behavior" to "var behaving" -- it seems
FF3.5 did not like the duplicate reference within the function block
for the initial "var behavior".

Diff for patching:
http://pastie.org/529840

Or you can grab the full patch from:
http://eightysevenfour.com/lowpro_ff_3_5_patch.js

I've tested this so far in FF3.5 on XP and Leopard, FF3.0.11 on
Leopard, Safari 4 on Leopard, Safari 3.2 on XP, Chrome, and our good
friends Internet Explorer 6 and 7. Worksforme, testing with an
interface chock-full of fairly complex Behaviors.

Best,
Alex Wallace

On 30 June, 13:16, Dan Webb <d...@danwebb.net> wrote:


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Dan Webb  
View profile   Translate to Translated (View Original)
 More options 30 June, 21:16
From: Dan Webb <d...@danwebb.net>
Date: Tue, 30 Jun 2009 21:16:48 +0100
Local: Tues 30 June 2009 21:16
Subject: Re: Firefox 3.5...let's fix it.
That's really interesting.  I'd just worked out the fix was to remove:

var behavior = arguments.callee;

You fix actually works but only bby mistake.  The reason it works is
because of this:

return function() {
            var initArgs = [this].concat(args);
            behavior.attach.apply(behaving, initArgs);
          };

The fact that you forgot to change the first reference of behavior to
behaving got around the bug.  Man, there's something strange going on
there.  I did learn something though.  The function behavior actually
already has a reference to itself through the closure so that
arguments.callee stuff is completely redundant.

Thanks very much for that Alex.  That really pointed me in the right
direction....I'd have been there for hours otherwise.  If we ever meet
I owe you a beer :)

Fixed version is on github.

--
Dan Webb
http://massiverobot.co.uk
http://www.danwebb.net

aim: danwrong123
twitter/skype: danwrong


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
alex wallace  
View profile   Translate to Translated (View Original)
 More options 30 June, 21:23
From: alex wallace <alexmlwall...@gmail.com>
Date: Tue, 30 Jun 2009 13:23:44 -0700 (PDT)
Local: Tues 30 June 2009 21:23
Subject: Re: Firefox 3.5...let's fix it.
Nice one. I thought the trick was to separate the variable names so
that attach could have the newly distinct var applied to it - but it
seems the naming conflict was easily avoided. Glad to have it fixed
and back in trunk.

Best,
A

On 30 June, 16:16, Dan Webb <d...@danwebb.net> wrote:


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Dan Webb  
View profile   Translate to Translated (View Original)
 More options 30 June, 21:25
From: Dan Webb <d...@danwebb.net>
Date: Tue, 30 Jun 2009 21:25:24 +0100
Local: Tues 30 June 2009 21:25
Subject: Re: Firefox 3.5...let's fix it.
And now I can go back to playing Fallout 3...  Thanks again, Alex!

--
Dan Webb
http://massiverobot.co.uk
http://www.danwebb.net

aim: danwrong123
twitter/skype: danwrong


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Nathan  
View profile   Translate to Translated (View Original)
 More options 30 June, 23:09
From: Nathan <XGamer...@gmail.com>
Date: Tue, 30 Jun 2009 15:09:47 -0700 (PDT)
Local: Tues 30 June 2009 23:09
Subject: Re: Firefox 3.5...let's fix it.
Thanks guys for addressing this issue so quickly! I was panicking when
I saw that ff3.5 reverted to being problematic with lowpro. Amazing
how the problem resurfaced after RC2 for the final release. I am glad
the fix was so easy as removing that one line with a redundant
variable though.

    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »

Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy
©2009 Google