Last Comment Bug 847376 - Implement HTMLMediaElement.getStartDate()
: Implement HTMLMediaElement.getStartDate()
Status: NEW
: dev-doc-needed, html5
Product: Core
Classification: Components
Component: DOM (show other bugs)
: Trunk
: All All
: -- normal with 3 votes (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
Depends on: 742206
Blocks: html5
  Show dependency treegraph
 
Reported: 2013-03-04 05:56 PST by Andrea Marchesini (:baku)
Modified: 2013-10-19 11:48 PDT (History)
6 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Impl (3.46 KB, patch)
2013-10-08 20:23 PDT, Andrew Quartey [:drexler]
no flags Details | Diff | Review
Impl (2.82 KB, patch)
2013-10-08 20:27 PDT, Andrew Quartey [:drexler]
cpearce: review-
Details | Diff | Review

Description Andrea Marchesini (:baku) 2013-03-04 05:56:10 PST

    
Comment 1 Boris Zbarsky [:bz] 2013-03-04 07:12:17 PST
It's "  readonly attribute Date startDate;", no?

We should just add Date to WebIDL; we need it for HTMLInputElement anyway.
Comment 2 Andrew Quartey [:drexler] 2013-10-08 20:21:29 PDT
This got changed to a method...see: http://html5.org/tools/web-apps-tracker?from=8112&to=8113
Comment 3 Andrew Quartey [:drexler] 2013-10-08 20:23:45 PDT
Created attachment 814690 [details] [diff] [review]
Impl
Comment 4 Andrew Quartey [:drexler] 2013-10-08 20:27:34 PDT
Created attachment 814691 [details] [diff] [review]
Impl

Forgot to qref.
Comment 5 Chris Pearce (:cpearce) 2013-10-13 18:17:41 PDT
Comment on attachment 814691 [details] [diff] [review]
Impl

Review of attachment 814691 [details] [diff] [review]:
-----------------------------------------------------------------

I don't think fragment start time is what you want here. What you need here is the timestamp which is typically stored in the metadata. We don't expose that yet, and it would be a bunch of work to get to it for all the formats we support, so I suggest we leave it for now.

We'd need a mochitest for this too.

Additionally, do any other browsers implements this feature? If no other browser does, I'm inclined to direct your efforts elsewhere first. I tested Chrome and IE on Windows, and they don't support HTMLMediaElement.startDate().
Comment 6 Andrew Quartey [:drexler] 2013-10-19 11:48:51 PDT
Thanks for the review. 

> I don't think fragment start time is what you want here. What you need here
> is the timestamp which is typically stored in the metadata. We don't expose
> that yet, and it would be a bunch of work to get to it for all the formats
> we support...

Perhaps it'll be best to ping you on IRC to know what to look for in exposing the timestamps from the metadata and how to go about it. This is just to get more familiar with the media formats. 
 
> Additionally, do any other browsers implements this feature? 
None upon checking.

Note You need to log in before you can comment on or make changes to this bug.