Bug 120872

Summary: MediaStream API: Moving Add/Remove track from WebMediaStreamCenterClient to WebMediaStream
Product: WebKit Reporter: Eric Carlson <eric.carlson>
Component: WebCore Misc.Assignee: Eric Carlson <eric.carlson>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, dino, glenn, hta, jer.noble, rniwa, tommyw
Priority: P2 Keywords: BlinkMergeCandidate
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 121101    
Attachments:
Description Flags
Proposed patch
none
Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2 none

Description Eric Carlson 2013-09-06 11:45:58 PDT
Consider merging https://chromium.googlesource.com/chromium/blink/+/7d740fd27c1f345b59c846df01c3adce53c18e6e

Simple cleanup that makes the API cleaner.
Comment 1 Eric Carlson 2013-09-07 13:15:20 PDT
Created attachment 210918 [details]
Proposed patch
Comment 2 Build Bot 2013-09-07 15:18:34 PDT
Comment on attachment 210918 [details]
Proposed patch

Attachment 210918 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/1714648

New failing tests:
fast/workers/termination-with-port-messages.html
Comment 3 Build Bot 2013-09-07 15:18:35 PDT
Created attachment 210926 [details]
Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-13  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.4
Comment 4 Eric Carlson 2013-09-07 17:06:02 PDT
(In reply to comment #3)
> Created an attachment (id=210926) [details]
> Archive of layout-test-results from webkit-ews-13 for mac-mountainlion-wk2
> 
> The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
> Bot: webkit-ews-13  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.4

Unexpected flakiness: text-only failures (2)
  http/tests/inspector/inspect-element.html [ Failure Pass ]
  inspector/storage-panel-dom-storage-update.html [ Failure Pass ]


Regressions: Unexpected crashes (1)
  fast/workers/termination-with-port-messages.html [ Crash ]

These have nothing to do with this patch.
Comment 5 Jer Noble 2013-09-09 10:23:04 PDT
Comment on attachment 210918 [details]
Proposed patch

LGTM. r=me.
Comment 6 WebKit Commit Bot 2013-09-09 10:47:53 PDT
Comment on attachment 210918 [details]
Proposed patch

Clearing flags on attachment: 210918

Committed r155362: <http://trac.webkit.org/changeset/155362>
Comment 7 WebKit Commit Bot 2013-09-09 10:47:55 PDT
All reviewed patches have been landed.  Closing bug.