Bug 169923 - [MediaStream] Make mock video source work on iOS
Summary: [MediaStream] Make mock video source work on iOS
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Carlson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-03-21 14:10 PDT by Eric Carlson
Modified: 2017-04-03 09:09 PDT (History)
5 users (show)

See Also:


Attachments
Proposed patch. (15.24 KB, patch)
2017-03-21 14:22 PDT, Eric Carlson
no flags Details | Formatted Diff | Diff
Patch for landing. (14.85 KB, patch)
2017-03-21 15:23 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 2017-03-21 14:10:27 PDT
Generate IOSurface backed CVPixelBuffers.
Comment 1 Eric Carlson 2017-03-21 14:22:42 PDT
Created attachment 305027 [details]
Proposed patch.
Comment 2 youenn fablet 2017-03-21 15:00:09 PDT
Comment on attachment 305027 [details]
Proposed patch.

View in context: https://bugs.webkit.org/attachment.cgi?id=305027&action=review

> Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.h:56
> +    RetainPtr<CVPixelBufferRef> pixelBufferFromCGImage(CGImageRef);

Keep it const and make m_bufferPool mutable?

> Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.h:60
> +    bool applySize(const IntSize&) override;

Use final.
Comment 3 Eric Carlson 2017-03-21 15:23:39 PDT
Created attachment 305035 [details]
Patch for landing.
Comment 4 WebKit Commit Bot 2017-03-22 10:46:04 PDT
Comment on attachment 305035 [details]
Patch for landing.

Rejecting attachment 305035 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 305035, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
fs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 214253 = da3f5293315f5782d6a2a6abfd0b26e35e202cbc
r214254 = 6679bded9e36e6fc6215a0654356302f85106dc7
r214255 = 420029cd551f8817a0ccd2a5c2c8e5c383d5ce99
r214256 = b7e4e1d9db541c7b59a415ff7275e27f5d6578aa
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: http://webkit-queues.webkit.org/results/3388345
Comment 5 Eric Carlson 2017-03-23 08:25:31 PDT
Comment on attachment 305035 [details]
Patch for landing.

Set cq+ again to see if the bots are feeling better.
Comment 6 Build Bot 2017-03-23 08:28:03 PDT
Attachment 305035 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/cocoa/CoreVideoSoftLink.cpp:42:  Missing space after ,  [whitespace/comma] [3]
ERROR: Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.mm:116:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.mm:120:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 3 in 7 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 7 WebKit Commit Bot 2017-03-23 08:54:52 PDT
Comment on attachment 305035 [details]
Patch for landing.

Clearing flags on attachment: 305035

Committed r214302: <http://trac.webkit.org/changeset/214302>
Comment 8 Matthew Hanson 2017-03-23 09:08:20 PDT
Bump (to import this into Radar)
Comment 9 Radar WebKit Bug Importer 2017-03-23 09:08:41 PDT
<rdar://problem/31220631>