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

Unified Diff: Source/modules/netinfo/NetworkInfoConnection.idl

Issue 291203002: Adds NetInfo v3 Web API to Blink. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@netinfo1
Patch Set: Simplified layout tests Created 6 years, 7 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: Source/modules/netinfo/NetworkInfoConnection.idl
diff --git a/Source/modules/netinfo/NetworkInfoConnection.idl b/Source/modules/netinfo/NetworkInfoConnection.idl
new file mode 100644
index 0000000000000000000000000000000000000000..99380fc0c8204d265cc80678d5576fb6a2561359
--- /dev/null
+++ b/Source/modules/netinfo/NetworkInfoConnection.idl
@@ -0,0 +1,24 @@
+// Copyright 2014 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.
+
+enum NetworkInfoConnectionType {
adamk 2014/05/27 21:44:27 This name doesn't match the spec (which just says
jkarlin 2014/05/28 13:31:30 Done.
+ "cellular",
+ "bluetooth",
+ "ethernet",
+ "wifi",
+ "other",
+ "none"
+};
+
+[
+ RuntimeEnabled=NetworkInfo,
+ Exposed=Window&Worker,
adamk 2014/05/27 21:44:27 I don't see any worker support or tests in this ch
jkarlin 2014/05/28 13:31:30 Done.
+ WillBeGarbageCollected,
+ ActiveDOMObject
+]
+interface NetworkInfoConnection : EventTarget {
adamk 2014/05/27 21:44:27 Nit: I think this line should be merged into the p
adamk 2014/05/27 21:44:27 The spec calls this "NetworkInformation", why the
jkarlin 2014/05/28 13:31:30 Done.
jkarlin 2014/05/28 13:31:30 Done.
+ readonly attribute NetworkInfoConnectionType type;
+ attribute EventHandler ontypechange;
+};
+

Powered by Google App Engine
This is Rietveld 408576698