net: add IsMethodSafe() method to HttpUtil

net: add IsMethodSafe() method to HttpUtil

This consolidates the three copies of it in a single and accessible place.

BUG=675760
TEST=net_unittests
R=mmenke@chromium.org

Review-Url: https://codereview.chromium.org/2860323002
Cr-Commit-Position: refs/heads/master@{#470211}
Committed: https://chromium.googlesource.com/chromium/src/+/dbf45973be46d9dd01f8d7d3dadddef0eeeaff06

[tfarina, 2017-05-05 01:25:55]

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

[commit-bot: I haz the power, 2017-05-05 01:26:28]

Dry run: CQ is trying da patch.

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

[tfarina, 2017-05-05 01:28:22]

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

[commit-bot: I haz the power, 2017-05-05 01:28:51]

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

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

[commit-bot: I haz the power, 2017-05-05 01:48:04]

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

[tfarina, 2017-05-05 02:06:59]

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

[commit-bot: I haz the power, 2017-05-05 02:07:22]

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-05-05 03:11:59]

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

[commit-bot: I haz the power, 2017-05-05 03:12:00]

Dry run: This issue passed the CQ dry run.

[mmenke, 2017-05-05 14:37:27]

LGTM!

[tfarina, 2017-05-05 16:31:46]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2017-05-05 16:33:05]

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-05-05 16:42:32]

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

[commit-bot: I haz the power, 2017-05-05 16:42:33]

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

[tfarina, 2017-05-05 17:09:48]

tfarina@chromium.org changed reviewers:
+ skyostil@chromium.org

[tfarina, 2017-05-05 17:09:49]

+skyostil for headless/ OWNERS.

[Sami, 2017-05-08 17:16:22]

headless/ lgtm, thank you!

[tfarina, 2017-05-08 17:21:31]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2017-05-08 17:22:12]

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-05-08 21:58:50]

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

[commit-bot: I haz the power, 2017-05-08 21:58:51]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[mmenke, 2017-05-08 22:01:03]

The CQ bit was checked by mmenke@chromium.org

[mmenke, 2017-05-08 22:01:20]

On 2017/05/08 21:58:51, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
>
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

Looks like flake.

[commit-bot: I haz the power, 2017-05-08 22:02:24]

CQ is trying da patch.

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

[tfarina, 2017-05-08 22:41:11]

The CQ bit was checked by tfarina@chromium.org

[tfarina, 2017-05-08 22:41:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
skyostil@chromium.org, mmenke@chromium.org
Link to the patchset: https://codereview.chromium.org/2860323002/#ps60001
(title: "REBASE")

[tfarina, 2017-05-08 22:42:49]

The CQ bit was checked by tfarina@chromium.org

[tfarina, 2017-05-08 22:42:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
skyostil@chromium.org, mmenke@chromium.org
Link to the patchset: https://codereview.chromium.org/2860323002/#ps80001
(title: "remove IsMethodSafe() from generic_url_request_job.cc")

[commit-bot: I haz the power, 2017-05-08 22:44:30]

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-05-09 00:21:55]

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

[commit-bot: I haz the power, 2017-05-09 00:21:57]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[tfarina, 2017-05-09 00:36:04]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2017-05-09 00:40:13]

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-05-09 04:36:12]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1494290164132870,
"parent_rev": "06f4ba114078db197f8b4a641145d2554d118d06", "commit_rev":
"dbf45973be46d9dd01f8d7d3dadddef0eeeaff06"}

[commit-bot: I haz the power, 2017-05-09 04:36:20]

Description was changed from

==========
net: add IsMethodSafe() method to HttpUtil

This consolidates the three copies of it in a single and accessible place.

BUG=675760
TEST=net_unittests
R=mmenke@chromium.org
==========

to

==========
net: add IsMethodSafe() method to HttpUtil

This consolidates the three copies of it in a single and accessible place.

BUG=675760
TEST=net_unittests
R=mmenke@chromium.org

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

[commit-bot: I haz the power, 2017-05-09 04:36:21]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/dbf45973be46d9dd01f8d7d3dadd...