Re: [webkit-dev] map.d.co.il broken - back out r66156?

2010-09-19 Thread Jeremy Moskovich
Thanks Maciej,

The original bug had no information on the sites that were fixed, very good
to hear that this improves compatibility!

Can you comment on whether this is being actively worked on?  Or is this
waiting for someone to pick it up?  Just want to make sure we can fix this
regression ASAP.

Thanks,
Jeremy

On Thu, Sep 16, 2010 at 6:42 PM, Maciej Stachowiak m...@apple.com wrote:


 On Sep 16, 2010, at 9:14 AM, Jeremy Moskovich wrote:

 Hi,

 http://trac.webkit.org/changeset/66156/ broke map.d.co.il which is one of
 the larger maps sites in Israel.

 https://bugs.webkit.org/show_bug.cgi?id=45679 was filed to track this.

 Andy: do you have an eta for a fix? Are you ok reverting the change in the
 interim?


 How about we take a shot at fixing it? We don't usually revert just because
 because one site is broken, even for more popular sites than this one. The
 original change fixed serious problems on multiple sites, including at least
 two Alexa top 500 sites, so I'm not sure reverting would even make WebKit
 more compatible on the whole.

 Regards,
 Maciej



___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


Re: [webkit-dev] map.d.co.il broken - back out r66156?

2010-09-19 Thread Andy Estes
Hi Jeremy,

I'm working on this. Thanks for bringing it to my attention!

-Andy

On Sep 19, 2010, at 2:17 AM, Jeremy Moskovich wrote:

 Thanks Maciej,
 
 The original bug had no information on the sites that were fixed, very good 
 to hear that this improves compatibility!
 
 Can you comment on whether this is being actively worked on?  Or is this 
 waiting for someone to pick it up?  Just want to make sure we can fix this 
 regression ASAP.
 
 Thanks,
 Jeremy
 
 On Thu, Sep 16, 2010 at 6:42 PM, Maciej Stachowiak m...@apple.com wrote:
 
 On Sep 16, 2010, at 9:14 AM, Jeremy Moskovich wrote:
 
 Hi,
 
 http://trac.webkit.org/changeset/66156/ broke map.d.co.il which is one of 
 the larger maps sites in Israel.
 
 https://bugs.webkit.org/show_bug.cgi?id=45679 was filed to track this.
 
 Andy: do you have an eta for a fix? Are you ok reverting the change in the 
 interim?
 
 How about we take a shot at fixing it? We don't usually revert just because 
 because one site is broken, even for more popular sites than this one. The 
 original change fixed serious problems on multiple sites, including at least 
 two Alexa top 500 sites, so I'm not sure reverting would even make WebKit 
 more compatible on the whole.
 
 Regards,
 Maciej
 
 
 

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


[webkit-dev] map.d.co.il broken - back out r66156?

2010-09-16 Thread Jeremy Moskovich
Hi,

http://trac.webkit.org/changeset/66156/ broke map.d.co.il which is one of
the larger maps sites in Israel.

https://bugs.webkit.org/show_bug.cgi?id=45679 was filed to track this.

Andy: do you have an eta for a fix? Are you ok reverting the change in the
interim?

Best regards,
Jeremy
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


Re: [webkit-dev] map.d.co.il broken - back out r66156?

2010-09-16 Thread Maciej Stachowiak

On Sep 16, 2010, at 9:14 AM, Jeremy Moskovich wrote:

 Hi,
 
 http://trac.webkit.org/changeset/66156/ broke map.d.co.il which is one of the 
 larger maps sites in Israel.
 
 https://bugs.webkit.org/show_bug.cgi?id=45679 was filed to track this.
 
 Andy: do you have an eta for a fix? Are you ok reverting the change in the 
 interim?

How about we take a shot at fixing it? We don't usually revert just because 
because one site is broken, even for more popular sites than this one. The 
original change fixed serious problems on multiple sites, including at least 
two Alexa top 500 sites, so I'm not sure reverting would even make WebKit more 
compatible on the whole.

Regards,
Maciej


___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev