Need to know the frame calling MediaPermissionDispatcher::RequestPermission |
||||
Issue descriptionWebUserGestureIndicator::IsProcessingUserGesture needs to know the frame making the request. The frame parameter defaults to null, and currently MediaPermissionDispatcher::RequestPermission is the only one that doesn't pass a WebLocalFrame*.
,
Jun 22 2018
,
Sep 14
,
Sep 27
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8d866e5ff141efbd1f78cd5557a9cccb37995805 commit 8d866e5ff141efbd1f78cd5557a9cccb37995805 Author: Mustaq Ahmed <mustaq@google.com> Date: Thu Sep 27 22:09:00 2018 MediaPermissionDispatcher: Added a pointer to embedder frame for UAv2. Bug: 817027 Change-Id: I0a7e3fb3014017a07158225082aab19b51ccdf24 Reviewed-on: https://chromium-review.googlesource.com/1240199 Reviewed-by: Alex Moshchuk <alexmos@chromium.org> Reviewed-by: Yuri Wiitala <miu@chromium.org> Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Cr-Commit-Position: refs/heads/master@{#594892} [modify] https://crrev.com/8d866e5ff141efbd1f78cd5557a9cccb37995805/content/renderer/media/media_permission_dispatcher.cc [modify] https://crrev.com/8d866e5ff141efbd1f78cd5557a9cccb37995805/content/renderer/media/media_permission_dispatcher.h [modify] https://crrev.com/8d866e5ff141efbd1f78cd5557a9cccb37995805/content/renderer/render_frame_impl.cc [modify] https://crrev.com/8d866e5ff141efbd1f78cd5557a9cccb37995805/content/renderer/render_frame_impl.h
,
Sep 28
|
||||
►
Sign in to add a comment |
||||
Comment 1 by mustaq@chromium.org
, Feb 27 2018