Bug 155663

Summary: Add a WebRTC specific compile flag
Product: WebKit Reporter: Eric Carlson <eric.carlson>
Component: MediaAssignee: Eric Carlson <eric.carlson>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, jer.noble, rniwa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch for the bots.
buildbot: commit-queue-
Archive of layout-test-results from ews101 for mac-yosemite
none
Archive of layout-test-results from ews105 for mac-yosemite-wk2
none
Archive of layout-test-results from ews114 for mac-yosemite
none
Updated patch.
buildbot: commit-queue-
Archive of layout-test-results from ews104 for mac-yosemite-wk2
none
Archive of layout-test-results from ews102 for mac-yosemite
none
Archive of layout-test-results from ews114 for mac-yosemite
none
Updated patch.
none
Another updated patch
buildbot: commit-queue-
Archive of layout-test-results from ews106 for mac-yosemite-wk2
none
Another updated patch none

Description Eric Carlson 2016-03-18 14:37:50 PDT
Define a new WebRTC-specific compile flag since Media Streams is a stand-alone feature.
Comment 1 Eric Carlson 2016-03-18 14:45:51 PDT
Created attachment 274454 [details]
Patch for the bots.
Comment 2 WebKit Commit Bot 2016-03-18 14:47:39 PDT
Attachment 274454 [details] did not pass style-queue:


ERROR: Source/WebCore/rendering/RenderLayerBacking.cpp:2293:  Should have a space between // and comment  [whitespace/comments] [4]
Total errors found: 1 in 113 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Build Bot 2016-03-18 15:40:43 PDT
Comment on attachment 274454 [details]
Patch for the bots.

Attachment 274454 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/1001612

New failing tests:
js/dom/global-constructors-attributes.html
Comment 4 Build Bot 2016-03-18 15:40:45 PDT
Created attachment 274460 [details]
Archive of layout-test-results from ews101 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 5 Build Bot 2016-03-18 15:44:39 PDT
Comment on attachment 274454 [details]
Patch for the bots.

Attachment 274454 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/1001624

New failing tests:
fast/mediastream/RTCPeerConnection-overloaded-operations.html
fast/mediastream/RTCSessionDescription.html
js/dom/global-constructors-attributes.html
fast/mediastream/RTCPeerConnection-closed-state.html
fast/mediastream/RTCRtpSender-replaceTrack.html
fast/mediastream/RTCPeerConnection-add-removeTrack.html
fast/mediastream/RTCPeerConnection-overloaded-operations-params.html
Comment 6 Build Bot 2016-03-18 15:44:41 PDT
Created attachment 274462 [details]
Archive of layout-test-results from ews105 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 7 Build Bot 2016-03-18 16:02:11 PDT
Comment on attachment 274454 [details]
Patch for the bots.

Attachment 274454 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/1001628

New failing tests:
js/dom/global-constructors-attributes.html
Comment 8 Build Bot 2016-03-18 16:02:14 PDT
Created attachment 274466 [details]
Archive of layout-test-results from ews114 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 9 Eric Carlson 2016-03-18 16:12:12 PDT
Created attachment 274469 [details]
Updated patch.
Comment 10 Build Bot 2016-03-18 17:05:12 PDT
Comment on attachment 274469 [details]
Updated patch.

Attachment 274469 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/1001953

New failing tests:
fast/mediastream/RTCPeerConnection-overloaded-operations.html
fast/mediastream/RTCSessionDescription.html
js/dom/global-constructors-attributes.html
fast/mediastream/RTCPeerConnection-closed-state.html
fast/mediastream/RTCRtpSender-replaceTrack.html
fast/mediastream/RTCPeerConnection-add-removeTrack.html
fast/mediastream/RTCPeerConnection-overloaded-operations-params.html
Comment 11 Build Bot 2016-03-18 17:05:15 PDT
Created attachment 274481 [details]
Archive of layout-test-results from ews104 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 12 Build Bot 2016-03-18 17:07:32 PDT
Comment on attachment 274469 [details]
Updated patch.

Attachment 274469 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/1001968

New failing tests:
js/dom/global-constructors-attributes.html
Comment 13 Build Bot 2016-03-18 17:07:34 PDT
Created attachment 274482 [details]
Archive of layout-test-results from ews102 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 14 Build Bot 2016-03-18 17:22:29 PDT
Comment on attachment 274469 [details]
Updated patch.

Attachment 274469 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/1001989

New failing tests:
js/dom/global-constructors-attributes.html
Comment 15 Build Bot 2016-03-18 17:22:31 PDT
Created attachment 274484 [details]
Archive of layout-test-results from ews114 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 16 Eric Carlson 2016-03-19 09:30:40 PDT
Created attachment 274517 [details]
Updated patch.
Comment 17 WebKit Commit Bot 2016-03-19 09:32:31 PDT
Attachment 274517 [details] did not pass style-queue:


ERROR: Source/WebCore/rendering/RenderLayerBacking.cpp:2293:  Should have a space between // and comment  [whitespace/comments] [4]
Total errors found: 1 in 116 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 18 Eric Carlson 2016-03-19 10:05:34 PDT
Created attachment 274518 [details]
Another updated patch
Comment 19 Build Bot 2016-03-19 11:03:47 PDT
Comment on attachment 274518 [details]
Another updated patch

Attachment 274518 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/1006051

New failing tests:
fast/mediastream/RTCPeerConnection-overloaded-operations.html
fast/mediastream/RTCSessionDescription.html
fast/mediastream/RTCPeerConnection-closed-state.html
fast/mediastream/RTCRtpSender-replaceTrack.html
fast/mediastream/RTCPeerConnection-add-removeTrack.html
fast/mediastream/RTCPeerConnection-overloaded-operations-params.html
Comment 20 Build Bot 2016-03-19 11:03:51 PDT
Created attachment 274519 [details]
Archive of layout-test-results from ews106 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 21 Eric Carlson 2016-03-20 17:29:43 PDT
Created attachment 274552 [details]
Another updated patch
Comment 22 Eric Carlson 2016-03-21 10:11:24 PDT
<rdar://problem/25116653>
Comment 23 WebKit Commit Bot 2016-03-21 10:52:33 PDT
Comment on attachment 274552 [details]
Another updated patch

Clearing flags on attachment: 274552

Committed r198492: <http://trac.webkit.org/changeset/198492>
Comment 24 WebKit Commit Bot 2016-03-21 10:52:38 PDT
All reviewed patches have been landed.  Closing bug.