Title: [168775] tags/Safari-538.34.4/Source/WebCore
Diff
Modified: tags/Safari-538.34.4/Source/WebCore/ChangeLog (168774 => 168775)
--- tags/Safari-538.34.4/Source/WebCore/ChangeLog 2014-05-14 03:39:52 UTC (rev 168774)
+++ tags/Safari-538.34.4/Source/WebCore/ChangeLog 2014-05-14 03:43:42 UTC (rev 168775)
@@ -1,5 +1,21 @@
2014-05-13 Babak Shafiei <[email protected]>
+ Merge r168742
+
+ 2014-05-13 Brady Eidson <[email protected]>
+
+ Followup to: Update positioning/drawing of the image controls button.
+ <rdar://problem/16885077> and https://bugs.webkit.org/show_bug.cgi?id=132883
+
+ Reviewed by Tim Horton.
+
+ Cleared up the actual intent behind review feedback on the original patch.
+
+ * html/shadow/mac/ImageControlsButtonElementMac.cpp:
+ (WebCore::ImageControlsButtonElementMac::maybeCreate):
+
+2014-05-13 Babak Shafiei <[email protected]>
+
Merge r168728
2014-05-13 Brady Eidson <[email protected]>
Modified: tags/Safari-538.34.4/Source/WebCore/html/shadow/mac/ImageControlsButtonElementMac.cpp (168774 => 168775)
--- tags/Safari-538.34.4/Source/WebCore/html/shadow/mac/ImageControlsButtonElementMac.cpp 2014-05-14 03:39:52 UTC (rev 168774)
+++ tags/Safari-538.34.4/Source/WebCore/html/shadow/mac/ImageControlsButtonElementMac.cpp 2014-05-14 03:43:42 UTC (rev 168775)
@@ -98,10 +98,10 @@
button->setAttribute(HTMLNames::classAttr, "x-webkit-image-controls-button");
IntSize positionOffset = document.page()->theme().imageControlsButtonPositionOffset();
- button->setInlineStyleProperty(CSSPropertyTop, String::format("%ipx", positionOffset.height()));
+ button->setInlineStyleProperty(CSSPropertyTop, positionOffset.height(), CSSPrimitiveValue::CSS_PX);
// FIXME: Why is right: 0px off the right edge of the parent?
- button->setInlineStyleProperty(CSSPropertyRight, String::format("%ipx", positionOffset.width()));
+ button->setInlineStyleProperty(CSSPropertyRight, positionOffset.width(), CSSPrimitiveValue::CSS_PX);
return button.release();
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
