Update documentation for vpnProvider

Update documentation for vpnProvider

This CL updates the documentation for vpnProvider interface to
reflect the behavior of the API.

BUG=459609
TBR=kalman@chromium.org

Committed: https://crrev.com/64d88feb786ce07f5d40e5d66f15c6997a011bd4
Cr-Commit-Position: refs/heads/master@{#317343}

[kaliamoorthi, 2015-02-20 16:24:45]

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

[bartfab (slow), 2015-02-20 16:34:32]

https://codereview.chromium.org/941343002/diff/1/extensions/common/api/vpn_pr...
File extensions/common/api/vpn_provider.idl (right):

https://codereview.chromium.org/941343002/diff/1/extensions/common/api/vpn_pr...
extensions/common/api/vpn_provider.idl:25: // tunnel. By default all user
traffic is routed to the tunnel. This
1: You forgot to remove "By default all user traffic is routed to the tunnel."
2: Should we document that if an include and an exclude list with the same
length prefix match, the result is undefined?

[kaliamoorthi, 2015-02-20 16:52:45]

PTAL

[bartfab (slow), 2015-02-20 16:56:53]

https://codereview.chromium.org/941343002/diff/20001/extensions/common/api/vp...
File extensions/common/api/vpn_provider.idl (right):

https://codereview.chromium.org/941343002/diff/20001/extensions/common/api/vp...
extensions/common/api/vpn_provider.idl:25: // eliminated and the exact duplicate
entry that would be eliminated is
Nit: s/would/will/

https://codereview.chromium.org/941343002/diff/20001/extensions/common/api/vp...
extensions/common/api/vpn_provider.idl:29: // tunnel. This parameter can be used
to setup split tunnel. By default no
Nit: s/setup/set up a/

https://codereview.chromium.org/941343002/diff/20001/extensions/common/api/vp...
extensions/common/api/vpn_provider.idl:36: // eliminated and the exact duplicate
entry that would be eliminated is
Nit: s/would/will/

[kaliamoorthi, 2015-02-20 17:00:59]

The CQ bit was checked by kaliamoorthi@chromium.org

[kaliamoorthi, 2015-02-20 17:01:15]

On 2015/02/20 16:56:53, bartfab wrote:
>
https://codereview.chromium.org/941343002/diff/20001/extensions/common/api/vp...
> File extensions/common/api/vpn_provider.idl (right):
> 
>
https://codereview.chromium.org/941343002/diff/20001/extensions/common/api/vp...
> extensions/common/api/vpn_provider.idl:25: // eliminated and the exact
duplicate
> entry that would be eliminated is
> Nit: s/would/will/
> 
>
https://codereview.chromium.org/941343002/diff/20001/extensions/common/api/vp...
> extensions/common/api/vpn_provider.idl:29: // tunnel. This parameter can be
used
> to setup split tunnel. By default no
> Nit: s/setup/set up a/
> 
>
https://codereview.chromium.org/941343002/diff/20001/extensions/common/api/vp...
> extensions/common/api/vpn_provider.idl:36: // eliminated and the exact
duplicate
> entry that would be eliminated is
> Nit: s/would/will/

Thanks Bartosz, your comments are fixed.

[commit-bot: I haz the power, 2015-02-20 17:02:07]

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

[commit-bot: I haz the power, 2015-02-20 17:27:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-20 17:28:06]

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