Ensure support for Internet Explorer 9

Bug #622401 reported by Lthere
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
FakeSmile
Fix Released
Medium
Lthere

Bug Description

Given that IE9 is implementing SVG but not implementing SMIL, this means another great opportunity for FakeSmile to shine and bring SMIL support (even if partial) to IE9 users as well.

I've just quickly tested the FakeSmile homepage [1] using the IE9 platform preview 4 [2] and it almost works OK.

Couple of things noticed:

1. A script error is thrown as FakeSmile makes use of overrideMimeType in XMLHttpRequests. This API seems to be part of version two of the specification [3] and is not supported by IE, a simple workaround is proposed [3] - just replace:
  if(request) {
    request.overrideMimeType('text/xml');
    request.onreadystatechange = xhrCallback;
  }
By the following:
  if(request) {
    if(request.overrideMimeType){
      // part of XMLHttpRequest2, IE doesn't implement this method
      request.overrideMimeType('text/xml');
    }
    request.onreadystatechange = xhrCallback;
This avoids the script error (which apparently wasn't breaking anything special in the example [1], but was annoying). I'm not sure if it an XMLHttpRequest2 will be made available by the time IE9 reaches the release state, as it is stated in the current release notes that:
"Network stack improvements previously described on the IEBlog are not available in the IE9 Platform Preview builds. The platform preview builds contain the IE8 version of WinINET.dll" [4]
Anyway, the change won't break anything and will start behaving as expected once it is implemented. :-)

2. The onclick animation triggered for the Firefox icon isn't working. I haven't yet tried hard to debug why, but opening the SVG file directly [5] didn't help (at least, helped understanding it isn't an SVG-embedding-into-HTML issue).

Hope this helps,
 Helder

[1] http://leunen.d.free.fr/fakesmile/
[2] http://blogs.msdn.com/b/ie/archive/2010/08/04/html5-modernized-fourth-ie9-platform-preview-available-for-developers.aspx
[3] http://www.w3.org/TR/XMLHttpRequest2/#the-overridemimetype-method
[3] http://robert.accettura.com/blog/2006/12/02/xmlhttpreqoverridemimetype-in-ie7/
[4] http://ie.microsoft.com/testdrive/info/ReleaseNotes/Default.html
[] http://leunen.d.free.fr/fakesmile/example.svg

Tags: ie9

Related branches

Revision history for this message
Fenring (leunen-d) wrote :

Thanks Helder.

I applied your point #1 patch.

I'm not able to install IE9. I leave this bug open for anyone who wants to help.

Changed in smil:
status: New → Confirmed
Revision history for this message
Lthere (helder-magalhaes) wrote :

Taking bug as I've been doing a few test using IE9 in the latest few days and already noticed a couple of things for fix.

Changed in smil:
assignee: nobody → HelderMagalhaes (helder-magalhaes)
Revision history for this message
Lthere (helder-magalhaes) wrote :

Can no longer reproduce item number 2 on IE9 (9.0.4) so assuming this was fixed before the final release (report was originally made using IE9 platform preview 4). Test case, previously referred to as [5], is now available at [1].

Leaving this issue open for a follow-up commit which will fix a minor issue which I'm still experiencing.

[1] http://leunen.me/fakesmile/example2.svg

Revision history for this message
Lthere (helder-magalhaes) wrote :

Changing to medium importance as IE9 user base is increasing.

Changed in smil:
status: Confirmed → In Progress
importance: Undecided → Medium
Changed in smil:
status: In Progress → Fix Committed
Changed in smil:
status: Fix Committed → Fix Released
Changed in smil:
status: Fix Released → Fix Committed
Revision history for this message
Lthere (helder-magalhaes) wrote :

Rolling back status - fix is not "released" yet: official download location (http://leunen.me/fakesmile/ - mentioned by David) needs to be updated to at least revision 37.

Revision history for this message
Lthere (helder-magalhaes) wrote :

Revision 48 is now seen at the official download location, http://leunen.me/fakesmile/ - thanks for mentioning that, David!

Changed in smil:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.