Bug 132443

Summary: CSS-based Media Controls Show Different times content longer than 1 hour.
Product: WebKit Reporter: Jeremy Jones <jeremyj-wk>
Component: MediaAssignee: Jeremy Jones <jeremyj-wk>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, calvaris, commit-queue, eric.carlson, glenn, jeremyj-wk, jer.noble, jonlee, philipj, rniwa, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: iPhone / iPad   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from webkit-ews-05 for mac-mountainlion
none
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2
none
Archive of layout-test-results from webkit-ews-07 for mac-mountainlion
none
Patch none

Description Jeremy Jones 2014-05-01 15:41:11 PDT
CSS-based Media Controls Show Different times content longer than 1 hour.
Comment 1 Jeremy Jones 2014-05-01 15:46:22 PDT
Created attachment 230619 [details]
Patch
Comment 2 Jer Noble 2014-05-01 15:53:13 PDT
Comment on attachment 230619 [details]
Patch

Nice! r=me.
Comment 3 Build Bot 2014-05-01 17:09:40 PDT
Comment on attachment 230619 [details]
Patch

Attachment 230619 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5643440008200192

New failing tests:
media/controls-without-preload.html
media/video-no-audio.html
media/controls-strict.html
media/audio-controls-rendering.html
media/controls-after-reload.html
Comment 4 Build Bot 2014-05-01 17:09:44 PDT
Created attachment 230630 [details]
Archive of layout-test-results from webkit-ews-05 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-05  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 5 Build Bot 2014-05-01 17:48:04 PDT
Comment on attachment 230619 [details]
Patch

Attachment 230619 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6218291852869632

New failing tests:
media/controls-without-preload.html
media/video-no-audio.html
media/controls-strict.html
media/audio-controls-rendering.html
Comment 6 Build Bot 2014-05-01 17:48:07 PDT
Created attachment 230636 [details]
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-09  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 7 Build Bot 2014-05-01 18:16:56 PDT
Comment on attachment 230619 [details]
Patch

Attachment 230619 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/6237186487746560

New failing tests:
media/controls-without-preload.html
media/video-no-audio.html
media/controls-strict.html
media/audio-controls-rendering.html
media/controls-after-reload.html
Comment 8 Build Bot 2014-05-01 18:16:59 PDT
Created attachment 230639 [details]
Archive of layout-test-results from webkit-ews-07 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-07  Port: mac-mountainlion  Platform: Mac OS X 10.8.5
Comment 9 Jeremy Jones 2014-05-02 10:13:56 PDT
Created attachment 230667 [details]
Patch
Comment 10 Jeremy Jones 2014-05-02 10:15:35 PDT
(In reply to comment #2)
> (From update of attachment 230619 [details])
> Nice! r=me.

One more small change to pass tests. Mac always wants two digit minutes.
Comment 11 WebKit Commit Bot 2014-05-02 13:08:48 PDT
Comment on attachment 230667 [details]
Patch

Rejecting attachment 230667 [details] from commit-queue.

jeremyj-wk@apple.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 12 WebKit Commit Bot 2014-05-02 14:43:20 PDT
Comment on attachment 230667 [details]
Patch

Rejecting attachment 230667 [details] from commit-queue.

jeremyj-wk@apple.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 13 WebKit Commit Bot 2014-05-02 15:58:31 PDT
Comment on attachment 230667 [details]
Patch

Rejecting attachment 230667 [details] from commit-queue.

jeremyj-wk@apple.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 14 WebKit Commit Bot 2014-05-02 16:28:16 PDT
Comment on attachment 230667 [details]
Patch

Rejecting attachment 230667 [details] from commit-queue.

jeremyj-wk@apple.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 15 WebKit Commit Bot 2014-05-02 18:12:59 PDT
Comment on attachment 230667 [details]
Patch

Clearing flags on attachment: 230667

Committed r168215: <http://trac.webkit.org/changeset/168215>
Comment 16 WebKit Commit Bot 2014-05-02 18:13:07 PDT
All reviewed patches have been landed.  Closing bug.
Comment 17 Xabier Rodríguez Calvar 2014-05-05 00:13:35 PDT
Sorry for the late comment, but I think it would have been awesome to have a test about this.
Comment 18 Jon Lee 2014-05-05 08:24:47 PDT
<rdar://problem/16173777>