Bug 72149 - [chromium] failing media/track/tracklist-is-reachable
Summary: [chromium] failing media/track/tracklist-is-reachable
Status: RESOLVED DUPLICATE of bug 72179
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tony Gentilcore
URL:
Keywords:
Depends on:
Blocks: 43668
  Show dependency tree / graph
 
Reported: 2011-11-11 10:13 PST by Tony Gentilcore
Modified: 2011-11-14 13:22 PST (History)
4 users (show)

See Also:

Attachments
Patch (1.37 KB, patch)
2011-11-11 10:20 PST, Tony Gentilcore 		no flags 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 Tony Gentilcore 2011-11-11 10:13:08 PST 
Chromium fails the media/track/tracklist-is-reachable test which was introduced by r99984.

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=media%2Ftrack%2Ftracklist-is-reachable.html

Apparently V8 doesn't allow us to set expando properties on non-DOM objects that are also EventTargets. Eric Carlson informs me the JSC code for this is in JSTextTrackListOwner::isReachableFromOpaqueRoots

We probably need to do something similar.
Comment 1 Tony Gentilcore 2011-11-11 10:20:01 PST 
Created attachment 114725 [details]
Patch
Comment 2 Tony Gentilcore 2011-11-11 10:20:34 PST 
Committed r99987: <http://trac.webkit.org/changeset/99987>
Comment 3 Tony Gentilcore 2011-11-11 10:21:09 PST 
My patch just sets expectations. Reopening to track the real fix.
Comment 4 Adam Barth 2011-11-13 23:01:35 PST 
Yeah, we need to set up the proper object groupings in V8GCController.
Comment 5 Anna Cavender 2011-11-14 13:22:32 PST 

*** This bug has been marked as a duplicate of bug 72179 ***