Bug 7628
| Summary: | Gap.com, banannarepublic.com, oldnavy.com say Safari 417.9 won't work | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Geoffrey Garen <ggaren> |
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | accessibleweb |
| Priority: | P2 | ||
| Version: | 420+ | ||
| Hardware: | Mac | ||
| OS: | OS X 10.4 | ||
| Bug Depends on: | |||
| Bug Blocks: | 7670, 7671 | ||
Geoffrey Garen
This bug is also in Radar as <rdar://4464698>
* SUMMARY
Safari 2.0.3 (417.9): www.gap.com doesn't support Safari even though it says it does
* STEPS TO REPRODUCE
1. Launch Safari 417.9
2. Go to www.gap.com
* RESULTS
We're sorry but we do not support the version of the browser you are using. Our site works best with the following browsers:
Netscape 7
Mozilla (including Firefox)
Safari 2.0.3 and above (Safari 2.0.3 is part of the Mac OS X v10.4.4 update)
See image.
* REGRESSION
Occurs:
Safari 2.0.3 (417.9)
Safari 2.0.3 (417.9)
Doesn't occur:
Safari 2.0.3 (417.8)
<GMT02-Mar-2006 20:30:05GMT> Leslie Case:
Doesn't occur with www.gapinc.com, but does occur with all their brands.
<GMT02-Mar-2006 22:55:23GMT> Alice Liu:
problem seen:
(417.9)
Safari TOT (420+)
problem not seen:
(417.8)
Leopard (521.7.2)
I talked to Tim Hatcher about this and he thinks that Gap is probably doing their Safari version check too strictly or incorrectly, and that it's not a regression on the Safari side. This made sense until I tested Leopard and it worked.
<GMT03-Mar-2006 23:22:04GMT> Alice Liu:
this might end up being an evangelism issue, but we should see what we can figure out on our end first.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Geoffrey Garen
gap.com's sniffing code compares for ">= 417.7". When we bumped the WebKit version number from 417.9 to 417.10, it became numerically less than 417.7.
We can fix this temporarily by bumping the WebKit version number to 418, but we'll need to get in touch with them about fixing it on their end as well.
David Kilzer (:ddkilzer)
Nightly builds don't work on their site, either. Their browser sniffing code needs more work. Isn't this a duplicate of Bug 6623?
Geoffrey Garen
Not a dup because bug 6623 tracks failure to render.
Brian Hagan
This should have been fixed as of 3/28. Can someone please hit one of the three sites with a nightly build?
Alice Liu
working on 10.4.6 (417.9.2) but not TOT (420+).
Geoffrey Garen
The sites still fail to parse "420+", which is the version number of our latest sources, but they work with 417.9. Since we don't use "+" in released versions of Safari, I think we can close this bug. But it would be nice if the sites' parsing improved to include "420+" as well.