Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(845)

Unified Diff: extensions/common/api/networking_private.idl

Issue 2818593003: [CrOS Tether] Add tether network properties (battery percentage, carrier, and signal strength) to t… (Closed)
Patch Set: Tether networks connectable by default, and generated Closure compiler externs. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/common/api/networking_private.idl
diff --git a/extensions/common/api/networking_private.idl b/extensions/common/api/networking_private.idl
index 3cd81883fa9c173f84c50147485b3f8c30eddbab..fa1bc7eeb8e131dbb7a3e62c868191c3db974676 100644
--- a/extensions/common/api/networking_private.idl
+++ b/extensions/common/api/networking_private.idl
@@ -58,7 +58,7 @@ namespace networkingPrivate {
};
enum NetworkType {
- All, Cellular, Ethernet, VPN, Wireless, WiFi, WiMAX
+ All, Cellular, Ethernet, Tether, VPN, Wireless, WiFi, WiMAX
};
enum ProxySettingsType {
@@ -202,14 +202,14 @@ namespace networkingPrivate {
DOMString? CommonName;
DOMString? Locality;
DOMString? Organization;
- DOMString? OrganizationalUnit;
+ DOMString? OrganizationalUnit;
};
dictionary ManagedIssuerSubjectPattern {
ManagedDOMString? CommonName;
ManagedDOMString? Locality;
ManagedDOMString? Organization;
- ManagedDOMString? OrganizationalUnit;
+ ManagedDOMString? OrganizationalUnit;
};
dictionary CertificatePattern {
@@ -574,6 +574,18 @@ namespace networkingPrivate {
DOMString Authentication;
};
+ dictionary TetherProperties {
+ long? BatteryPercentage;
+ DOMString? Carrier;
stevenjb 2017/04/14 22:04:49 Isn't Carrier required? (no '?' for required prope
Kyle Horimoto 2017/04/17 21:05:11 Done.
+ long? SignalStrength;
+ };
+
+ dictionary ManagedTetherProperties {
+ long? BatteryPercentage;
stevenjb 2017/04/14 22:04:49 ManagedLong, etc.
Kyle Horimoto 2017/04/17 21:05:11 Done.
+ DOMString? Carrier;
+ long? SignalStrength;
+ };
+
dictionary VPNProperties {
boolean? AutoConnect;
DOMString? Host;
@@ -689,6 +701,7 @@ namespace networkingPrivate {
IPConfigProperties? StaticIPConfig;
IPConfigProperties? SavedIPConfig;
DOMString? Source;
+ TetherProperties? Tether;
NetworkType Type;
VPNProperties? VPN;
WiFiProperties? WiFi;
@@ -713,6 +726,7 @@ namespace networkingPrivate {
ManagedIPConfigProperties? StaticIPConfig;
IPConfigProperties? SavedIPConfig;
DOMString? Source;
+ ManagedTetherProperties? Tether;
NetworkType Type;
ManagedVPNProperties? VPN;
ManagedWiFiProperties? WiFi;
@@ -729,6 +743,7 @@ namespace networkingPrivate {
DOMString? Name;
long? Priority;
DOMString? Source;
+ TetherProperties? Tether;
NetworkType Type;
VPNStateProperties? VPN;
WiFiStateProperties? WiFi;

Powered by Google App Engine
This is Rietveld 408576698