Bug 82095 - Remove unnecessary ASSERT from LoadableTextTrack::trackElementIndex
Summary: Remove unnecessary ASSERT from LoadableTextTrack::trackElementIndex
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Carlson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2012-03-23 14:45 PDT by Eric Carlson
Modified: 2012-03-23 17:34 PDT (History)
3 users (show)

See Also:


Attachments
Proposed patch (1.28 KB, patch)
2012-03-23 15:11 PDT, Eric Carlson
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Carlson 2012-03-23 14:45:01 PDT
I left a bogus ASSERT in LoadableTextTrack::trackElementIndex when I committed http://trac.webkit.org/changeset/111581.  It is perfectly fine for the node to not be in the DOM, and the method handles the situation.
Comment 1 Radar WebKit Bug Importer 2012-03-23 14:49:09 PDT
<rdar://problem/11112196>
Comment 2 Eric Carlson 2012-03-23 15:11:29 PDT
Created attachment 133569 [details]
Proposed patch
Comment 3 WebKit Review Bot 2012-03-23 17:34:49 PDT
Comment on attachment 133569 [details]
Proposed patch

Clearing flags on attachment: 133569

Committed r111941: <http://trac.webkit.org/changeset/111941>
Comment 4 WebKit Review Bot 2012-03-23 17:34:53 PDT
All reviewed patches have been landed.  Closing bug.