Fix memory usage bug.

Fix memory usage bug.

Don't tell multibuffer that it's ok to use 25Mb if we don't need it. Instead, use the bitrate to figure out what the max buffer should be (but clamp it to 25Mb).

BUG=595827
Committed: https://crrev.com/c3694d28aca6e32401771f6e9c47bb47a457fd3e
Cr-Commit-Position: refs/heads/master@{#385963}

[hubbe, 2016-04-07 23:23:04]

hubbe@chromium.org changed reviewers:
+ dalecurtis@google.com

[hubbe, 2016-04-07 23:36:54]

Description was changed from

==========
Fix memory usage bug.

This fixes a bug where preloading uses the whole 25Mb buffer instead of 10
seconds of video.

BUG=595827
==========

to

==========
Fix memory usage bug.

Don't tell multibuffer that it's ok to use 25Mb if we don't need it. Instead,
use the bitrate to figure out what the max buffer should be (but clamp it to
25Mb).

BUG=595827
==========

[DaleCurtis, 2016-04-07 23:56:19]

dalecurtis@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[DaleCurtis, 2016-04-07 23:56:19]

lgtm

[DaleCurtis, 2016-04-07 23:56:28]

dalecurtis@chromium.org changed reviewers:
- dalecurtis@google.com

[hubbe, 2016-04-08 00:10:28]

The CQ bit was checked by hubbe@chromium.org

[commit-bot: I haz the power, 2016-04-08 00:10:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1869203003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1869203003/1

[commit-bot: I haz the power, 2016-04-08 03:07:29]

Description was changed from

==========
Fix memory usage bug.

Don't tell multibuffer that it's ok to use 25Mb if we don't need it. Instead,
use the bitrate to figure out what the max buffer should be (but clamp it to
25Mb).

BUG=595827
==========

to

==========
Fix memory usage bug.

Don't tell multibuffer that it's ok to use 25Mb if we don't need it. Instead,
use the bitrate to figure out what the max buffer should be (but clamp it to
25Mb).

BUG=595827
==========

[commit-bot: I haz the power, 2016-04-08 03:07:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-08 03:08:55]

Description was changed from

==========
Fix memory usage bug.

Don't tell multibuffer that it's ok to use 25Mb if we don't need it. Instead,
use the bitrate to figure out what the max buffer should be (but clamp it to
25Mb).

BUG=595827
==========

to

==========
Fix memory usage bug.

Don't tell multibuffer that it's ok to use 25Mb if we don't need it. Instead,
use the bitrate to figure out what the max buffer should be (but clamp it to
25Mb).

BUG=595827

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

[commit-bot: I haz the power, 2016-04-08 03:08:55]

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