[cros] Flip enable-html5-camera switch.

[cros] Flip enable-html5-camera switch.


BUG=122764
TEST=Manual


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148214

[Nikita (slow), 2012-07-24 17:59:19]

lgtm

[commit-bot: I haz the power, 2012-07-24 18:00:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ivankr@chromium.org/10810073/1

[commit-bot: I haz the power, 2012-07-24 18:00:39]

Presubmit check for 10810073-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/ui/webui/options2

[Ivan Korotkov, 2012-07-24 18:14:24]

Adding James for OWNERS approval

[James Hawkins, 2012-07-24 18:22:19]

LGTM with indentation nits.

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
File chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc
(right):

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc:71:
HasSwitch(switches::kDisableHtml5Camera)) {
Indentation is off here.  I believe it should be 4 spaces from the start of the
'('.

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc:145:
HasSwitch(switches::kDisableHtml5Camera)) {
Indentation.

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
File chrome/browser/ui/webui/options2/chromeos/change_picture_options_handler.cc
(right):

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
chrome/browser/ui/webui/options2/chromeos/change_picture_options_handler.cc:105:
HasSwitch(switches::kDisableHtml5Camera)) {
Indentation.

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
chrome/browser/ui/webui/options2/chromeos/change_picture_options_handler.cc:371:
HasSwitch(switches::kDisableHtml5Camera)) {
Indentation.

[Ivan Korotkov, 2012-07-24 20:11:57]

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
File chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc
(right):

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc:71:
HasSwitch(switches::kDisableHtml5Camera)) {
On 2012/07/24 18:22:19, James Hawkins wrote:
> Indentation is off here.  I believe it should be 4 spaces from the start of
the
> '('.

Oops, right. Emacs issues.

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
chrome/browser/ui/webui/chromeos/login/user_image_screen_handler.cc:145:
HasSwitch(switches::kDisableHtml5Camera)) {
On 2012/07/24 18:22:19, James Hawkins wrote:
> Indentation.

Done.

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
File chrome/browser/ui/webui/options2/chromeos/change_picture_options_handler.cc
(right):

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
chrome/browser/ui/webui/options2/chromeos/change_picture_options_handler.cc:105:
HasSwitch(switches::kDisableHtml5Camera)) {
On 2012/07/24 18:22:19, James Hawkins wrote:
> Indentation.

Done.

https://chromiumcodereview.appspot.com/10810073/diff/1/chrome/browser/ui/webu...
chrome/browser/ui/webui/options2/chromeos/change_picture_options_handler.cc:371:
HasSwitch(switches::kDisableHtml5Camera)) {
On 2012/07/24 18:22:19, James Hawkins wrote:
> Indentation.

Done.

[commit-bot: I haz the power, 2012-07-24 20:13:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ivankr@chromium.org/10810073/6002

[commit-bot: I haz the power, 2012-07-24 21:58:43]

Change committed as 148214