Chromium Code Reviews| Index: content/browser/browser_plugin/browser_plugin_guest.cc |
| diff --git a/content/browser/browser_plugin/browser_plugin_guest.cc b/content/browser/browser_plugin/browser_plugin_guest.cc |
| index e8085777e249815b371503fcf3b6050ab4f303ba..0d673dc3b1618c8eabe894916d03d624a3aa7ca9 100644 |
| --- a/content/browser/browser_plugin/browser_plugin_guest.cc |
| +++ b/content/browser/browser_plugin/browser_plugin_guest.cc |
| @@ -107,39 +107,6 @@ class BrowserPluginGuest::DownloadRequest : public PermissionRequest { |
| base::Callback<void(bool)> callback_; |
| }; |
| -class BrowserPluginGuest::MediaRequest : public PermissionRequest { |
| - public: |
| - MediaRequest(const base::WeakPtr<BrowserPluginGuest>& guest, |
| - const MediaStreamRequest& request, |
| - const MediaResponseCallback& callback) |
| - : PermissionRequest(guest), |
| - request_(request), |
| - callback_(callback) { |
| - RecordAction( |
| - base::UserMetricsAction("BrowserPlugin.Guest.PermissionRequest.Media")); |
| - } |
| - |
| - virtual void RespondImpl(bool should_allow, |
| - const std::string& user_input) OVERRIDE { |
| - WebContentsImpl* web_contents = guest_->embedder_web_contents(); |
| - if (should_allow && web_contents) { |
| - // Re-route the request to the embedder's WebContents; the guest gets the |
| - // permission this way. |
| - web_contents->RequestMediaAccessPermission(request_, callback_); |
| - } else { |
| - // Deny the request. |
| - callback_.Run(MediaStreamDevices(), |
| - MEDIA_DEVICE_INVALID_STATE, |
| - scoped_ptr<MediaStreamUI>()); |
| - } |
| - } |
| - |
| - private: |
| - virtual ~MediaRequest() {} |
| - MediaStreamRequest request_; |
| - MediaResponseCallback callback_; |
| -}; |
| - |
| class BrowserPluginGuest::NewWindowRequest : public PermissionRequest { |
| public: |
| NewWindowRequest(const base::WeakPtr<BrowserPluginGuest>& guest, |
| @@ -1644,16 +1611,14 @@ void BrowserPluginGuest::RequestMediaAccessPermission( |
| WebContents* web_contents, |
| const MediaStreamRequest& request, |
| const MediaResponseCallback& callback) { |
| - base::DictionaryValue request_info; |
| - request_info.Set( |
| - browser_plugin::kURL, |
| - base::Value::CreateStringValue(request.security_origin.spec())); |
| + if (!delegate_) { |
| + callback.Run(MediaStreamDevices(), |
| + MEDIA_DEVICE_INVALID_STATE, |
|
lazyboy
2014/04/11 20:13:20
indent
Fady Samuel
2014/04/11 21:52:31
Done.
|
| + scoped_ptr<MediaStreamUI>()); |
| + return; |
| + } |
| - RequestPermission(BROWSER_PLUGIN_PERMISSION_TYPE_MEDIA, |
| - new MediaRequest(weak_ptr_factory_.GetWeakPtr(), |
| - request, |
| - callback), |
| - request_info); |
| + delegate_->RequestMediaAccessPermission(request, callback); |
| } |
| bool BrowserPluginGuest::PreHandleGestureEvent( |