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

Unified Diff: chrome/common/extensions/api/location.idl

Issue 19763008: Adding dev documentation for chrome.location (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 5 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: chrome/common/extensions/api/location.idl
diff --git a/chrome/common/extensions/api/location.idl b/chrome/common/extensions/api/location.idl
index 0900c8fc8024b19a1de599599a9ad67e9e958aca..c9896de9bc98c48a742d520cd7f3880c0ddfd18d 100644
--- a/chrome/common/extensions/api/location.idl
+++ b/chrome/common/extensions/api/location.idl
@@ -3,9 +3,8 @@
// found in the LICENSE file.
// TODO(vadimt): Consider reusing WebKit/Blink types, if this is possible.
-// The <code>chrome.location</code> API augments the events from the
-// <a href="http://dev.w3.org/geo/api/spec-source.html">HTML Geolocation API</a>
-// to be event page compatible.
+// Use the <code>chrome.location</code> API to retrieve geographical location of
+// the hosting machine.
not at google - send to devlin 2013/07/27 01:51:49 i preferred the old description
vadimt 2013/07/27 02:08:55 That was a pretty head-scratchy description for th
Jeffrey Yasskin 2013/07/27 03:00:29 Can we combine them? I agree that your new descrip
not at google - send to devlin 2013/07/29 14:45:12 Yes exactly. This was going to be my next comment.
vadimt 2013/07/30 01:58:41 Done. Probably, a similar comment should be done f
namespace location {
// Coordinates part of the Location object.
dictionary Coordinates {

Powered by Google App Engine
This is Rietveld 408576698