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

Issue 12211065: Add --enable-usermedia-screen-capturing in about:flags (Closed)

Created:
7 years, 10 months ago by Sergey Ulanov
Modified:
7 years, 10 months ago
Reviewers:
Wez
CC:
chromium-reviews
Visibility:
Public.

Description

Add --enable-usermedia-screen-capturing in about:flags Then new flag allows to enable screen capture support for getUserMedia(). BUG=134249 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181405

Patch Set 1 #

Total comments: 8

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -0 lines) Patch
M chrome/app/generated_resources.grd View 1 2 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/browser/about_flags.cc View 1 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
Sergey Ulanov
7 years, 10 months ago (2013-02-07 02:41:53 UTC) #1
Wez
LGTM https://chromiumcodereview.appspot.com/12211065/diff/1/chrome/app/generated_resources.grd File chrome/app/generated_resources.grd (right): https://chromiumcodereview.appspot.com/12211065/diff/1/chrome/app/generated_resources.grd#newcode7153 chrome/app/generated_resources.grd:7153: <message name="IDS_FLAGS_ENABLE_SCREEN_CAPTURING_NAME" desc="Name of the flag to enable ...
7 years, 10 months ago (2013-02-07 22:20:08 UTC) #2
Sergey Ulanov
https://chromiumcodereview.appspot.com/12211065/diff/1/chrome/app/generated_resources.grd File chrome/app/generated_resources.grd (right): https://chromiumcodereview.appspot.com/12211065/diff/1/chrome/app/generated_resources.grd#newcode7153 chrome/app/generated_resources.grd:7153: <message name="IDS_FLAGS_ENABLE_SCREEN_CAPTURING_NAME" desc="Name of the flag to enable screen ...
7 years, 10 months ago (2013-02-08 00:12:54 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sergeyu@chromium.org/12211065/2003
7 years, 10 months ago (2013-02-08 00:13:39 UTC) #4
Wez
lgtm https://chromiumcodereview.appspot.com/12211065/diff/1/chrome/app/generated_resources.grd File chrome/app/generated_resources.grd (right): https://chromiumcodereview.appspot.com/12211065/diff/1/chrome/app/generated_resources.grd#newcode7157 chrome/app/generated_resources.grd:7157: Allow the web pages to request screen capturing ...
7 years, 10 months ago (2013-02-08 00:54:55 UTC) #5
Sergey Ulanov
7 years, 10 months ago (2013-02-08 02:02:59 UTC) #6
Sergey Ulanov
7 years, 10 months ago (2013-02-08 02:03:15 UTC) #7
https://chromiumcodereview.appspot.com/12211065/diff/1/chrome/app/generated_r...
File chrome/app/generated_resources.grd (right):

https://chromiumcodereview.appspot.com/12211065/diff/1/chrome/app/generated_r...
chrome/app/generated_resources.grd:7157: Allow the web pages to request screen
capturing using getUserMedia() API.
On 2013/02/08 00:54:55, Wez wrote:
> On 2013/02/08 00:12:54, sergeyu wrote:
> > On 2013/02/07 22:20:08, Wez wrote:
> > > Suggest "Allow web pages to capture the screen contents via the
> getUserMedia()
> > > API."
> > 
> > Done, though with this wording some people may freak out - one may assume
that
> > access is given to any page automatically when this flag is enabled.
> 
> "Allow web pages to request access to the screen contents via the
> getUserMedia()" then?

Done.

Powered by Google App Engine
This is Rietveld 408576698