Removed some unused variables in Windows code.

Removed some unused variables in Windows code.

Note: Regarding the ICMP6_CLOSE_FUNC variable in winping.cc,
Icmp6CloseHandle does not exist, and IcmpCloseHandle is the correct way
to close an IPv6 handle. Therefore the existing code is correct to use
close_ on both types of connections and this variable is unnecessary.

BUG=505319

[Matt Giuca, 2015-07-03 01:58:27]

mgiuca@chromium.org changed reviewers:
+ sergeyu@chromium.org

[Sergey Ulanov, 2015-07-06 17:59:32]

lgtm

[Matt Giuca, 2015-07-07 00:13:03]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2015-07-07 00:13:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1214713005/1

[commit-bot: I haz the power, 2015-07-07 00:14:18]

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

[commit-bot: I haz the power, 2015-07-07 00:14:19]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_arm64_rel/build...)
  android_gn on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_gn/builds/4500)
  android_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_rel/builds/6471)
  ios on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios/builds/8376)
  ios64_sim on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios64_sim/builds/947)
  ios_arm64_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_arm64_rel/builds/3183)
  linux_msan on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/3536)
  linux_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_rel/builds/8233)
  mac on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac/builds/8532)
  mac_asan on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_asan/builds/8215)
  mac_x64_gn_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_x64_gn_rel/builds/2832)
  mac_x64_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_x64_rel/builds/8095)
  presubmit on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/104)
  win_x64_gn on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_gn/builds/2835)
  win_x64_gn_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_gn_rel/builds/2811)

(exceeded global retry quota)

[Matt Giuca, 2015-07-07 00:29:54]

The CQ bit was checked by mgiuca@chromium.org

[Matt Giuca, 2015-07-07 00:29:55]

The patchset sent to the CQ was uploaded after l-g-t-m from sergeyu@chromium.org
Link to the patchset: https://codereview.webrtc.org/1214713005/#ps20001 (title:
"Rebase.")

[commit-bot: I haz the power, 2015-07-07 00:30:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1214713005/20001

[commit-bot: I haz the power, 2015-07-07 00:30:56]

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

[commit-bot: I haz the power, 2015-07-07 00:30:57]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_arm64_rel/build...)
  ios32_sim on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios32_sim/builds/956)
  ios64_sim on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios64_sim/builds/949)
  ios_arm64_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_arm64_rel/builds/3185)
  linux_msan on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/3538)
  mac_asan on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_asan/builds/8217)
  mac_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_rel/builds/8364)
  mac_x64_gn_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_x64_gn_rel/builds/2834)
  mac_x64_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_x64_rel/builds/8097)
  presubmit on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/106)

(exceeded global retry quota)

[Matt Giuca, 2015-07-07 00:32:18]

Sergey: I keep getting patch failures on the try bots:

While running git apply --index -3 -p1 --verbose;
  Checking patch base/winping.cc...
  error: base/winping.cc: does not exist in index

I've just rebased against the latest head and base/winping.cc is definitely
present in the git repo. Is there some trick to this?

[Matt Giuca, 2015-07-08 03:15:17]

Patchset #3 (id:40001) has been deleted

[Matt Giuca, 2015-07-08 03:26:17]

Ping. I saw in your documentation that I'm to use "git cl land".

I tried that:
Fetching pending ref refs/pending/heads/master...
fatal: Couldn't find remote ref refs/pending/heads/master
fatal: Couldn't find remote ref refs/pending/heads/master

Is this just a permission problem? Not sure why it would say that. Can you
possibly land this for me or tell me how to land it?

[Sergey Ulanov, 2015-07-08 19:31:38]

On 2015/07/08 03:26:17, Matt Giuca wrote:
> Ping. I saw in your documentation that I'm to use "git cl land".
> 
> I tried that:
> Fetching pending ref refs/pending/heads/master...
> fatal: Couldn't find remote ref refs/pending/heads/master
> fatal: Couldn't find remote ref refs/pending/heads/master
> 
> Is this just a permission problem? Not sure why it would say that. Can you
> possibly land this for me or tell me how to land it?

I think the problem is that this CL is for
https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git, but  in the
main webrtc repo:
https://chromium.googlesource.com/external/webrtc.git . If I understand
correctly the ../webrtc/trunk/webrtc.git repo is a mirror of the webrtc
directory inside external/webrtc.git - heritage from when main webrtc code repo
was in SVN :-(. Can you please create a separate CL for
https://chromium.googlesource.com/external/webrtc.git ? See
http://www.webrtc.org/native-code/development for instructions on how to
checkout webrtc code outside of chromium.

[Matt Giuca, 2015-07-09 00:41:33]

Oh, I didn't notice the difference in the URLs :( This is confusing.

See:
https://codereview.webrtc.org/1231653003/