GitLab

As part of the effort to bring the new Network Service online, we are
deprecating URLFetcher in favor of SimpleURLLoader.

Background: https://docs.google.com/document/d/1YZDPeg3bf46QPU_vUotFwOEPHquww36b-UdRlO3ZpMU/edit
Previous CL: https://chromium-review.googlesource.com/c/chromium/src/+/1070110
Reverted: https://chromium-review.googlesource.com/c/chromium/src/+/1073656

Refactoring exposed an underlying bug in the existing code (max_retries_on_5xx_ could
be used before being initialized). This CL initializes it to 0 in the constructor.

Bug: 844937
Change-Id: I419565dc899b2b48118b351206ade4f15090836a
TBR: groby@chromium.org, charleszhao@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1076407
Commit-Queue: Mark Pilgrim <pilgrim@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Andrew Moylan <amoylan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#562806}