Fix media element time display for times over an hour.

Fix media element time display for times over an hour.

Moves a misplaced modulo so that hours can become nonzero when constructing the time and duration strings for HTMLMediaElement.

Also added a unit test for it.

This is the post-merge version of https://codereview.chromium.org/1360203002/

BUG=535046
Committed: https://crrev.com/4025bad149997db1a957651f63e3b82389c23d1c
Cr-Commit-Position: refs/heads/master@{#351055}

[liberato (no reviews please), 2015-09-25 16:24:53]

liberato@chromium.org changed reviewers:
+ philipj@opera.com

[liberato (no reviews please), 2015-09-25 16:25:59]

this is the post-blink-merge version of
https://codereview.chromium.org/1360203002/ PS4.

thanks
-fl

[DaleCurtis, 2015-09-25 17:40:03]

dalecurtis@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[DaleCurtis, 2015-09-25 17:40:04]

lgtm % for() fix.

https://codereview.chromium.org/1369763002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/LayoutThemeTest.cpp (right):

https://codereview.chromium.org/1369763002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutThemeTest.cpp:132: for (unsigned i =
0; i < sizeof(tests) / sizeof(tests[0]); i++)  {
for (const auto& expectation : tests)

[philipj_slow, 2015-09-28 08:55:53]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-09-28 08:55:53]

lgtm

[commit-bot: I haz the power, 2015-09-28 08:56:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1369763002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1369763002/20001

[commit-bot: I haz the power, 2015-09-28 11:17:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-28 11:18:52]

Patchset 2 (id:??) landed as
https://crrev.com/4025bad149997db1a957651f63e3b82389c23d1c
Cr-Commit-Position: refs/heads/master@{#351055}