Make Resource::m_response private

Make Resource::m_response private

This CL makes |Resource::m_response| private and replaces the usage of
this with |response()| calls.

There was only one place where |m_response| was updated in the
subclasses of Resource. This was in
|ImageResource::onePartInMultipartReceived| and replaced with
|setResponse()|. |setResponse()| updates not only |m_response| but
also |m_cacheHandler|. Calling |setReponse()| is more appropriate
for caching of a response which is multipart and fetched from
ServiceWorker.

BUG=n/a
TEST=webkit_unit_tests --gtest_filter=ImageResourceTest.*

Committed: https://crrev.com/5654f67165722ecac55bf646fd3821fc4f354b5c
Cr-Commit-Position: refs/heads/master@{#408399}

[hajimehoshi, 2016-07-28 10:26:58]

hajimehoshi@chromium.org changed reviewers:
+ hiroshige@chromium.org, mkwst@chromium.org

[hajimehoshi, 2016-07-28 10:26:59]

PTAL

[Mike West, 2016-07-28 10:55:13]

LGTM if the bots are happy.

[hajimehoshi, 2016-07-28 11:31:27]

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

[commit-bot: I haz the power, 2016-07-28 11:31:38]

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, 2016-07-28 12:46:39]

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

[commit-bot: I haz the power, 2016-07-28 12:46:40]

Dry run: This issue passed the CQ dry run.

[hajimehoshi, 2016-07-28 15:34:04]

The CQ bit was checked by hajimehoshi@chromium.org

[commit-bot: I haz the power, 2016-07-28 15:35:04]

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, 2016-07-28 15:41:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-28 15:44:06]

Description was changed from

==========
Make Resource::m_response private

This CL makes |Resource::m_response| private and replaces the usage of
this with |response()| calls.

There was only one place where |m_response| was updated in the
subclasses of Resource. This was in
|ImageResource::onePartInMultipartReceived| and replaced with
|setResponse()|. |setResponse()| updates not only |m_response| but
also |m_cacheHandler|. Calling |setReponse()| is more appropriate
for caching of a response which is multipart and fetched from
ServiceWorker.

BUG=n/a
TEST=webkit_unit_tests --gtest_filter=ImageResourceTest.*
==========

to

==========
Make Resource::m_response private

This CL makes |Resource::m_response| private and replaces the usage of
this with |response()| calls.

There was only one place where |m_response| was updated in the
subclasses of Resource. This was in
|ImageResource::onePartInMultipartReceived| and replaced with
|setResponse()|. |setResponse()| updates not only |m_response| but
also |m_cacheHandler|. Calling |setReponse()| is more appropriate
for caching of a response which is multipart and fetched from
ServiceWorker.

BUG=n/a
TEST=webkit_unit_tests --gtest_filter=ImageResourceTest.*

Committed: https://crrev.com/5654f67165722ecac55bf646fd3821fc4f354b5c
Cr-Commit-Position: refs/heads/master@{#408399}
==========

[commit-bot: I haz the power, 2016-07-28 15:44:07]

Patchset 1 (id:??) landed as
https://crrev.com/5654f67165722ecac55bf646fd3821fc4f354b5c
Cr-Commit-Position: refs/heads/master@{#408399}