Bug 100160

Summary: Remove Chromium specific expectations for video-media-source-add-and-remove-buffers.html
Product: WebKit Reporter: Aaron Colwell <acolwell>
Component: MediaAssignee: Aaron Colwell <acolwell>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric.carlson, feature-media-reviews, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Aaron Colwell
Reported 2012-10-23 14:38:37 PDT
There was an error in the original expectations for http/tests/media/media-source/video-media-source-add-and-remove-buffers.html. When http://crbug.com/155226 was fixed it caused the LayoutTest to started failing because the existing expectations were just capturing the old incorrect behavior. The Chromium WebKit gardener created Chromium specific expectations to compensate for the failed test (http://trac.webkit.org/changeset/131086). The attached patch will remove the Chromium specific expectations and update the common expectations to reflect the correct behavior.
Attachments
Patch (3.95 KB, patch)
2012-10-23 14:44 PDT, Aaron Colwell 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Aaron Colwell
Comment 1 2012-10-23 14:44:23 PDT
Created attachment 170238 [details] Patch
Aaron Colwell
Comment 2 2012-10-26 12:32:48 PDT
abarth@ can I get a quick review please.
Adam Barth
Comment 3 2012-10-26 14:16:17 PDT
Comment on attachment 170238 [details] Patch Great!
WebKit Review Bot
Comment 4 2012-10-26 14:26:07 PDT
Comment on attachment 170238 [details] Patch Clearing flags on attachment: 170238 Committed r132694: <http://trac.webkit.org/changeset/132694>
WebKit Review Bot
Comment 5 2012-10-26 14:26:10 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.