Bug 11764 - buy.com shopping cart box disabled in Safari
Summary: buy.com shopping cart box disabled in Safari
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Evangelism (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL: http://www.buy.com
Keywords:
Depends on:
Blocks:
 
Reported: 2006-12-05 13:33 PST by rahul abrol
Modified: 2007-12-04 13:00 PST (History)
2 users (show)

See Also:


Attachments
Screenshot of shopping cart box from Firefox 2.0 (31.88 KB, image/png)
2006-12-06 03:19 PST, David Kilzer (:ddkilzer)
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description rahul abrol 2006-12-05 13:33:12 PST
buy.com, for example, has disabled their shopping cart box (top right of page) exclusively in safari.  i'm not sure why (they didn't respond to my queries), since enabling it manually and testing shows no errors.

they're using navigator.vendor to identify safari.  i was thinking of a simple fix to allow spoofing, like:

Index: WebCore/bindings/js/kjs_navigator.cpp
===================================================================
--- WebCore/bindings/js/kjs_navigator.cpp       (revision 18029)
+++ WebCore/bindings/js/kjs_navigator.cpp       (working copy)
@@ -181,6 +181,8 @@ JSValue* Navigator::getValueProperty(Exe
   case ProductSub:
     return jsString("20030107");
   case Vendor:
+    if (userAgent.find("Safari") == -1)
+      return jsString("");
     return jsString("Apple Computer, Inc.");
   case VendorSub:
     return jsString("");
Comment 1 David Kilzer (:ddkilzer) 2006-12-05 14:45:36 PST
Changing sense of this bug to be a buy.com evangelism issue.
Comment 2 David Kilzer (:ddkilzer) 2006-12-06 03:18:50 PST
Confirming that the shopping cart box does not appear in a locally-built debug build of WebKit r18014 on Mac OS X 10.4.8 (8L127).

Comment 3 David Kilzer (:ddkilzer) 2006-12-06 03:19:48 PST
Created attachment 11749 [details]
Screenshot of shopping cart box from Firefox 2.0
Comment 4 rahul abrol 2006-12-06 09:14:24 PST
actually i was just using buy.com as an example.  i meant for the bug to be about improving user agent spoofing.  i guess i'll leave it to someone more knowledgeable to decide if it's worth filing a separate bug for that.
Comment 5 David Kilzer (:ddkilzer) 2006-12-06 11:23:15 PST
(In reply to comment #4)
> actually i was just using buy.com as an example.  i meant for the bug to be
> about improving user agent spoofing.  i guess i'll leave it to someone more
> knowledgeable to decide if it's worth filing a separate bug for that.

That tends to be a slippery slope.  Once you start making small modifications here and there for individual web sites, it may start affecting other sites adversely that want to support the browser properly.

Also, changing the web site to support a browser properly is a lot easier (only has to be changed in one or a few places) than changing the browser code itself (which must be distributed to thousands, if not millions, of users).

Having said that, it's good that you spotted this difference in behavior!  I don't think I would have spotted it, especially if I hadn't used anything other than Safari to browse the web.
Comment 6 rahul abrol 2006-12-06 13:40:32 PST
FWIW, here's the response i just got from buy.com:

"Buy.com can only be accessed by using internet explorer. Buy.com can not be accessed by another web browser."

touché.
Comment 7 rahul abrol 2007-12-04 13:00:20 PST
site redesigned, no longer a buy.com issue.

marking as WONTFIX, since spoofing patch was ignored.