Make Resource::setResponse() private

Make Resource::setResponse() private

This CL makes setResponse() private and make callers to
use responseReceived() instead.

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

The only non-test modification is in ImageDocumentParser::finish().
All other changes are unit-test-only.

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

BUG=667641

[hiroshige, 2017-01-24 02:04:54]

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:05:48]

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:19:51]

Description was changed from

==========
Make Resource::setResponse() private

In order to centralize modifications of |m_response| mostly to
Resource::responseReceived(), this CL makes setResponse() private and
make callers to use responseReceived() instead.

All other changes are unit-test-only.

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

to

==========
Make Resource::setResponse() private

This CL makes setResponse() private and make callers to
use responseReceived() instead.

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

The only non-test modification is in ImageDocumentParser::finish().
All other changes are unit-test-only.

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

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

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

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

Description was changed from

==========
Make Resource::setResponse() private

This CL makes setResponse() private and make callers to
use responseReceived() instead.

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

The only non-test modification is in ImageDocumentParser::finish().
All other changes are unit-test-only.

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

to

==========
Make Resource::setResponse() private

This CL makes setResponse() private and make callers to
use responseReceived() instead.

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

The only non-test modification is in ImageDocumentParser::finish().
All other changes are unit-test-only.

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

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

[commit-bot: I haz the power, 2017-01-24 03:25:11]

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

[commit-bot: I haz the power, 2017-01-24 03:25:11]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[hiroshige, 2017-01-24 04:15:46]

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

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

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 04:17:18]

hiroshige@chromium.org changed reviewers:
- japhet@chromium.org, kouhei@chromium.org, yhirano@chromium.org

[commit-bot: I haz the power, 2017-01-24 05:22:20]

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

[commit-bot: I haz the power, 2017-01-24 05:22:20]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)