updroid / jquery-jstore

Automatically exported from code.google.com/p/jquery-jstore
1 stars 0 forks source link

Flash on Google Chrome #1

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Google chrome is not currently recognizing flash, and is allowing only
google gears access. I'll do some investigation into the problem and see if
I can figure out why.

Original issue reported on code.google.com by gars...@gmail.com on 27 Feb 2009 at 4:46

GoogleCodeExporter commented 9 years ago
I found I had to remove the check for window.swfobject in jStore.Flash - since 
the
plugin doesn't require SWFObject I'm not quite sure why this check is in place.

Original comment by thefutur...@googlemail.com on 22 Apr 2009 at 12:00

GoogleCodeExporter commented 9 years ago
Ah, great! I think that was doing it. It was a stitch of vestigial code which 
was in
the flash engine back from when it used swfobject instead of the jQuery 
equivalent.
Thanks for finding that. I'll be pushing out a new release with some of these 
fixes
later today.

Original comment by gars...@gmail.com on 22 Apr 2009 at 1:33

GoogleCodeExporter commented 9 years ago
Hm that wasn't enough to get it working in Chrome. I was getting an error that 
Chrome 
couldn't call f_get_cookie on an undefined object (this.db). I figured out this 
was 
because it was trying to find the domain via $('embed', '#jstore-flash-www-
my.domain.com'). Those extra dots were interpreted as classes, thus it never 
found 
the embed.

The solution was to do a global regular expression in jStore.js (/\./g) instead 
of a 
one-time replace (/\./):

project = project || $.jStore.defaults.project || 
location.hostname.replace(/\./g, '-
') || 'unknown';

Now Flash works great in Chrome.

Original comment by thefutur...@googlemail.com on 23 Apr 2009 at 4:36

GoogleCodeExporter commented 9 years ago
Thanks again for your help, future! I've added the global regex flag, and 
released
your patch as jStore-1.0.3.

Original comment by gars...@gmail.com on 23 Apr 2009 at 4:53