Make the uses of Resource::m_resourceRequest/m_response const if applicable

Make the uses of Resource::m_resourceRequest/m_response const if applicable

This CL
- Marks some Resource methods as const, and
- Replace |m_resourceRequest| and |m_response| with with
  resourceRequest() and response(), respectively, to make it clear
  that these references are const.

BUG=667641
Review-Url: https://codereview.chromium.org/2656443003
Cr-Commit-Position: refs/heads/master@{#445668}
Committed: https://chromium.googlesource.com/chromium/src/+/999a78abf761b51b7d02ee13c1cc3a1bc75b6db7

[hiroshige, 2017-01-24 02:03:19]

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

[commit-bot: I haz the power, 2017-01-24 02:04:02]

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

[hiroshige, 2017-01-24 02:17:15]

Description was changed from

==========
Mark Resource methods and references to m_resourceRequest/m_response as const

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

to

==========
Make uses of Resource::m_resourceRequest/m_response const if applicable

This CL
- Marks some Resource methods as const, and
- Replace |m_resourceRequest| and |m_response| with with
  resourceRequest() and response(), respectively, to make it clear
  that these references are const.

This is to make responseReceived() a single point of modification to
|m_response| (except for (encoded|decoded)(Body|Data)Length),
together with [1].

[1] https://codereview.chromium.org/2649853003/

BUG=667641
==========

[hiroshige, 2017-01-24 02:17:15]

hiroshige@chromium.org changed reviewers:
+ kouhei@chromium.org, yhirano@chromium.org

[commit-bot: I haz the power, 2017-01-24 04:24:38]

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

[commit-bot: I haz the power, 2017-01-24 04:24:39]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2017-01-24 05:07:00]

Description was changed from

==========
Make uses of Resource::m_resourceRequest/m_response const if applicable

This CL
- Marks some Resource methods as const, and
- Replace |m_resourceRequest| and |m_response| with with
  resourceRequest() and response(), respectively, to make it clear
  that these references are const.

This is to make responseReceived() a single point of modification to
|m_response| (except for (encoded|decoded)(Body|Data)Length),
together with [1].

[1] https://codereview.chromium.org/2649853003/

BUG=667641
==========

to

==========
Make the uses of Resource::m_resourceRequest/m_response const if applicable

This CL
- Marks some Resource methods as const, and
- Replace |m_resourceRequest| and |m_response| with with
  resourceRequest() and response(), respectively, to make it clear
  that these references are const.

BUG=667641
==========

[hiroshige, 2017-01-24 05:07:13]

hiroshige@chromium.org changed reviewers:
- kouhei@chromium.org

[hiroshige, 2017-01-24 05:07:23]

PTAL.

[yhirano, 2017-01-24 05:43:19]

lgtm

[hiroshige, 2017-01-24 06:47:38]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2017-01-24 06:47:55]

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-01-24 06:51:55]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485240458280260,
"parent_rev": "85f0afc642c1ce8bcfe452d24400f8d5c8281758", "commit_rev":
"999a78abf761b51b7d02ee13c1cc3a1bc75b6db7"}

[commit-bot: I haz the power, 2017-01-24 06:53:02]

Description was changed from

==========
Make the uses of Resource::m_resourceRequest/m_response const if applicable

This CL
- Marks some Resource methods as const, and
- Replace |m_resourceRequest| and |m_response| with with
  resourceRequest() and response(), respectively, to make it clear
  that these references are const.

BUG=667641
==========

to

==========
Make the uses of Resource::m_resourceRequest/m_response const if applicable

This CL
- Marks some Resource methods as const, and
- Replace |m_resourceRequest| and |m_response| with with
  resourceRequest() and response(), respectively, to make it clear
  that these references are const.

BUG=667641

Review-Url: https://codereview.chromium.org/2656443003
Cr-Commit-Position: refs/heads/master@{#445668}
Committed:
https://chromium.googlesource.com/chromium/src/+/999a78abf761b51b7d02ee13c1cc...
==========

[commit-bot: I haz the power, 2017-01-24 06:53:03]

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