A video can be fullscreen with native controls or custom controls. We would like to detect when fullscreen of any sort happen. This bug is for the detection heuristics.
Issue 692098 has been merged into this issue.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12 commit abc082487c58110e72e8d8e9ff6fa0b7abc9dd12 Author: zqzhang <zqzhang@chromium.org> Date: Thu Mar 02 16:07:14 2017 [Blink>Media] Add heuristic for dominant video detection for Android This CL adds the heuristic for dominant video detection for Android, in which we use the fullscreen state of the document and the viewport intersection to determine if a video is the dominant content in fullscreen mode (as the page may fullscreen the embedding div instead of the video element). BUG= 695462 Review-Url: https://codereview.chromium.org/2696893002 Cr-Commit-Position: refs/heads/master@{#454272} [modify] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/media/blink/webmediaplayer_impl.cc [modify] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/media/blink/webmediaplayer_impl.h [modify] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/third_party/WebKit/Source/core/BUILD.gn [modify] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/third_party/WebKit/Source/core/html/BUILD.gn [modify] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/third_party/WebKit/Source/core/html/HTMLVideoElement.cpp [modify] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/third_party/WebKit/Source/core/html/HTMLVideoElement.h [add] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/third_party/WebKit/Source/core/html/MediaCustomControlsFullscreenDetector.cpp [add] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/third_party/WebKit/Source/core/html/MediaCustomControlsFullscreenDetector.h [add] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/third_party/WebKit/Source/core/html/MediaCustomControlsFullscreenDetectorTest.cpp [modify] https://crrev.com/abc082487c58110e72e8d8e9ff6fa0b7abc9dd12/third_party/WebKit/public/platform/WebMediaPlayer.h
Comment 1 by zqzh...@chromium.org
, Feb 24 2017