Remove check of security origin of all ancestors when getUserMedia is called.

chrome/browser/media/media_stream_devices_controller.cc

 // Copyright (c) 2012 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/media_stream_devices_controller.h"
 
 #include "base/metrics/histogram.h"
 #include "base/prefs/scoped_user_pref_update.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
@@ skipping 405 matching lines @@
              content::MEDIA_DEVICE_NO_HARDWARE : content::MEDIA_DEVICE_OK,
          ui.Pass());
 }
 
 void MediaStreamDevicesController::Deny(
     bool update_content_setting,
     content::MediaStreamRequestResult result) {
   DLOG(WARNING) << "MediaStreamDevicesController::Deny: " << result;
   NotifyUIRequestDenied();
 
-  if (update_content_setting && request_.all_ancestors_have_same_origin) {
-    // Store sticky permissions if |update_content_setting| and the request
-    // is not done from an iframe where the ancestor has a different origin.
+  if (update_content_setting) {
+    // Store sticky permissions if |update_content_setting|.

[tommi (sloooow) - chröme, 2015/03/03 11:43:21]

nit: I don't think this comment adds much

     CHECK_EQ(content::MEDIA_DEVICE_PERMISSION_DENIED, result);
     StorePermission(false);
   }
 
   content::MediaResponseCallback cb = callback_;
   callback_.Reset();
   cb.Run(content::MediaStreamDevices(),
          result,
          scoped_ptr<content::MediaStreamUI>());
 }
@@ skipping 54 matching lines @@
   UMA_HISTOGRAM_ENUMERATION("Media.DevicePermissionActions",
                             kCancel, kPermissionActionsMax);
   Deny(false, content::MEDIA_DEVICE_PERMISSION_DISMISSED);
 }
 
 void MediaStreamDevicesController::RequestFinished() {
   delete this;
 }
 
 bool MediaStreamDevicesController::IsRequestAllowedByDefault() const {
-  // If not all ancestors of the requesting frame have the same origin, do not
-  // allow the request per default.
-  if (!request_.all_ancestors_have_same_origin)
-    return false;
-
   // The request from internal objects like chrome://URLs is always allowed.
   if (CheckAllowAllMediaStreamContentForOrigin(profile_,
                                                request_.security_origin)) {
     return true;
   }
 
   struct {
     bool has_capability;
     const char* policy_name;
     const char* list_policy_name;
@@ skipping 165 matching lines @@
 }
 
 bool MediaStreamDevicesController::IsCaptureDeviceRequestAllowed() const {
 #if defined(OS_ANDROID)
   // Don't approve device requests if the tab was hidden.
   // TODO(qinmin): Add a test for this. http://crbug.com/396869.
   return web_contents_->GetRenderWidgetHostView()->IsShowing();
 #endif
   return true;
 }