Bug 159365

Summary: REGRESSION (r202641): Netflix playback stalls after a few seconds
Product: WebKit Reporter: Jer Noble <jer.noble>
Component: MediaAssignee: Jer Noble <jer.noble>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, rniwa
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews102 for mac-yosemite
none
Archive of layout-test-results from ews106 for mac-yosemite-wk2
none
Archive of layout-test-results from ews116 for mac-yosemite
none
Patch
none
Patch eric.carlson: review+

Description Jer Noble 2016-07-01 13:21:59 PDT
REGRESSION (r202641): Netflix playback stalls after a few seconds
Comment 1 Jer Noble 2016-07-01 15:21:34 PDT
Created attachment 282591 [details]
Patch
Comment 2 Build Bot 2016-07-01 16:16:40 PDT
Comment on attachment 282591 [details]
Patch

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

New failing tests:
http/tests/media/media-source/mediasource-play.html
http/tests/media/media-source/mediasource-remove.html
media/media-source/media-source-remove.html
Comment 3 Build Bot 2016-07-01 16:16:42 PDT
Created attachment 282602 [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 4 Eric Carlson 2016-07-01 16:25:17 PDT
Comment on attachment 282591 [details]
Patch

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

> Source/WebCore/Modules/mediasource/SourceBuffer.cpp:666
> +    PlatformTimeRanges erasedRanges;

The elf bot's WTFMove macro doesn't like you returning this.

> Source/WebCore/Modules/mediasource/SourceBuffer.cpp:1617
> +        // disjoint ranges separated by less than a "fudge factor"

Nit: no period at the end of the sentence.
Comment 5 Build Bot 2016-07-01 16:25:46 PDT
Comment on attachment 282591 [details]
Patch

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

New failing tests:
http/tests/media/media-source/mediasource-play.html
http/tests/media/media-source/mediasource-remove.html
media/media-source/media-source-remove.html
Comment 6 Build Bot 2016-07-01 16:25:49 PDT
Created attachment 282605 [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 7 Build Bot 2016-07-01 16:29:39 PDT
Comment on attachment 282591 [details]
Patch

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

New failing tests:
http/tests/media/media-source/mediasource-play.html
http/tests/media/media-source/mediasource-remove.html
media/media-source/media-source-remove.html
Comment 8 Build Bot 2016-07-01 16:29:42 PDT
Created attachment 282606 [details]
Archive of layout-test-results from ews116 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 9 Jer Noble 2016-07-01 17:48:44 PDT
Created attachment 282619 [details]
Patch
Comment 10 Jer Noble 2016-07-05 12:47:22 PDT
Created attachment 282807 [details]
Patch
Comment 11 Eric Carlson 2016-07-05 13:19:56 PDT
Comment on attachment 282807 [details]
Patch

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

> Source/WebCore/Modules/mediasource/SourceBuffer.cpp:1617
> +        // Eliminate small gaps between buffered ranges by coalescing
> +        // disjoint ranges separated by less than a "fudge factor"

Nit: this sentence needs a period.
Comment 12 Jer Noble 2016-07-05 13:23:37 PDT
Committed r202829: <http://trac.webkit.org/changeset/202829>