Title: [151460] trunk/Source/WebCore
Revision
151460
Author
jer.no...@apple.com
Date
2013-06-11 12:58:28 -0700 (Tue, 11 Jun 2013)

Log Message

Set the font size for in-band captions on the display box, not the :cue.
https://bugs.webkit.org/show_bug.cgi?id=117515

Reviewed by Eric Carlson.

Make the in-band font size behavior match the out-of-band behavior.

* html/track/TextTrackCueGeneric.cpp:
(WebCore::TextTrackCueGeneric::setFontSize):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (151459 => 151460)


--- trunk/Source/WebCore/ChangeLog	2013-06-11 19:53:06 UTC (rev 151459)
+++ trunk/Source/WebCore/ChangeLog	2013-06-11 19:58:28 UTC (rev 151460)
@@ -1,3 +1,15 @@
+2013-06-11  Jer Noble  <jer.no...@apple.com>
+
+        Set the font size for in-band captions on the display box, not the :cue.
+        https://bugs.webkit.org/show_bug.cgi?id=117515
+
+        Reviewed by Eric Carlson.
+
+        Make the in-band font size behavior match the out-of-band behavior.
+
+        * html/track/TextTrackCueGeneric.cpp:
+        (WebCore::TextTrackCueGeneric::setFontSize):
+
 2013-06-11  Brent Fulgham  <bfulg...@apple.com>
 
         [Windows] Implement 'attributeValue' accessor to support testing.

Modified: trunk/Source/WebCore/html/track/TextTrackCueGeneric.cpp (151459 => 151460)


--- trunk/Source/WebCore/html/track/TextTrackCueGeneric.cpp	2013-06-11 19:53:06 UTC (rev 151459)
+++ trunk/Source/WebCore/html/track/TextTrackCueGeneric.cpp	2013-06-11 19:58:28 UTC (rev 151460)
@@ -147,7 +147,7 @@
     double size = videoSize.height() * baseFontSizeRelativeToVideoHeight() / 100;
     if (fontSizeMultiplier())
         size *= fontSizeMultiplier() / 100;
-    element()->setInlineStyleProperty(CSSPropertyFontSize, String::number(lround(size)) + "px");
+    displayTreeInternal()->setInlineStyleProperty(CSSPropertyFontSize, String::number(lround(size)) + "px");
 
     LOG(Media, "TextTrackCueGeneric::setFontSize - setting cue font size to %li", lround(size));
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to