Bug 177928 - Make RealtimeIncomingAudioSources and RealtimeOutgoingAudioSources port agnostic
Summary: Make RealtimeIncomingAudioSources and RealtimeOutgoingAudioSources port agnostic
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebRTC (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alejandro G. Castro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-10-05 03:42 PDT by Alejandro G. Castro
Modified: 2017-10-16 09:41 PDT (History)
3 users (show)

See Also:


Attachments
Patch (71.61 KB, patch)
2017-10-05 04:00 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.18 KB, patch)
2017-10-12 22:08 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.17 KB, patch)
2017-10-12 22:25 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.09 KB, patch)
2017-10-13 09:08 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.02 KB, patch)
2017-10-13 09:15 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.04 KB, patch)
2017-10-13 09:28 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.03 KB, patch)
2017-10-13 09:35 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.04 KB, patch)
2017-10-13 09:51 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.05 KB, patch)
2017-10-13 10:12 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.05 KB, patch)
2017-10-13 10:25 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (69.05 KB, patch)
2017-10-13 10:45 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (68.99 KB, patch)
2017-10-13 11:01 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (68.98 KB, patch)
2017-10-13 11:48 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch (68.98 KB, patch)
2017-10-13 13:47 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch for landing (69.00 KB, patch)
2017-10-16 07:16 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff
Patch for landing (69.12 KB, patch)
2017-10-16 09:01 PDT, Alejandro G. Castro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alejandro G. Castro 2017-10-05 03:42:28 PDT
We need this in the GTK port in order to use libwebrtc backend and endpoint classes.
Comment 1 Alejandro G. Castro 2017-10-05 04:00:56 PDT
Created attachment 322814 [details]
Patch
Comment 2 youenn fablet 2017-10-05 04:39:53 PDT
Comment on attachment 322814 [details]
Patch

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

LGTM, with the same comments as its sister patch.

> Source/WebCore/platform/mediastream/mac/RealtimeIncomingAudioSourceMac.cpp:70
> +    // FIXME: For the moment we skip them, we should find a better solution at libwebrtc level to not be called until getting some real data.

I would be interested to see whether GTK port will have the same issue here or not.
Maybe this issue is coming from the libwebrtc WebKit/Mac audio rendering integration.
Comment 3 Alejandro G. Castro 2017-10-12 22:08:49 PDT
Created attachment 323635 [details]
Patch
Comment 4 Alejandro G. Castro 2017-10-12 22:25:52 PDT
Created attachment 323638 [details]
Patch
Comment 5 Alejandro G. Castro 2017-10-13 09:08:14 PDT
Created attachment 323677 [details]
Patch
Comment 6 Alejandro G. Castro 2017-10-13 09:15:30 PDT
Created attachment 323679 [details]
Patch
Comment 7 Alejandro G. Castro 2017-10-13 09:28:26 PDT
Created attachment 323681 [details]
Patch
Comment 8 Alejandro G. Castro 2017-10-13 09:35:52 PDT
Created attachment 323684 [details]
Patch
Comment 9 Alejandro G. Castro 2017-10-13 09:51:46 PDT
Created attachment 323687 [details]
Patch
Comment 10 Alejandro G. Castro 2017-10-13 10:12:35 PDT
Created attachment 323689 [details]
Patch
Comment 11 Alejandro G. Castro 2017-10-13 10:25:09 PDT
Created attachment 323694 [details]
Patch
Comment 12 Alejandro G. Castro 2017-10-13 10:45:19 PDT
Created attachment 323699 [details]
Patch
Comment 13 Alejandro G. Castro 2017-10-13 11:01:11 PDT
Created attachment 323706 [details]
Patch
Comment 14 Alejandro G. Castro 2017-10-13 11:48:47 PDT
Created attachment 323715 [details]
Patch
Comment 15 Alejandro G. Castro 2017-10-13 13:47:46 PDT
Created attachment 323739 [details]
Patch
Comment 16 youenn fablet 2017-10-13 16:28:33 PDT
Comment on attachment 323739 [details]
Patch

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

> Source/WebCore/platform/mediastream/mac/RealtimeIncomingAudioSourceCocoa.cpp:61
> +

Double line
Comment 17 Alejandro G. Castro 2017-10-16 07:16:13 PDT
Created attachment 323892 [details]
Patch for landing
Comment 18 WebKit Commit Bot 2017-10-16 07:56:03 PDT
Comment on attachment 323892 [details]
Patch for landing

Rejecting attachment 323892 [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-02', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 323892, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
.webkit.org/git/WebKit
   b9f4d12..9032476  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 223389 = b9f4d12279749dd6113b170a020ca3bd5ea37fe0
r223406 = 9032476984cacdd3aaf521d9babf7105e2c5f8c3
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/4871101
Comment 19 Alejandro G. Castro 2017-10-16 09:01:05 PDT
Created attachment 323898 [details]
Patch for landing
Comment 20 WebKit Commit Bot 2017-10-16 09:40:51 PDT
Comment on attachment 323898 [details]
Patch for landing

Clearing flags on attachment: 323898

Committed r223412: <https://trac.webkit.org/changeset/223412>
Comment 21 WebKit Commit Bot 2017-10-16 09:40:53 PDT
All reviewed patches have been landed.  Closing bug.
Comment 22 Radar WebKit Bug Importer 2017-10-16 09:41:49 PDT
<rdar://problem/35007894>