Bug 103264

Summary:

INVALID_STATE_ERR error upon calling .addCue() for an already-in-use cue

Product:

WebKit

Reporter:

Antoine Quint <graouts>

Component:

Media

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, dino, eric.carlson, feature-media-reviews, gyuyoung.kim, ojan, rakuco, webkit-bug-importer, webkit.review.bot

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

URL:

http://w3c-test.org/html/tests/submission/Opera/media/interfaces/TextTrack/addCue.html

Attachments:

Description	Flags
Patch	none
Patch	none
Patch	eric.carlson: review+, webkit.review.bot: commit-queue-

Description Antoine Quint 2012-11-26 08:43:21 PST

The first test of the Opera-submitted test at http://w3c-test.org/html/tests/submission/Opera/media/interfaces/TextTrack/addCue.html fails because we try adding a cue to a TextTrack and that cue was already added to a previous track. The spec says that the cue must first be removed from the previous track before being added to the new track (http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html#dom-texttrack-addcue). However, when we try to do that we get an INVALID_STATE_ERR error thrown, which is odd because the .addCue() method does not seem to throw exceptions.

Comment 1 Antoine Quint 2012-11-27 01:54:07 PST

<rdar://problem/12756208>

Comment 2 Antoine Quint 2012-12-04 02:28:58 PST

Created attachment 177456 [details]
Patch

Comment 3 Antoine Quint 2012-12-04 02:29:47 PST

*** Bug 103265 has been marked as a duplicate of this bug. ***

Comment 4 WebKit Review Bot 2012-12-04 05:34:08 PST

Comment on attachment 177456 [details]
Patch

Rejecting attachment 177456 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
it line 152.

Failed to run "['Tools/Scripts/update-webkit', '--chromium', '--force-update']" exit_code: 2
Updating OpenSource
From http://git.chromium.org/external/Webkit
 + 9347176...16632b5 HEAD       -> origin/HEAD  (forced update)
error: Ref refs/remotes/origin/master is at 16632b5a2ddd0ea34bb124d8f915fd9521310601 but expected 93471767af9bca55738741a12786ee27fce6e006
 ! 9347176...16632b5 master     -> origin/master  (unable to update local ref)
Died at Tools/Scripts/update-webkit line 152.

Full output: http://queues.webkit.org/results/15133401

Comment 5 Antoine Quint 2012-12-04 06:32:21 PST

Created attachment 177473 [details]
Patch

Comment 6 Antoine Quint 2012-12-04 08:03:34 PST

Created attachment 177481 [details]
Patch

Comment 7 WebKit Review Bot 2012-12-04 08:31:55 PST

Comment on attachment 177481 [details]
Patch

Rejecting attachment 177481 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
ripts/update-webkit line 152.

Failed to run "['Tools/Scripts/update-webkit', '--chromium', '--force-update']" exit_code: 2
Updating OpenSource
From http://git.chromium.org/external/Webkit
   700060b..82ee932  HEAD       -> origin/HEAD
error: Ref refs/remotes/origin/master is at 82ee9326539c61f4c4cb100778394db7ec65bec2 but expected 700060b2bfffbae7da2848a0b266fb18f70e8b39
 ! 700060b..82ee932  master     -> origin/master  (unable to update local ref)
Died at Tools/Scripts/update-webkit line 152.

Full output: http://queues.webkit.org/results/15121796

Comment 8 Dean Jackson 2012-12-04 10:49:05 PST

Committed r136528: <http://trac.webkit.org/changeset/136528>