Description[CrOS Tether] Update NetworkState to include tether properties and integrate into NetworkStateHandler.
This CL:
(1) Adds tether properties (battery percentage, carrier, signal strength) to NetworkState.
(2) Updates NetworkStateHandler::AddTetherNetworkState() to include parameters for these properties, and adds an UpdateTetherNetworkProperties() function to update these values at a later time.
(3) Correctly returns tether networks for NetworkStateHandler's getter functions: ConnectedNetworkByType(), ConnectingNetworkByType(), FirstNetworkByType(), GetNetworkListByType()
(4) Updates the tether component's HostScanner to pass tether properties to NetworkStateHandler when scan results come in.
With this CL, the chrome.networkingPrivate API successfully returns tether networks!
BUG=672263
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
Review-Url: https://codereview.chromium.org/2819383002
Cr-Commit-Position: refs/heads/master@{#466085}
Committed: https://chromium.googlesource.com/chromium/src/+/71c16b98c68a2b66290b656da91cd1992664655e
Patch Set 1 #Patch Set 2 : Comments. #
Total comments: 1
Patch Set 3 : Fix comments, add style change, change a LOG to a DCHECK. #
Total comments: 31
Patch Set 4 : hansberry@ comments. #Patch Set 5 : Update TODOs. #
Total comments: 20
Patch Set 6 : Rebased. #Patch Set 7 : Rebased. #
Total comments: 8
Patch Set 8 : hansberry@ comments. #Patch Set 9 : Made GetTetherNetworkList() public again. #Patch Set 10 : Remove unused function. #Patch Set 11 : stevenjb@ and hansberry@ comments. #
Total comments: 10
Patch Set 12 : stevenjb@ comments. #Patch Set 13 : stevenjb@ comments. #
Total comments: 2
Patch Set 14 : stevenjb@ comments. #
Total comments: 2
Patch Set 15 : stevenjb@ comment. #Messages
Total messages: 24 (6 generated)
|