Commit bb0d3d40 authored by sandersd@chromium.org's avatar sandersd@chromium.org

Rename MediaHasVideo bit to MediaCanToggleControls.

Blink side change is https://codereview.chromium.org/200723010/.

BUG=179489

Review URL: https://codereview.chromium.org/201573013

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@261528 0039d316-1c4b-4281-b951-d872f2087c98
parent 9d5dc41a
...@@ -1253,12 +1253,9 @@ bool RenderViewContextMenu::IsCommandIdEnabled(int id) const { ...@@ -1253,12 +1253,9 @@ bool RenderViewContextMenu::IsCommandIdEnabled(int id) const {
(params_.media_flags & (params_.media_flags &
WebContextMenuData::MediaInError) == 0; WebContextMenuData::MediaInError) == 0;
// Media controls can be toggled only for video player. If we toggle
// controls for audio then the player disappears, and there is no way to
// return it back.
case IDC_CONTENT_CONTEXT_CONTROLS: case IDC_CONTENT_CONTEXT_CONTROLS:
return (params_.media_flags & return (params_.media_flags &
WebContextMenuData::MediaHasVideo) != 0; WebContextMenuData::MediaCanToggleControls) != 0;
case IDC_CONTENT_CONTEXT_ROTATECW: case IDC_CONTENT_CONTEXT_ROTATECW:
case IDC_CONTENT_CONTEXT_ROTATECCW: case IDC_CONTENT_CONTEXT_ROTATECCW:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment