[CrOS Tether] Create the ActiveHost class, which stores metadata related to the current connection …

chromeos/components/tether/pref_names.h

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROMEOS_COMPONENTS_TETHER_PREF_NAMES_H_
 #define CHROMEOS_COMPONENTS_TETHER_PREF_NAMES_H_
 
 namespace chromeos {
 
 namespace tether {
 
 namespace prefs {
 
 // Preference name for the preference which stores IDs corresponding to devices
 // which have most recently replied to a TetherAvailabilityRequest with a
 // response code indicating that tethering is available. The value stored is a
 // ListValue, with the most recent response residing at the start of the list.
 extern const char kMostRecentTetherAvailablilityResponderIds[];
 
 // Preference name for the preference which stores the ID corresponding to
 // the device which has most recently replied to a ConnectTetheringRequest with
 // a response code indicating that its hotspot has started up successfully.
 extern const char kMostRecentConnectTetheringResponderId[];
 
+// The status of the active host. The value stored for this key is the integer
+// version of an ActiveHost::ActiveHostStatus enumeration value.
+extern const char kActiveHostStatus[];
+
+// The device ID of the active host. If there is no active host, the value at

[Ryan Hansberry, 2017/03/09 19:18:20]

value of

[Kyle Horimoto, 2017/03/09 19:29:28]

I think you're misreading this. The value of this key is a constant and can
never be "". The value in prefs at this key is the thing that can be "".

[Ryan Hansberry, 2017/03/09 19:48:20]

Oops :)

+// this key is "".
+extern const char kActiveHostDeviceId[];
+
+// The network ID of the active host. If there is no active host, the value at

[Ryan Hansberry, 2017/03/09 19:18:20]

value of

[Kyle Horimoto, 2017/03/09 19:29:28]

Same.

+// this key is "".
+extern const char kTetherNetworkId[];

[Ryan Hansberry, 2017/03/09 19:18:20]

In the network stack, there will simultaneously be the tether network (the
visible network in the UI) and the wifi network (the real network that is
hidden). Because this network_id is referring to the wifi network, please
instead name it kWifiNetworkId. Please also apply this to the rest of the CL.

[Kyle Horimoto, 2017/03/09 19:29:28]

Done.

+
 }  // namespace prefs
 
 }  // namespace tether
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_COMPONENTS_TETHER_PREF_NAMES_H_