Revert of Fix MultibufferDataSource::GetSize

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
Committed: https://crrev.com/b2271028e1934c5b9c607b75e31b27b195c18c6e
Cr-Commit-Position: refs/heads/master@{#403211}

[Geoff Lang, 2016-06-30 17:40:24]

The CQ bit was checked by geofflang@chromium.org

[Geoff Lang, 2016-06-30 17:40:24]

Created Revert of Fix MultibufferDataSource::GetSize

[commit-bot: I haz the power, 2016-06-30 17:40:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Geoff Lang, 2016-06-30 17:40:55]

Description was changed from

==========
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%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28A...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20Retina%20Release...

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=
==========

to

==========
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%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28A...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20Retina%20Release...

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
==========

[commit-bot: I haz the power, 2016-06-30 17:42:25]

Description was changed from

==========
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%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28A...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20Retina%20Release...

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
==========

to

==========
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%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28A...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20Retina%20Release...

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
==========

[commit-bot: I haz the power, 2016-06-30 17:42:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-30 17:42:37]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 17:45:26]

Description was changed from

==========
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%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28A...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20Retina%20Release...

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
==========

to

==========
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%28NV...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Linux%20Release%20%28A...
https://build.chromium.org/p/chromium.gpu.fyi/builders/Mac%20Retina%20Release...

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

Committed: https://crrev.com/b2271028e1934c5b9c607b75e31b27b195c18c6e
Cr-Commit-Position: refs/heads/master@{#403211}
==========

[commit-bot: I haz the power, 2016-06-30 17:45:27]

Patchset 1 (id:??) landed as
https://crrev.com/b2271028e1934c5b9c607b75e31b27b195c18c6e
Cr-Commit-Position: refs/heads/master@{#403211}