Merge branch 'bug-fix/video-scrollbar' into develop

[SVCS-671]
Closes: #318

Author: cslzchen

mfr/extensions/video/templates/viewer.mako

@@ -1,3 +1,11 @@
+<style>
+    ## Setting margin and padding to 0 for body fixes the scrollbar flickering issue
+    body {
+        margin: 0;
+        padding: 0;
+    }
+</style>
+
 <video controls height="100%">
   <source src="${url}">
   Your browser does not support the video tag.
@@ -7,4 +15,4 @@
 <script src="/static/js/mfr.child.js"></script>
 <script>
     window.pymChild.sendMessage('embed', 'embed-responsive-16by9');
-</script>
+</script>

tests/extensions/video/test_renderer.py

@@ -1,13 +1,13 @@
 import pytest
 
 from mfr.core.provider import ProviderMetadata
-
 from mfr.extensions.video import VideoRenderer
 
 
 @pytest.fixture
 def metadata():
-    return ProviderMetadata('test', '.mp4', 'text/plain', '1234', 'http://wb.osf.io/file/test.mp4?token=1234')
+    return ProviderMetadata('test', '.mp4', 'text/plain', '1234',
+                            'http://wb.osf.io/file/test.mp4?token=1234')
 
 
 @pytest.fixture
@@ -41,6 +41,7 @@ def test_render_video(self, renderer, url):
         body = renderer.render()
         assert '<video controls' in body
         assert 'src="{}"'.format(metadata().download_url) in body
+        assert '<style>body{margin:0;padding:0;}</style>' in ''.join(body.split())
 
     def test_render_video_file_required(self, renderer):
         assert renderer.file_required is False