Bug 133240

Summary:

[Mac] preload AVAssetTrack properties before asking for them

Product:

WebKit

Reporter:

Eric Carlson <eric.carlson>

Component:

Media

Assignee:

Eric Carlson <eric.carlson>

Status:

ASSIGNED ---

Severity:

Normal

CC:

commit-queue, glenn, jer.noble, philipj, sergio, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Proposed patch.	jer.noble: review+
Patch for landing.	none

Description Eric Carlson 2014-05-23 16:25:41 PDT

r168979 started checking AVAssetTrack preferredTransform and naturalSize, so we should preload them along with the other track properties we use.

Comment 1 Eric Carlson 2014-05-23 16:26:21 PDT

<rdar://problem/17022444>

Comment 2 Eric Carlson 2014-05-23 16:28:55 PDT

Created attachment 232001 [details]
Proposed patch.

Comment 3 WebKit Commit Bot 2014-05-23 16:30:58 PDT

Attachment 232001 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:2558:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.

Comment 4 Jer Noble 2014-05-23 18:31:28 PDT

Comment on attachment 232001 [details]
Proposed patch.

r=me

Comment 5 Eric Carlson 2014-05-24 23:24:42 PDT

Created attachment 232030 [details]
Patch for landing.

Comment 6 WebKit Commit Bot 2014-05-25 00:00:46 PDT

Comment on attachment 232030 [details]
Patch for landing.

Clearing flags on attachment: 232030

Committed r169317: <http://trac.webkit.org/changeset/169317>