Chrome OS: Fix taking user pictures to be used as user POD images.

chrome/browser/media/webrtc/media_permission.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/media/webrtc/media_permission.h"
 
 #include "chrome/browser/media/webrtc/media_capture_devices_dispatcher.h"
 #include "chrome/browser/media/webrtc/media_stream_device_permissions.h"
 #include "chrome/browser/permissions/permission_context_base.h"
 #include "chrome/browser/permissions/permission_manager.h"
@@ skipping 51 matching lines @@
   chromeos::WebUILoginView* webui_login_view =
       login_display_host ? login_display_host->GetWebUILoginView() : nullptr;
   content::WebContents* login_web_contents =
       webui_login_view ? webui_login_view->GetWebContents() : nullptr;
   if (web_contents_ == login_web_contents) {
     if (content_type_ == CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC) {
       *denial_reason = content::MEDIA_DEVICE_PERMISSION_DENIED;
       return CONTENT_SETTING_BLOCK;
     }
 
+    // When creating new user (including supervised user), we must
+    // be able to use photo for user image.
+    constexpr char pattern[] = "chrome://oobe/";

[Sergey Ulanov, 2017/03/29 23:26:17]

Use kChromeUIOobeURL from chrome/common/url_constants.h here?
If it can't be used here then this const should have a const-style name, also
it's an origin, not a pattern (kOobeOrigin would be a better name).

[Alexander Alekseev, 2017/03/30 02:04:34]

Done.

+    if (requesting_origin_.possibly_invalid_spec() == pattern) {

[Sergey Ulanov, 2017/03/29 23:26:17]

use spec() instead of possibly_invalid_spec() here?

[Alexander Alekseev, 2017/03/30 02:04:34]

Done.

+      return CONTENT_SETTING_ALLOW;
+    }
+
     const chromeos::CrosSettings* const settings =
         chromeos::CrosSettings::Get();
     if (!settings) {
       *denial_reason = content::MEDIA_DEVICE_PERMISSION_DENIED;
       return CONTENT_SETTING_BLOCK;
     }
 
     const base::Value* const raw_list_value =
         settings->GetPref(chromeos::kLoginVideoCaptureAllowedUrls);
     if (!raw_list_value) {
@@ skipping 26 matching lines @@
   // Check policy and content settings.
   ContentSetting content_setting =
       permission_manager
           ->GetPermissionStatus(content_type_, requesting_origin_,
                                 embedding_origin_)
           .content_setting;
   if (content_setting == CONTENT_SETTING_BLOCK)
     *denial_reason = content::MEDIA_DEVICE_PERMISSION_DENIED;
   return content_setting;
 }