[webkit-reviews] review granted: [Bug 60132] media/controls-css-overload.html is timing out on chromium bots : [Attachment 95084] Patch

[bugzilla-daemon at webkit.org]

Eric Carlson <eric.carlson at apple.com> has granted Steve Lacey
<sjl at chromium.org>'s request for review:
Bug 60132: media/controls-css-overload.html is timing out on chromium bots
https://bugs.webkit.org/show_bug.cgi?id=60132

Attachment 95084: Patch
https://bugs.webkit.org/attachment.cgi?id=95084&action=review

------- Additional Comments from Eric Carlson <eric.carlson at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=95084&action=review

> LayoutTests/media/controls-after-reload.html:33
> +	   video.src = findMediaFile("video", "content/test");
> +	   video.load();

the video.load() is unnecessary and wasteful. load() used to be required after
changing src, but the spec was changed so setting src triggers the load
algorithm some time ago.