ResourceHandler::OnWillStart: Remove min_size argument.

ResourceHandler::OnWillStart: Remove min_size argument.

Nothing sets it to anything but -1, even MimeTypeResourceHandler, the
one class that really wants a buffer of at least a certain size.  The
major ResourceHandler subclasses just DCHECK it's -1, so it serves no
purpose.

BUG=none

Review-Url: https://codereview.chromium.org/2660723002
Cr-Commit-Position: refs/heads/master@{#447791}
Committed: https://chromium.googlesource.com/chromium/src/+/7551f612ef47cc7d5eaf27771728cde7e0b51f26

[mmenke, 2017-01-27 21:59:33]

The CQ bit was checked by mmenke@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-27 22:00:35]

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

[mmenke, 2017-01-27 22:02:16]

mmenke@chromium.org changed reviewers:
+ csharrison@chromium.org

[mmenke, 2017-01-27 22:02:17]

I'm having trouble coming up with a comment adequate to the silliness of this
CL.

[mmenke, 2017-01-27 22:03:02]

On 2017/01/27 22:02:17, mmenke (Out Feb 4 to March 5) wrote:
> I'm having trouble coming up with a comment adequate to the silliness of this
> CL.

Worth noting this is on top of the huge CL, because I didn't feel like rebasing
that one on top of this one.

[Charlie Harrison, 2017-01-27 22:06:09]

LOL-worthy LGTM

[commit-bot: I haz the power, 2017-01-28 01:15:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-28 01:15:47]

Dry run: This issue passed the CQ dry run.

[mmenke, 2017-01-30 03:23:33]

mmenke@chromium.org changed reviewers:
+ asanka@chromium.org

[mmenke, 2017-01-30 03:23:33]

[+asanka]:  Please review content/browser/download.

[asanka, 2017-01-30 21:37:33]

lgtm

[mmenke, 2017-02-02 16:32:02]

The CQ bit was checked by mmenke@chromium.org

[mmenke, 2017-02-02 16:32:03]

The patchset sent to the CQ was uploaded after l-g-t-m from asanka@chromium.org,
csharrison@chromium.org
Link to the patchset: https://codereview.chromium.org/2660723002/#ps40001
(title: "Merge")

[commit-bot: I haz the power, 2017-02-02 16:32:30]

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

[commit-bot: I haz the power, 2017-02-02 17:40:10]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486053122889620,
"parent_rev": "f6a5c39ae29d703bb4ecc6663477de3f9d3284a6", "commit_rev":
"7551f612ef47cc7d5eaf27771728cde7e0b51f26"}

[commit-bot: I haz the power, 2017-02-02 17:40:47]

Description was changed from

==========
ResourceHandler::OnWillStart: Remove min_size argument.

Nothing sets it to anything but -1, even MimeTypeResourceHandler, the
one class that really wants a buffer of at least a certain size.  The
major ResourceHandler subclasses just DCHECK it's -1, so it serves no
purpose.

BUG=none
==========

to

==========
ResourceHandler::OnWillStart: Remove min_size argument.

Nothing sets it to anything but -1, even MimeTypeResourceHandler, the
one class that really wants a buffer of at least a certain size.  The
major ResourceHandler subclasses just DCHECK it's -1, so it serves no
purpose.

BUG=none

Review-Url: https://codereview.chromium.org/2660723002
Cr-Commit-Position: refs/heads/master@{#447791}
Committed:
https://chromium.googlesource.com/chromium/src/+/7551f612ef47cc7d5eaf27771728...
==========

[commit-bot: I haz the power, 2017-02-02 17:40:49]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/7551f612ef47cc7d5eaf27771728...