Ignore ERROR_ADDRESS_NOT_ASSOCIATED from IpReleaseAddress in WiFiService::ResetDHCP.

Ignore ERROR_ADDRESS_NOT_ASSOCIATED from IpReleaseAddress in WiFiService::ResetDHCP. 
It is reported on Win 8.1 running on MS Surface Pro after connection to factory-reset Chromecast device.

BUG=366582
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266117

[mef, 2014-04-23 20:31:41]

Hi guys, please take a look. 
Toni, did you enter a crbug for this? 
Antonio, thanks for suggestion, I'm *very* happy that I didn't have to deal with
WMI.

[tbarzic, 2014-04-23 20:53:55]

I haven't created crbug.. feel free to do so..

lgtm

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_test.cc
File components/wifi/wifi_test.cc (right):

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_test.cc...
components/wifi/wifi_test.cc:61:
wifi_service_->GetProperties(network_guid_list[0], &properties, &error);
quit loop if the target network connection status != connecting?

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_test.cc...
components/wifi/wifi_test.cc:177: properties.Pass(),
indent

[afontan, 2014-04-23 21:04:16]

lgtm

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_service...
File components/wifi/wifi_service_win.cc (right):

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_service...
components/wifi/wifi_service_win.cc:908: LOG(ERROR) << error;
You are logging it twice as ResetDHCP already logs the error.

[mef, 2014-04-23 21:41:22]

Thanks! I'll wait for https://codereview.chromium.org/249683002/ to land and
then commit this one.

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_service...
File components/wifi/wifi_service_win.cc (right):

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_service...
components/wifi/wifi_service_win.cc:908: LOG(ERROR) << error;
On 2014/04/23 21:04:16, afontan wrote:
> You are logging it twice as ResetDHCP already logs the error.

Done.

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_test.cc
File components/wifi/wifi_test.cc (right):

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_test.cc...
components/wifi/wifi_test.cc:61:
wifi_service_->GetProperties(network_guid_list[0], &properties, &error);
On 2014/04/23 20:53:56, tbarzic wrote:
> quit loop if the target network connection status != connecting?

It also gets notified about previously connected network being disconnected.

https://codereview.chromium.org/250223002/diff/1/components/wifi/wifi_test.cc...
components/wifi/wifi_test.cc:177: properties.Pass(),
On 2014/04/23 20:53:56, tbarzic wrote:
> indent

Done.

[mef, 2014-04-24 11:27:35]

The CQ bit was checked by mef@chromium.org

[commit-bot: I haz the power, 2014-04-24 11:31:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mef@chromium.org/250223002/20001

[commit-bot: I haz the power, 2014-04-24 12:14:55]

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

[commit-bot: I haz the power, 2014-04-24 12:14:55]

Try jobs failed on following builders:
  tryserver.chromium on chromium_presubmit

[mef, 2014-04-24 14:24:58]

The CQ bit was checked by mef@chromium.org

[commit-bot: I haz the power, 2014-04-24 14:25:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mef@chromium.org/250223002/20001

[commit-bot: I haz the power, 2014-04-24 15:47:13]

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

[commit-bot: I haz the power, 2014-04-24 15:47:14]

Retried try job too often on android_clang_dbg for step(s) slave_steps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[mef, 2014-04-25 01:27:57]

The CQ bit was checked by mef@chromium.org

[commit-bot: I haz the power, 2014-04-25 01:33:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mef@chromium.org/250223002/20001

[commit-bot: I haz the power, 2014-04-25 03:19:18]

Change committed as 266117