Drop [IsIndex] IDL extended attribute from Speech types

Drop [IsIndex] IDL extended attribute from Speech types

Drop [IsIndex] IDL extended attribute from Speech types as the corresponding
specifications do not mandate that we should throw an IndexsizeError
exception on negative values:
https://dvcs.w3.org/hg/speech-api/raw-file/tip/speechapi.html
http://lists.w3.org/Archives/Public/public-xg-htmlspeech/2011Feb/att-0020/api-draft.html#speech_input_result_list_interface

BUG=237739
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=155587

[do-not-use, 2013-08-05 15:04:47]

Firefox does not seem to support Web Speech or Speech input.

Also, this should be a safe change for compatibility since we stop throwing.

[arv (Not doing code reviews), 2013-08-05 15:15:28]

LGTM

https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
File LayoutTests/fast/speech/input-onspeechchange-event.html (right):

https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
LayoutTests/fast/speech/input-onspeechchange-event.html:10: function
onSpeechChange() {
Can you get rid of the deprecated window.event?

var result;
function onSpeechChange(e) {
  result = e.result;
  ...
}

[haraken, 2013-08-05 15:26:17]

LGTM. Needs an approval from an API owner.

[do-not-use, 2013-08-05 15:27:45]

https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
File LayoutTests/fast/speech/input-onspeechchange-event.html (right):

https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
LayoutTests/fast/speech/input-onspeechchange-event.html:10: function
onSpeechChange() {
On 2013/08/05 15:15:28, arv wrote:
> Can you get rid of the deprecated window.event?
> 
> var result;
> function onSpeechChange(e) {
>   result = e.result;
>   ...
> }

This does not seem to work. There is apparently no argument passed to
onSpeechChange().

https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
LayoutTests/fast/speech/input-onspeechchange-event.html:10: function
onSpeechChange() {
On 2013/08/05 15:15:28, arv wrote:
> Can you get rid of the deprecated window.event?
> 
> var result;
> function onSpeechChange(e) {
>   result = e.result;
>   ...
> }

This does not seem to work. There is apparently no argument passed to
onSpeechChange().

[arv (Not doing code reviews), 2013-08-05 15:36:00]

https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
File LayoutTests/fast/speech/input-onspeechchange-event.html (right):

https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
LayoutTests/fast/speech/input-onspeechchange-event.html:10: function
onSpeechChange() {
On 2013/08/05 15:27:45, Christophe Dumez wrote:
> On 2013/08/05 15:15:28, arv wrote:
> > Can you get rid of the deprecated window.event?
> > 
> > var result;
> > function onSpeechChange(e) {
> >   result = e.result;
> >   ...
> > }
> 
> This does not seem to work. There is apparently no argument passed to
> onSpeechChange().

Did you update the inline event handler attribute?

<input id='speechInput' x-webkit-speech
onwebkitspeechchange="onSpeechChange(event)">

[do-not-use, 2013-08-05 15:39:08]

On 2013/08/05 15:36:00, arv wrote:
>
https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
> File LayoutTests/fast/speech/input-onspeechchange-event.html (right):
> 
>
https://codereview.chromium.org/22208002/diff/1/LayoutTests/fast/speech/input...
> LayoutTests/fast/speech/input-onspeechchange-event.html:10: function
> onSpeechChange() {
> On 2013/08/05 15:27:45, Christophe Dumez wrote:
> > On 2013/08/05 15:15:28, arv wrote:
> > > Can you get rid of the deprecated window.event?
> > > 
> > > var result;
> > > function onSpeechChange(e) {
> > >   result = e.result;
> > >   ...
> > > }
> > 
> > This does not seem to work. There is apparently no argument passed to
> > onSpeechChange().
> 
> Did you update the inline event handler attribute?
> 
> <input id='speechInput' x-webkit-speech
> onwebkitspeechchange="onSpeechChange(event)">

Ok, let me go and hide... :)
This is fixed now, it was indeed the issue, thanks.

[tkent, 2013-08-05 22:21:14]

lgtm

[commit-bot: I haz the power, 2013-08-06 06:30:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/22208002/8001

[commit-bot: I haz the power, 2013-08-06 08:03:12]

Change committed as 155587