Title: [161743] trunk/Source/WebCore
- Revision
- 161743
- Author
- [email protected]
- Date
- 2014-01-10 22:18:35 -0800 (Fri, 10 Jan 2014)
Log Message
Fix RenderObject.o for iOS. Unreviewed build fix.
r156285 renamed firstChild() to firstChildSlow(), so update
occurances in IOS_TEXT_AUTOSIZING code. Also account for a
RenderObject::style reference / pointer change.
* rendering/RenderObject.cpp:
(WebCore::RenderObject::traverseNext):
(WebCore::includeNonFixedHeight):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (161742 => 161743)
--- trunk/Source/WebCore/ChangeLog 2014-01-11 06:08:50 UTC (rev 161742)
+++ trunk/Source/WebCore/ChangeLog 2014-01-11 06:18:35 UTC (rev 161743)
@@ -1,5 +1,17 @@
2014-01-10 Joseph Pecoraro <[email protected]>
+ Fix RenderObject.o for iOS. Unreviewed build fix.
+
+ r156285 renamed firstChild() to firstChildSlow(), so update
+ occurances in IOS_TEXT_AUTOSIZING code. Also account for a
+ RenderObject::style reference / pointer change.
+
+ * rendering/RenderObject.cpp:
+ (WebCore::RenderObject::traverseNext):
+ (WebCore::includeNonFixedHeight):
+
+2014-01-10 Joseph Pecoraro <[email protected]>
+
Fix MediaPlayerPrivateMediaSourceAVFObjC.o for iOS. Unreviewed build fix.
Import CALayer, which Mac must have been getting some other way.
Modified: trunk/Source/WebCore/rendering/RenderObject.cpp (161742 => 161743)
--- trunk/Source/WebCore/rendering/RenderObject.cpp 2014-01-11 06:08:50 UTC (rev 161742)
+++ trunk/Source/WebCore/rendering/RenderObject.cpp 2014-01-11 06:18:35 UTC (rev 161743)
@@ -289,9 +289,10 @@
// Inspired by Node::traverseNextNode.
RenderObject* RenderObject::traverseNext(const RenderObject* stayWithin) const
{
- if (firstChild()) {
- ASSERT(!stayWithin || firstChild()->isDescendantOf(stayWithin));
- return firstChild();
+ RenderObject* child = firstChildSlow();
+ if (child) {
+ ASSERT(!stayWithin || child->isDescendantOf(stayWithin));
+ return child;
}
if (this == stayWithin)
return 0;
@@ -315,7 +316,7 @@
BlockContentHeightType overflowType;
// Check for suitable children.
- for (RenderObject* child = firstChild(); child; child = child->nextSibling()) {
+ for (RenderObject* child = firstChildSlow(); child; child = child->nextSibling()) {
overflowType = inclusionFunction(child);
if (overflowType != FixedHeight) {
currentDepth++;
@@ -359,7 +360,7 @@
RenderObject* RenderObject::traverseNext(const RenderObject* stayWithin, TraverseNextInclusionFunction inclusionFunction) const
{
- for (RenderObject* child = firstChild(); child; child = child->nextSibling()) {
+ for (RenderObject* child = firstChildSlow(); child; child = child->nextSibling()) {
if (inclusionFunction(child)) {
ASSERT(!stayWithin || child->isDescendantOf(stayWithin));
return child;
@@ -398,18 +399,16 @@
static RenderObject::BlockContentHeightType includeNonFixedHeight(const RenderObject* render)
{
- RenderStyle* style = render->style();
- if (style) {
- if (style->height().type() == Fixed) {
- if (render->isRenderBlock()) {
- const RenderBlock* block = toRenderBlock(render);
- // For fixed height styles, if the overflow size of the element spills out of the specified
- // height, assume we can apply text auto-sizing.
- if (style->overflowY() == OVISIBLE && style->height().value() < block->layoutOverflowRect().maxY())
- return RenderObject::OverflowHeight;
- }
- return RenderObject::FixedHeight;
+ const RenderStyle& style = render->style();
+ if (style.height().type() == Fixed) {
+ if (render->isRenderBlock()) {
+ const RenderBlock* block = toRenderBlock(render);
+ // For fixed height styles, if the overflow size of the element spills out of the specified
+ // height, assume we can apply text auto-sizing.
+ if (style.overflowY() == OVISIBLE && style.height().value() < block->layoutOverflowRect().maxY())
+ return RenderObject::OverflowHeight;
}
+ return RenderObject::FixedHeight;
}
return RenderObject::FlexibleHeight;
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
