Commit 871e1bd9 authored by Kevin Cernekee's avatar Kevin Cernekee Committed by Commit Bot

Disconnect ARC VPN if ARC++ is shut down

Currently, if ARC++ is uninstalled while an Android VPN is running,
shill will not receive a disconnection event and this will break
networking on Chrome OS.  Fix this by forcing ARC VPN to disconnect
whenever ArcNetHostImpl shuts down.

BUG=768053
TEST=uninstall ARC++ through Settings with a VPN connected

Change-Id: Iccbdc20bb26bce154f8041a517b1220519a2b738
Reviewed-on: https://chromium-review.googlesource.com/778340
Commit-Queue: Kevin Cernekee <cernekee@chromium.org>
Reviewed-by: default avatarSteven Bennetts <stevenjb@chromium.org>
Reviewed-by: default avatarElijah Taylor <elijahtaylor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521121}
parent 018d2622
......@@ -399,6 +399,10 @@ void ArcNetHostImpl::OnConnectionReady() {
}
void ArcNetHostImpl::OnConnectionClosed() {
// Make sure shill doesn't leave an ARC VPN connected after Android
// goes down.
AndroidVpnStateChanged(arc::mojom::ConnectionStateType::NOT_CONNECTED);
if (!observing_network_state_)
return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment