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

Unified Diff: Source/core/html/shadow/MediaControls.cpp

Issue 823993004: Document the structure of the Media controls DOM (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Drop minor incorrectness. Created 5 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/shadow/MediaControls.cpp
diff --git a/Source/core/html/shadow/MediaControls.cpp b/Source/core/html/shadow/MediaControls.cpp
index 4c693c5c3e338a5f0f0062a53912d7e09f8d151a..8e57c27ebbdd23ffafd2b8d80f8b1f797bf99df7 100644
--- a/Source/core/html/shadow/MediaControls.cpp
+++ b/Source/core/html/shadow/MediaControls.cpp
@@ -81,6 +81,30 @@ PassRefPtrWillBeRawPtr<MediaControls> MediaControls::create(HTMLMediaElement& me
return nullptr;
}
+// The media controls DOM structure looks like:
+//
+// MediaControls (-webkit-media-controls)
+// +-MediaControlOverlayEnclosureElement (-webkit-media-controls-overlay-enclosure)
+// | +-MediaControlTextTrackContainerElement (-webkit-media-text-track-container)
+// | | {when text tracks are enabled}
+// | +-MediaControlOverlayPlayButtonElement (-webkit-media-controls-overlay-play-button)
+// | | {if mediaControlsOverlayPlayButtonEnabled}
+// | \-MediaControlCastButtonElement (-internal-media-controls-overlay-cast-button)
+// \-MediaControlPanelEnclosureElement (-webkit-media-controls-enclosure)
+// \-MediaControlPanelElement (-webkit-media-controls-panel)
+// +-MediaControlPlayButtonElement (-webkit-media-controls-play-button)
+// +-MediaControlTimelineElement (-webkit-media-controls-timeline)
+// +-MediaControlCurrentTimeDisplayElement (-webkit-media-controls-current-time-display)
+// +-MediaControlTimeRemainingDisplayElement (-webkit-media-controls-time-remaining-display)
+// +-MediaControlMuteButtonElement (-webkit-media-controls-mute-button)
+// +-MediaControlVolumeSliderElement (-webkit-media-controls-volume-slider)
+// +-MediaControlToggleClosedCaptionsButtonElement (-webkit-media-controls-toggle-closed-captions-button)
+// +-MediaControlCastButtonElement (-internal-media-controls-cast-button)
+// \-MediaControlFullscreenButtonElement (-webkit-media-controls-fullscreen-button)
+//
+// Most of the structure is built by MediaControls::initializeControls() - the
+// exception being MediaControlTextTrackContainerElement which is added
+// on-demand by MediaControls::createTextTrackDisplay().
bool MediaControls::initializeControls()
{
TrackExceptionState exceptionState;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698