read aloud fail in Safari

Bug #680305 reported by Jeff Kaplan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Internet Archive BookReader
Fix Released
High
raj

Bug Description

plays first chunk of text then stops. fail load bar never completes loading.

Safari 4.0.3 / MacOSX

Tags: qa-reopened
description: updated
Revision history for this message
mangtronix (mang) wrote :

Raj this report came in from QA. Can you please have a look?

Changed in bookreader:
assignee: nobody → raj (raj-archive)
importance: Undecided → High
milestone: none → booksinbrowsers
Revision history for this message
raj (raj-archive) wrote :

Jeff, I'm having trouble reproducing this.

Is this happening with all books for you, or this there a particular book that is failing?

Thanks!

Changed in bookreader:
assignee: raj (raj-archive) → Jeff Kaplan (kaplan-archive)
Revision history for this message
raj (raj-archive) wrote :

I can reproduce on the QA machine (Safari 5.0.2 / OS X 10.6.4) but not on my mac (Safari 5.0 / OS X 10.6.4).

Changed in bookreader:
assignee: Jeff Kaplan (kaplan-archive) → raj (raj-archive)
Revision history for this message
raj (raj-archive) wrote :

(Can reproduce on all books on the QA machine.)

Revision history for this message
raj (raj-archive) wrote :

The problem I was seeing on Safari 5.0.2 was due to broken HTML 5 audio support in Safari (and QT underneath: https://bugs.webkit.org/show_bug.cgi?id=32159). Upgrading to version V2.97a.20101010 of SoundManager2 fixes this.

Jeff originally reported this problem with Safari 4.0.3, so I will try to test there as well.

mangtronix (mang)
Changed in bookreader:
status: New → Confirmed
Revision history for this message
raj (raj-archive) wrote :

Well, read aloud on Jeff's machine failed in the same way it failed on the QA machine, so without thinking too hard, I applied the same fix. Jeff had OS X 10.6.0, so I modified SoundManager2 to disable html5 audio for Safari on all versions of OS X 10.6. Before, only 10.6.3 and 10.6.4 were being treated as broken..

We don't have a great amount of test coverage on this. What about on OS X 10.5? And updating soundmanager means retesting read aloud on all platforms. Also, it is hard to get Safari to force reload of the new soundmanager swfs, even though it loads the new js....

Revision history for this message
raj (raj-archive) wrote :

Checked soundmanager updates into the petabox tree. moving to triaged.

Changed in bookreader:
status: Confirmed → Triaged
mangtronix (mang)
tags: added: needs-qa
Revision history for this message
mangtronix (mang) wrote :

Please check the Read Aloud feature on Safari on OSX 10.5 (Leopard) as well as other platforms.

*You must quit your browser and restart it before testing* in order to reload the SoundManager plugin that we use for playing audio. Please report the browser / OS combinations tested here.

Changed in bookreader:
assignee: raj (raj-archive) → alexisrossi (alexis-archive)
Revision history for this message
alexisrossi (alexis-archive) wrote :

Tested on:
Mac OSX 10.6.2, Safari 4.0.4
(was previously failing on this setup)

Have tested several books, all read aloud as expected including turning pages to continue.

Revision history for this message
Jeff Kaplan (kaplan-archive) wrote :

Read aloud not working/loading in IE7 or IE8.

Minor quibble: starting read aloud while page is turning in autoflip results in previous page being read while you are looking at the subsequent page.

tags: added: qa-reopened
removed: needs-qa
Changed in bookreader:
assignee: alexisrossi (alexis-archive) → mangtronix (mang)
Revision history for this message
mangtronix (mang) wrote :

I get "Could not load soundManager 2" after pressing the speaker icon on IE7/XP.

Changed in bookreader:
assignee: mangtronix (mang) → raj (raj-archive)
status: Triaged → Confirmed
Revision history for this message
mangtronix (mang) wrote :

Jeff can you provide more details on IE8? We need the steps to reproduce and what you saw when the error happened.

Revision history for this message
raj (raj-archive) wrote :

Fix has been pushed to the petabox tree, and after the full-text reindexing, we can't reproduce any IE8 read aloud issues. closing.

Changed in bookreader:
status: Confirmed → 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.