Remove FetchRequest::mutableResourceRequest() where removing is trivial

Remove FetchRequest::mutableResourceRequest() where removing is trivial

This CL removes FetchRequest::mutableResourceRequest() calls that are
used to modify ResourceRequest created just before, i.e. replaces:
  FetchRequest request(ResourceRequest(foo), bar);
  request.mutableResourceRequest().baz();
with:
  ResourceRequest resourceRequest(foo);
  resourceRequest.baz();
  FetchRequest request(resourceRequest, bar);
To clarify what kind of modifications to ResourceRequest via
mutableResourceRequest() is needed (https://crbug.com/632580#c8).

BUG=632580
Review-Url: https://codereview.chromium.org/2740253002
Cr-Commit-Position: refs/heads/master@{#456169}
Committed: https://chromium.googlesource.com/chromium/src/+/a51669425d5a771fb9f80663779135c1deffdb00

[hiroshige, 2017-03-10 01:35:53]

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

[commit-bot: I haz the power, 2017-03-10 01:36:42]

Dry run: 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-03-10 02:58:17]

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

[commit-bot: I haz the power, 2017-03-10 02:58:18]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2017-03-10 19:51:16]

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

[commit-bot: I haz the power, 2017-03-10 19:51:46]

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

[hiroshige, 2017-03-10 20:02:51]

Description was changed from

==========
Remove FetchRequest::mutableResourceRequest() where removing is trivial

BUG=
==========

to

==========
Remove FetchRequest::mutableResourceRequest() where removing is trivial

This CL removes FetchRequest::mutableResourceRequest() calls that are
used to modify ResourceRequest created just before, i.e. replaces:
  FetchRequest request(ResourceRequest(foo), bar);
  request.mutableResourceRequest().baz();
with:
  ResourceRequest resourceRequest(foo);
  resourceRequest.baz();
  FetchRequest request(resourceRequest, bar);
To clarify what kind of modifications to ResourceRequest via
mutableResourceRequest() is needed (https://crbug.com/632580#c8).

BUG=632580
==========

[hiroshige, 2017-03-10 20:03:20]

hiroshige@chromium.org changed reviewers:
+ japhet@chromium.org

[hiroshige, 2017-03-10 20:03:21]

PTAL.

[Nate Chapin, 2017-03-10 20:04:41]

lgtm

[hiroshige, 2017-03-10 20:55:10]

The CQ bit was unchecked by hiroshige@chromium.org

[hiroshige, 2017-03-10 20:55:13]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2017-03-10 20:55:33]

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-03-10 21:35:23]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489179313364010,
"parent_rev": "c5f3fd4c29ff7ba57c39498f33236a8d8efc263f", "commit_rev":
"a51669425d5a771fb9f80663779135c1deffdb00"}

[commit-bot: I haz the power, 2017-03-10 21:36:07]

Description was changed from

==========
Remove FetchRequest::mutableResourceRequest() where removing is trivial

This CL removes FetchRequest::mutableResourceRequest() calls that are
used to modify ResourceRequest created just before, i.e. replaces:
  FetchRequest request(ResourceRequest(foo), bar);
  request.mutableResourceRequest().baz();
with:
  ResourceRequest resourceRequest(foo);
  resourceRequest.baz();
  FetchRequest request(resourceRequest, bar);
To clarify what kind of modifications to ResourceRequest via
mutableResourceRequest() is needed (https://crbug.com/632580#c8).

BUG=632580
==========

to

==========
Remove FetchRequest::mutableResourceRequest() where removing is trivial

This CL removes FetchRequest::mutableResourceRequest() calls that are
used to modify ResourceRequest created just before, i.e. replaces:
  FetchRequest request(ResourceRequest(foo), bar);
  request.mutableResourceRequest().baz();
with:
  ResourceRequest resourceRequest(foo);
  resourceRequest.baz();
  FetchRequest request(resourceRequest, bar);
To clarify what kind of modifications to ResourceRequest via
mutableResourceRequest() is needed (https://crbug.com/632580#c8).

BUG=632580

Review-Url: https://codereview.chromium.org/2740253002
Cr-Commit-Position: refs/heads/master@{#456169}
Committed:
https://chromium.googlesource.com/chromium/src/+/a51669425d5a771fb9f806637791...
==========

[commit-bot: I haz the power, 2017-03-10 21:36:10]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/a51669425d5a771fb9f806637791...