Rename MediaHasVideo bit to MediaCanToggleControls.

Only set this bit for non-fullscreen video, since toggling is ignored for fullscreen video.

Chromium side change is https://codereview.chromium.org/201573013.

BUG=179489

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

git-svn-id: svn://svn.chromium.org/blink/trunk@170701 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/web/ContextMenuClientImpl.cpp

@@ -242,8 +242,12 @@ void ContextMenuClientImpl::showContextMenu(const WebCore::ContextMenu* defaultM
             data.mediaFlags |= WebContextMenuData::MediaCanSave;
         if (mediaElement->hasAudio())
             data.mediaFlags |= WebContextMenuData::MediaHasAudio;
-        if (mediaElement->hasVideo())
-            data.mediaFlags |= WebContextMenuData::MediaHasVideo;
+        // 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. Don't set this bit for fullscreen video, since
+        // toggling is ignored in that case.
+        if (mediaElement->hasVideo() && !mediaElement->isFullscreen())
+            data.mediaFlags |= WebContextMenuData::MediaCanToggleControls;
         if (mediaElement->controls())
             data.mediaFlags |= WebContextMenuData::MediaControls;
     } else if (isHTMLObjectElement(*r.innerNonSharedNode()) || isHTMLEmbedElement(*r.innerNonSharedNode())) {

third_party/WebKit/public/web/WebContextMenuData.h

@@ -100,7 +100,9 @@ struct WebContextMenuData {
         MediaLoop = 0x8,
         MediaCanSave = 0x10,
         MediaHasAudio = 0x20,
+        // FIXME: Remove once Chromium references MediaCanToggleControls.
         MediaHasVideo = 0x40,
+        MediaCanToggleControls = 0x40,
         MediaControls = 0x80,
         MediaCanPrint = 0x100,
         MediaCanRotate = 0x200,