Remove unused was_fetched_via_proxy members

Remove unused was_fetched_via_proxy members

Committed: https://crrev.com/49714cce0664eadd15a9de6291b3b01d127156b4
Cr-Commit-Position: refs/heads/master@{#422123}

[Charlie Harrison, 2016-09-20 21:09:29]

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

[commit-bot: I haz the power, 2016-09-20 21:10:03]

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-09-20 21:22:38]

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

[commit-bot: I haz the power, 2016-09-20 21:22:39]

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

[Charlie Harrison, 2016-09-20 21:49:05]

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

[commit-bot: I haz the power, 2016-09-20 21:50:04]

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

[Charlie Harrison, 2016-09-20 22:55:12]

csharrison@chromium.org changed reviewers:
+ nasko@chromium.org

[Charlie Harrison, 2016-09-20 22:55:13]

If this patch causes runtime errors I will be terrified. So far so good...

Nasko, deleting unused code. yay or nay?

[commit-bot: I haz the power, 2016-09-20 23:27:20]

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

[commit-bot: I haz the power, 2016-09-20 23:27:22]

Dry run: This issue passed the CQ dry run.

[nasko, 2016-09-21 22:34:43]

All unused code removal is LGTM in my book :). I'd suggest pinging bengr@ just
in case, since they recently had a need to know whether something was fetched
through a proxy, but IIRC they went with a different design.

[Charlie Harrison, 2016-09-21 23:10:42]

csharrison@chromium.org changed reviewers:
+ bengr@chromium.org

[Charlie Harrison, 2016-09-21 23:10:42]

Thanks, +bengr in that case.

[tbansal1, 2016-09-23 15:39:00]

tbansal@chromium.org changed reviewers:
+ tbansal@chromium.org

[tbansal1, 2016-09-23 15:39:01]

drive-by-comment:

Thanks a ton for doing this. DRP definitely does not need proxy server on the
renderer side. As nasko said, we went with a different approach.

Also, it seems that now it is possible to remove the two variables (bool and
HostPortPair) from content/public/common/resource_response_info.h, 
content/common/resource_messages.h and resource_loader.cc.

We can do it in this CL or I can do it in the next one. Removing them would
definitely make my related CL (https://codereview.chromium.org/2334623003/) more
palatable :).

[Charlie Harrison, 2016-09-23 16:15:16]

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

[commit-bot: I haz the power, 2016-09-23 16:15:28]

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-09-23 18:00:36]

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

[commit-bot: I haz the power, 2016-09-23 18:00:37]

Dry run: This issue passed the CQ dry run.

[Charlie Harrison, 2016-09-23 18:01:49]

Thanks for the drive-by. Updated the CL per your request.

[tbansal1, 2016-09-23 18:05:05]

On 2016/09/23 18:01:49, Charlie Harrison wrote:
> Thanks for the drive-by. Updated the CL per your request.

Great. Thank you.

[bengr, 2016-09-23 18:23:19]

lgtm

[Charlie Harrison, 2016-09-23 18:27:32]

csharrison@chromium.org changed reviewers:
+ haraken@chromium.org

[Charlie Harrison, 2016-09-23 18:27:33]

Thanks. +haraken for platform changes.

[Charlie Harrison, 2016-09-23 18:28:38]

csharrison@chromium.org changed reviewers:
+ jochen@chromium.org
- haraken@chromium.org

[Charlie Harrison, 2016-09-23 18:28:39]

-haraken, +jochen for public and platform changes (simple).

[haraken, 2016-09-24 00:46:44]

On 2016/09/23 18:28:39, Charlie Harrison wrote:
> -haraken, +jochen for public and platform changes (simple).

LGTM

[jochen (gone - plz use gerrit), 2016-09-26 15:05:02]

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

[jochen (gone - plz use gerrit), 2016-09-26 15:05:03]

lgtm

[Charlie Harrison, 2016-09-26 15:06:12]

The CQ bit was checked by csharrison@chromium.org

[Charlie Harrison, 2016-09-26 15:06:12]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/2356013002/#ps40001
(title: "remove even more!")

[commit-bot: I haz the power, 2016-09-26 15:06:40]

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-09-26 15:46:24]

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

[commit-bot: I haz the power, 2016-09-26 15:46:26]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Charlie Harrison, 2016-09-30 14:32:30]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-09-30 14:32:42]

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-09-30 15:49:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-30 15:52:54]

Description was changed from

==========
Remove unused was_fetched_via_proxy members
==========

to

==========
Remove unused was_fetched_via_proxy members

Committed: https://crrev.com/49714cce0664eadd15a9de6291b3b01d127156b4
Cr-Commit-Position: refs/heads/master@{#422123}
==========

[commit-bot: I haz the power, 2016-09-30 15:52:56]

Patchset 3 (id:??) landed as
https://crrev.com/49714cce0664eadd15a9de6291b3b01d127156b4
Cr-Commit-Position: refs/heads/master@{#422123}