Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(358)

Unified Diff: third_party/WebKit/Source/core/html/HTMLVideoElement.h

Issue 2767823002: Media Remoting: Add interstitial elements to media element shadow dom. (Closed)
Patch Set: Remove cast text message element. Not assuming remoting interstitial is media control. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/html/HTMLVideoElement.h
diff --git a/third_party/WebKit/Source/core/html/HTMLVideoElement.h b/third_party/WebKit/Source/core/html/HTMLVideoElement.h
index 385cc3a62fc01554d991c0381d50fc32f3ae21d9..6016c155b36615ef23bab7d0240bbd4fa934f9d1 100644
--- a/third_party/WebKit/Source/core/html/HTMLVideoElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLVideoElement.h
@@ -43,6 +43,7 @@ namespace blink {
class ExceptionState;
class ImageBitmapOptions;
class MediaCustomControlsFullscreenDetector;
+class MediaRemotingInterstitial;
class CORE_EXPORT HTMLVideoElement final : public HTMLMediaElement,
public CanvasImageSource,
@@ -130,6 +131,11 @@ class CORE_EXPORT HTMLVideoElement final : public HTMLMediaElement,
bool isPersistent() const;
+ bool isMediaRemotingStarted() const override {
+ return m_mediaRemotingStarted;
+ }
+ void disableMediaRemoting();
+
private:
friend class MediaCustomControlsFullscreenDetectorTest;
friend class HTMLMediaElementEventListenersTest;
@@ -155,9 +161,20 @@ class CORE_EXPORT HTMLVideoElement final : public HTMLMediaElement,
void didMoveToNewDocument(Document& oldDocument) override;
void setDisplayMode(DisplayMode) override;
+ void mediaRemotingStarted() final;
+ void mediaRemotingStopped() final;
+
+ bool isMediaRemotingDisabled() const override {
+ return m_mediaRemotingDisabled;
+ }
+
+ bool m_mediaRemotingDisabled : 1;
+ bool m_mediaRemotingStarted : 1;
mlamouri (slow - plz ping) 2017/04/07 13:18:33 Could a `m_mediaRemotingStatus` be used with Disab
xjz 2017/04/07 23:07:01 Good suggestion. Done. Thanks! :)
+
Member<HTMLImageLoader> m_imageLoader;
Member<MediaCustomControlsFullscreenDetector>
m_customControlsFullscreenDetector;
+ Member<MediaRemotingInterstitial> m_remotingInterstitial;
AtomicString m_defaultPosterURL;

Powered by Google App Engine
This is Rietveld 408576698