Bug 124607 - Web Speech API crashes onboundary event handling with reload
Summary: Web Speech API crashes onboundary event handling with reload
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: chris fleizach
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2013-11-19 14:38 PST by chris fleizach
Modified: 2013-11-19 14:48 PST (History)
2 users (show)

See Also:

Attachments
patch (1.86 KB, patch)
2013-11-19 14:41 PST, chris fleizach 		thorton: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description chris fleizach 2013-11-19 14:38:50 PST 
The Web Speech API crashes Safari when given a paragraph of text.  It also display inconsistent behavior with onboundary call backs.
Comment 1 chris fleizach 2013-11-19 14:39:09 PST 
<rdar://problem/15507781>
Comment 2 chris fleizach 2013-11-19 14:41:13 PST 
Created attachment 217334 [details]
patch
Comment 3 chris fleizach 2013-11-19 14:44:02 PST 
Wrong radar: it should be <rdar://15490503>
Comment 4 Tim Horton 2013-11-19 14:44:17 PST 
Comment on attachment 217334 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=217334&action=review

> Source/WebCore/platform/mac/PlatformSpeechSynthesizerMac.mm:170
> +- (void)tearDown

maybe -clear? We use that name in a bunch of other places.
Comment 5 Tim Horton 2013-11-19 14:44:40 PST 
Alternatively, invalidate.
Comment 6 chris fleizach 2013-11-19 14:45:39 PST 
(In reply to comment #4)
> (From update of attachment 217334 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=217334&action=review
> 
> > Source/WebCore/platform/mac/PlatformSpeechSynthesizerMac.mm:170
> > +- (void)tearDown
> 
> maybe -clear? We use that name in a bunch of other places.

Will do
Comment 7 chris fleizach 2013-11-19 14:48:01 PST 
http://trac.webkit.org/changeset/159523

went with invalidate