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

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

Issue 14107010: Update tabCapture API docs with browser action info. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 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: chrome/common/extensions/api/tab_capture.idl
diff --git a/chrome/common/extensions/api/tab_capture.idl b/chrome/common/extensions/api/tab_capture.idl
index cfca548cc1372f8cfa219e87ad67a4d982f72c31..7d94369162f0d138f6e72ea2fb0d3466a29625fd 100644
--- a/chrome/common/extensions/api/tab_capture.idl
+++ b/chrome/common/extensions/api/tab_capture.idl
@@ -47,7 +47,11 @@ namespace tabCapture {
interface Functions {
// Captures the visible area of the currently active tab.
- // Extensions must have the "tabCapture" permission to use this method.
+ // Extensions must have the "tabCapture" permission to use this method. Use
not at google - send to devlin 2013/04/18 20:17:13 nit: "extensions must have the tabCapture permissi
justinlin 2013/04/18 21:06:44 Done.
+ // of this method for the currently active tab can only be granted
+ // through a user action. See:
not at google - send to devlin 2013/04/18 20:17:13 "User action" isn't quite accurate; it might imply
justinlin 2013/04/18 21:06:44 Done, something similar.
+ // http://developer.chrome.com/extensions/activeTab.html
not at google - send to devlin 2013/04/18 20:17:13 could you make this <a href="activeTab.html">acti
+ // Note that Chrome internal pages cannot be captured.
// |options| : Configures the returned media stream.
// |callback| : Callback with either the stream returned or null.
static void capture(CaptureOptions options,

Powered by Google App Engine
This is Rietveld 408576698