-
geofflang authored
Revert of Fix MultibufferDataSource::GetSize (patchset #1 id:1 of https://codereview.chromium.org/2110853006/ ) Reason for revert: I believe this CL is causing flakyness in the video related tests on the GPU FYI waterfall. Examples: https://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20%28NVIDIA%29/builds/25925 https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28ATI%29/builds/48492 https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20Retina%20Release/builds/5019 Seeing this in the output log: [27562:1287:0630/083352:WARNING:webmediaplayer_impl.cc(335)] Using MultibufferDataSource [27562:34423:0630/083352:ERROR:ffmpeg_demuxer.cc(1510)] OnReadFrameDone result=-541478725 IsMaxMemoryUsageReached=0 [27562:1287:0630/083352:WARNING:webmediaplayer_impl.cc(335)] Using MultibufferDataSource [27562:1287:0630/083352:FATAL:resource_multibuffer_data_provider.cc(75)] Check failed: byte_pos() < url_data_->length() (32768 vs. 10292) http://127.0.0.1:53903/resources/red-green.theora.ogv Original issue's description: > Fix MultibufferDataSource::GetSize > > When file loading is finished ResourceMultiBufferDataProvider count > DataBuffer queue size twice, so GetSize returned incorrect value. > > BUG= > > Committed: https://crrev.com/bbaba3c8053a39f6d11d78a6e4770b37a9639067 > Cr-Commit-Position: refs/heads/master@{#403123} TBR=hubbe@chromium.org,kostya-k@yandex-team.ru # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=624853 Review-Url: https://codereview.chromium.org/2115483003 Cr-Commit-Position: refs/heads/master@{#403211}
b2271028
