Add support for HTMLMediaElement::getStartDate()

Source/core/html/HTMLMediaElement.cpp

 /*
  * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 233 matching lines @@
     , m_asyncEventQueue(GenericEventQueue::create(this))
     , m_playbackRate(1.0f)
     , m_defaultPlaybackRate(1.0f)
     , m_networkState(NETWORK_EMPTY)
     , m_readyState(HAVE_NOTHING)
     , m_readyStateMaximum(HAVE_NOTHING)
     , m_volume(1.0f)
     , m_lastSeekTime(0)
     , m_previousProgressTime(numeric_limits<double>::max())
     , m_duration(numeric_limits<double>::quiet_NaN())
+    , m_timelineOffset(numeric_limits<double>::quiet_NaN())
     , m_lastTimeUpdateEventWallTime(0)
     , m_lastTimeUpdateEventMovieTime(numeric_limits<double>::max())
     , m_loadState(WaitingForSource)
     , m_webLayer(0)
     , m_preload(MediaPlayer::Auto)
     , m_displayMode(Unknown)
     , m_cachedTime(MediaPlayer::invalidTime())
     , m_cachedTimeWallClockUpdateTime(0)
     , m_minimumWallClockTimeToCacheMediaTime(0)
     , m_fragmentStartTime(MediaPlayer::invalidTime())
@@ skipping 405 matching lines @@
         //       If this changed the official playback position, then queue a task to fire a simple event named timeupdate at the media element.
         // FIXME: Add support for firing this event.
 
         // 4.8 - Set the initial playback position to 0.
         // FIXME: Make this less subtle. The position only becomes 0 because of the createMediaPlayer() call
         // above.
         refreshCachedTime();
         invalidateCachedTime();
 
         // 4.9 - Set the timeline offset to Not-a-Number (NaN).
+        m_timelineOffset = numeric_limits<double>::quiet_NaN();
+
         // 4.10 - Update the duration attribute to Not-a-Number (NaN).
 
 
         updateMediaController();
         updateActiveTextTrackCues(0);
     }
 
     // 5 - Set the playbackRate attribute to the value of the defaultPlaybackRate attribute.
     setPlaybackRate(defaultPlaybackRate());
 
@@ skipping 906 matching lines @@
             finishSeek();
     } else {
         if (wasPotentiallyPlaying && m_readyState < HAVE_FUTURE_DATA) {
             // 4.8.10.8
             scheduleTimeupdateEvent(false);
             scheduleEvent(EventTypeNames::waiting);
         }
     }
 
     if (m_readyState >= HAVE_METADATA && oldState < HAVE_METADATA) {
+        // This block corresponds to the "Once enough of the media data has been fetched to determine the duration of the media resource, its dimensions, and other metadata"
+        // section of step 4 of the resource fetch algorithm.
+
+        // FIXME: This appears to correspond to step 10 in the algorithm. Investigate moving this code to a position that matches the spec text.
         prepareMediaFragmentURI();
+
+        // 1. Establish the media timeline for the purposes of the current playback position, the earliest possible position, and the initial playback position, based on the media data.
+        // 2. Update the timeline offset to the date and time that corresponds to the zero time in the media timeline established in the previous step, if any. If no explicit time and date is given by the media resource, the timeline offset must be set to Not-a-Number (NaN).

[philipj_slow, 2014/05/06 11:13:52]

Nit: this is a very long line. There isn't a lot of consistency, but many spec
comments seem to be limited to 100 columns, so that would work.

+        m_timelineOffset = webMediaPlayer()->timelineOffset();
+
+        // 3. Set the current playback position and the official playback position to the earliest possible position.
+
+        // 4. Update the duration attribute with the time of the last frame of the resource, if known, on the media timeline established above. If it is not known (e.g. a stream that is in principle infinite), update the duration attribute to the value positive Infinity.
         scheduleEvent(EventTypeNames::durationchange);
+
+        // 5. For video elements, set the videoWidth and videoHeight attributes, and queue a task to fire a simple event named resize at the media element.
         if (isHTMLVideoElement(*this))
             scheduleEvent(EventTypeNames::resize);
+
+        // 6. Set the readyState attribute to HAVE_METADATA.
+        //   Note: A loadedmetadata DOM event will be fired as part of setting the readyState attribute to a new value.
         scheduleEvent(EventTypeNames::loadedmetadata);
+
+        // Steps 7-12 appear to be handled elsewhere.
+        // FIXME: Code should be refactored to make it easier to verify that these steps are implemented according to spec.
+
         if (hasMediaControls())
             mediaControls()->reset();
         if (renderer())
             renderer()->updateFromElement();
     }
 
     bool shouldUpdateDisplayState = false;
 
     if (m_readyState >= HAVE_CURRENT_DATA && oldState < HAVE_CURRENT_DATA && !m_haveFiredLoadedData) {
         m_haveFiredLoadedData = true;
@@ skipping 287 matching lines @@
     // implementation observing that the underlying engine has updated duration
     // and notifying the element to consult its MediaSource for current
     // duration. See http://crbug.com/266644
 
     if (m_mediaSource)
         return m_mediaSource->duration();
 
     return m_player->duration();
 }
 
+double HTMLMediaElement::getStartDate()
+{
+    return m_timelineOffset;

[philipj_slow, 2014/05/15 11:38:00]

My colleague has noticed that this value will be wrapped in a Date directly, so
it needs to be multiplied by 1000, probably here.

[acolwell GONE FROM CHROMIUM, 2014/05/15 13:01:22]

It is already supposed to be in milliseconds since
WebMediaPlayer::timelineOffset() that base::Time::ToJsTime() should be used to
generate this value. I've made a note to update the documentation on
m_timelineOffset.

[philipj_slow, 2014/05/15 13:28:29]

Thanks!

+}
+
 bool HTMLMediaElement::paused() const
 {
     return m_paused;
 }
 
 double HTMLMediaElement::defaultPlaybackRate() const
 {
     return m_defaultPlaybackRate;
 }
 
@@ skipping 1685 matching lines @@
 
 void HTMLMediaElement::trace(Visitor* visitor)
 {
     visitor->trace(m_textTracks);
     visitor->trace(m_textTracksWhenResourceSelectionBegan);
     Supplementable<HTMLMediaElement>::trace(visitor);
     HTMLElement::trace(visitor);
 }
 
 }