Title: [165859] trunk/Source/WebCore
Revision
165859
Author
[email protected]
Date
2014-03-18 17:38:01 -0700 (Tue, 18 Mar 2014)

Log Message

MediaDocument should set a max-width on its <video> element.
https://bugs.webkit.org/show_bug.cgi?id=114691

Reviewed by Eric Carlson.

Set a style attribute containing max-width:100% and max-height:100% so that large videos
are entirely visible within the browser window bounds.

* html/MediaDocument.cpp:
(WebCore::MediaDocumentParser::createDocumentStructure):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (165858 => 165859)


--- trunk/Source/WebCore/ChangeLog	2014-03-19 00:28:29 UTC (rev 165858)
+++ trunk/Source/WebCore/ChangeLog	2014-03-19 00:38:01 UTC (rev 165859)
@@ -1,3 +1,16 @@
+2014-03-18  Jer Noble  <[email protected]>
+
+        MediaDocument should set a max-width on its <video> element.
+        https://bugs.webkit.org/show_bug.cgi?id=114691
+
+        Reviewed by Eric Carlson.
+
+        Set a style attribute containing max-width:100% and max-height:100% so that large videos
+        are entirely visible within the browser window bounds.
+
+        * html/MediaDocument.cpp:
+        (WebCore::MediaDocumentParser::createDocumentStructure):
+
 2014-03-18  Thiago de Barros Lacerda  <[email protected]>
 
         [WebRTC] Fix layering violation in RTCStatsRequest

Modified: trunk/Source/WebCore/html/MediaDocument.cpp (165858 => 165859)


--- trunk/Source/WebCore/html/MediaDocument.cpp	2014-03-19 00:28:29 UTC (rev 165858)
+++ trunk/Source/WebCore/html/MediaDocument.cpp	2014-03-19 00:38:01 UTC (rev 165859)
@@ -91,6 +91,7 @@
     m_mediaElement->setAttribute(autoplayAttr, "");
 
     m_mediaElement->setAttribute(nameAttr, "media");
+    m_mediaElement->setAttribute(styleAttr, "max-width: 100%; max-height: 100%;");
 
     RefPtr<Element> sourceElement = document()->createElement(sourceTag, false);
     HTMLSourceElement& source = toHTMLSourceElement(*sourceElement);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to