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

Side by Side Diff: chrome/common/extensions/docs/static/mediaGalleries.html

Issue 10821110: Rephrase some of the media galleries permission information. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/common/extensions/docs/samples.json ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <div id="pageData-name" class="pageData">Media Galleries</div> 1 <div id="pageData-name" class="pageData">Media Galleries</div>
2 2
3 <!-- BEGIN AUTHORED CONTENT --> 3 <!-- BEGIN AUTHORED CONTENT -->
4 <p> 4 <p>
5 The media galleries API allows you to access media files (images, 5 The media galleries API allows you to access media files (images,
6 video, audio) from the user's local disks (with the user's consent). 6 video, audio) from the user's local disks (with the user's consent).
7 </p> 7 </p>
8 8
9 <p> 9 <p>
10 Using the API, you can prompt the user for permission to access the media 10 Using the API, you can prompt the user for permission to access the media
11 galleries. The permission dialog will contain common media locations for 11 galleries. The permission dialog will contain common media locations for
12 the platform and will allow the user to add additional locations. From those 12 the platform and will allow the user to add additional locations. From those
13 locations, only media files will be present in the file system objects. 13 locations, only media files will be present in the file system objects.
14 </P> 14 </P>
15 15
16 <h2 id="manifest">Manifest</h2> 16 <h2 id="manifest">Manifest</h2>
17 <p>You must declare the type of access to the media gallery you require 17 <p>You must declare the type of access to the media gallery you require
18 in your extension's manifest to use the API. Currently only the 18 in your extension's manifest to use the API. Currently the only access
19 "mediaGalleriesRead" permission is allowed. This permission does not 19 type permission is "mediaGalleriesRead," which will allow your extension
20 currently trigger an install time permission prompt because it requires 20 to read from the media galleries. This permission does not trigger an
21 the user to grant permission at runtime.</p> 21 install time permission prompt because the user must still grant access
22 to particular galleries at runtime unless the "mediaGalleriesAllGalleries"
23 permission is used.</p>
battre 2012/07/31 21:34:15 So do you need to request both "mediaGalleriesRead
vandebo (ex-Chrome) 2012/07/31 21:36:06 To get default access to all well known galleries,
battre 2012/07/31 21:47:18 Sorry, let me phrase this differently. Do you have
22 24
23 <p> 25 <p>
24 You may also specify the "mediaGalleriesAllGalleries" permission to get 26 You may also specify the "mediaGalleriesAllGalleries" permission to get
25 access to all auto-detected media galleries on the user's computer. This 27 access to all auto-detected media galleries on the user's computer. This
26 permission displays an install time prompt indicating that the extension 28 permission displays an install time prompt indicating that the extension
27 will have access to all of the user's media files. 29 will have access to all of the user's media files.
28 </p> 30 </p>
battre 2012/08/01 09:35:47 Here is my proposal: <h2 id="manifest">Manifest</h
29 31
30 <!-- END AUTHORED CONTENT --> 32 <!-- END AUTHORED CONTENT -->
OLDNEW
« no previous file with comments | « chrome/common/extensions/docs/samples.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698