Suppress network configuration UI when third-party VPN times out

Suppress network configuration UI when third-party VPN times out

BUG=470835
Committed: https://crrev.com/f9f1551951609a9f99ae2272c5a692a402e87a6e
Cr-Commit-Position: refs/heads/master@{#322397}

[emaxx, 2015-03-26 13:29:16]

emaxx@chromium.org changed reviewers:
+ kaliamoorthi@chromium.org

[kaliamoorthi, 2015-03-26 13:32:57]

On 2015/03/26 13:29:44, emaxx wrote:

LGTM, Lets see what stevenjb@ says.

[kaliamoorthi, 2015-03-26 13:33:10]

kaliamoorthi@chromium.org changed reviewers:
+ stevenjb@chromium.org

[stevenjb, 2015-03-26 15:38:02]

lgtm w/ nits

https://codereview.chromium.org/1034923002/diff/1/ui/chromeos/network/network...
File ui/chromeos/network/network_connect.cc (right):

https://codereview.chromium.org/1034923002/diff/1/ui/chromeos/network/network...
ui/chromeos/network/network_connect.cc:151: // Third-party VPNs are supposed to
do the UI itself.
nit: Third-party VPNs handle configuration UI themselves.

https://codereview.chromium.org/1034923002/diff/1/ui/chromeos/network/network...
ui/chromeos/network/network_connect.cc:154: }
nit: no brackets

[emaxx, 2015-03-26 15:56:37]

On 2015/03/26 15:38:02, stevenjb wrote:
> lgtm w/ nits
> 
>
https://codereview.chromium.org/1034923002/diff/1/ui/chromeos/network/network...
> File ui/chromeos/network/network_connect.cc (right):
> 
>
https://codereview.chromium.org/1034923002/diff/1/ui/chromeos/network/network...
> ui/chromeos/network/network_connect.cc:151: // Third-party VPNs are supposed
to
> do the UI itself.
> nit: Third-party VPNs handle configuration UI themselves.
> 
>
https://codereview.chromium.org/1034923002/diff/1/ui/chromeos/network/network...
> ui/chromeos/network/network_connect.cc:154: }
> nit: no brackets

Thanks Steven.

[emaxx, 2015-03-26 15:57:06]

The CQ bit was checked by emaxx@chromium.org

[emaxx, 2015-03-26 15:57:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
stevenjb@chromium.org, kaliamoorthi@chromium.org
Link to the patchset: https://codereview.chromium.org/1034923002/#ps20001
(title: "Style fixes")

[commit-bot: I haz the power, 2015-03-26 15:57:21]

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

[commit-bot: I haz the power, 2015-03-26 16:26:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-26 16:27:40]

Patchset 2 (id:??) landed as
https://crrev.com/f9f1551951609a9f99ae2272c5a692a402e87a6e
Cr-Commit-Position: refs/heads/master@{#322397}