Rename ResourceResponse::addToEncodedDataSize() to setEncodedDataSize()

Rename ResourceResponse::addToEncodedDataSize() to setEncodedDataSize()

ResourceResponse::addToEncodedDataSize is used only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
Committed: https://crrev.com/e0d49be76df5f71f17cf8aaf77772f386e1edae3
Cr-Commit-Position: refs/heads/master@{#433742}

[yhirano, 2016-11-21 07:43:55]

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

[commit-bot: I haz the power, 2016-11-21 07:44:04]

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

[yhirano, 2016-11-21 07:45:28]

Description was changed from

==========
Add ResourceResponse::setEncodedDataSize

... and retire ResourceResponse::addToEncodedDataSize.

ResourceResponse::addToEncodedDataSize is unsed only on a [Web]ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

to

==========
Add ResourceResponse::setEncodedDataSize

... and retire ResourceResponse::addToEncodedDataSize.

ResourceResponse::addToEncodedDataSize is unsed only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

[yhirano, 2016-11-21 11:38:58]

Description was changed from

==========
Add ResourceResponse::setEncodedDataSize

... and retire ResourceResponse::addToEncodedDataSize.

ResourceResponse::addToEncodedDataSize is unsed only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

to

==========
Add ResourceResponse::setEncodedDataSize

... and retire ResourceResponse::addToEncodedDataSize.

ResourceResponse::addToEncodedDataSize is used only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

[commit-bot: I haz the power, 2016-11-21 11:45:44]

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

[commit-bot: I haz the power, 2016-11-21 11:45:46]

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

[yhirano, 2016-11-21 11:45:53]

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

[hiroshige, 2016-11-21 11:57:44]

lgtm.

CL title:
How about "Rename addToEncodedDataSize() to setEncodedDataSize()"?

[yhirano, 2016-11-21 12:00:20]

Description was changed from

==========
Add ResourceResponse::setEncodedDataSize

... and retire ResourceResponse::addToEncodedDataSize.

ResourceResponse::addToEncodedDataSize is used only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

to

==========
Rename ResourceResponse::addToEncodedDataSize() to setEncodedDataSize()

ResourceResponse::addToEncodedDataSize is used only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

[yhirano, 2016-11-21 12:00:38]

On 2016/11/21 11:57:44, hiroshige wrote:
> lgtm.
> 
> CL title:
> How about "Rename addToEncodedDataSize() to setEncodedDataSize()"?

Done.

[yhirano, 2016-11-21 12:00:56]

yhirano@chromium.org changed reviewers:
+ jochen@chromium.org

[yhirano, 2016-11-21 12:00:56]

+jochen@ for OWNER review.

[jochen (gone - plz use gerrit), 2016-11-21 12:02:56]

lgtm

[yhirano, 2016-11-21 12:10:15]

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

[yhirano, 2016-11-21 12:10:29]

The CQ bit was unchecked by yhirano@chromium.org

[yhirano, 2016-11-21 12:10:29]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2016-11-21 12:10:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
hiroshige@chromium.org, jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2521503002/#ps20001
(title: "done")

[commit-bot: I haz the power, 2016-11-21 12:10:45]

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

[yhirano, 2016-11-21 12:20:13]

The CQ bit was unchecked by yhirano@chromium.org

[yhirano, 2016-11-21 12:20:33]

Patchset #2 (id:20001) has been deleted

[yhirano, 2016-11-21 12:22:39]

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

[commit-bot: I haz the power, 2016-11-21 12:22:52]

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

[yhirano, 2016-11-21 12:23:13]

The CQ bit was unchecked by yhirano@chromium.org

[yhirano, 2016-11-21 12:23:13]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2016-11-21 12:23:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
hiroshige@chromium.org, jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2521503002/#ps40001
(title: "fix")

[commit-bot: I haz the power, 2016-11-21 12:23:20]

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-11-21 16:24:24]

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

[commit-bot: I haz the power, 2016-11-21 16:24:26]

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

[yhirano, 2016-11-22 01:31:08]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2016-11-22 01:31: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, 2016-11-22 02:09:13]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1479778268697150,
"parent_rev": "708b174491c869a7085c0909cd1c66af85cc5db1", "commit_rev":
"2ab2c51e08feb255f32b72080682857410fbe7bd"}

[commit-bot: I haz the power, 2016-11-22 02:09:45]

Description was changed from

==========
Rename ResourceResponse::addToEncodedDataSize() to setEncodedDataSize()

ResourceResponse::addToEncodedDataSize is used only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

to

==========
Rename ResourceResponse::addToEncodedDataSize() to setEncodedDataSize()

ResourceResponse::addToEncodedDataSize is used only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

[commit-bot: I haz the power, 2016-11-22 02:09:47]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-11-22 02:16:24]

Description was changed from

==========
Rename ResourceResponse::addToEncodedDataSize() to setEncodedDataSize()

ResourceResponse::addToEncodedDataSize is used only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216
==========

to

==========
Rename ResourceResponse::addToEncodedDataSize() to setEncodedDataSize()

ResourceResponse::addToEncodedDataSize is used only on a ResourceResponse
whose encodedDataLength is 0. I'm planning to update the value at the response
completion timing and setEncodedDataSize is easier to use / understand.

BUG=666216

Committed: https://crrev.com/e0d49be76df5f71f17cf8aaf77772f386e1edae3
Cr-Commit-Position: refs/heads/master@{#433742}
==========

[commit-bot: I haz the power, 2016-11-22 02:16:25]

Patchset 2 (id:??) landed as
https://crrev.com/e0d49be76df5f71f17cf8aaf77772f386e1edae3
Cr-Commit-Position: refs/heads/master@{#433742}